DAC960 Bitrot

[Daniel Phillips <phillips@arcor.de>]

On Thursday 11 July 2002 12:08, Jens Axboe wrote:
> On Thu, Jul 11 2002, Daniel Phillips wrote:
> > On Thursday 11 July 2002 08:47, Jens Axboe wrote:
> > > Leonard has promised me to convert DAC960 to the "new" pci dma api for
> > > years (or so it seems, actual date may vary, no purchase necessary). I
> > > do have a Mylex controller here myself these days, so it's not
> > > completely impossible that I may do it on a rainy day.
> > 
> > Well, tell me what the new api is and I'll dive in there.  For the record,
> 
> Documentation/DMA-mapping.txt. Also, DAC960 initial bio conversion
> happened before the interface was finalized, so it may need changes in
> that regard as well. Documentation/block/biodoc.txt is your friend there
> :-)
> 
> a quick make drivers/block/DAC960.o shows the following stuff needs
> changing immediately:
> 
> 1) q->queue_lock is a pointer to a lock, not the lock itself. Probably
> add a per-controller spinlock to DAC960_Controller_T, and pass that to
> blk_init_queue(). Then change DAC960_AcquireControllerLock and friends
> in DAC960.h accordingly.

The big change here appears to be the move to per-device request queues.  
Somebody apparently already started to update this driver (you?) but 
obviously didn't try to compile it.  This is new territory for me, so I'll be 
moving gingerly in here for a while.

For those locks, I just removed the &'s, which seems like the right thing to 
do.   The "Controller" lock really seems to be a request queue lock.  Now I 
think I need to allocate and initialize a request queue, possibly in 
DAC960_CreateAuxiliaryStructures.  Am I getting warm?

-- 
Daniel