Subject: Re: more new scsi midlayer questions
To: Matthew Jacob <mjacob@feral.com>
From: Manuel Bouyer <bouyer@antioche.lip6.fr>
List: tech-kern
Date: 05/24/2001 21:04:47
On Wed, May 23, 2001 at 11:07:43AM -0700, Matthew Jacob wrote:
> [...]
> 
> > Also I think that the commit you did about negative freeze count should go
> > away. When something freeze a channel, the same thing should thaw it, with
> > the same count.
> 
> I don't agree. Theoretically, things should be paired. However, in case of
> mistakes, I still prefer my system to run.

At last a diag should be printed. If this happens this means the queue could
run before before everyone wants it to run, and this has to be fixed.

> 
> > For example, in your case you want to freeze the channel only one time, but
> > from different places. So you just don't want to freeze a channel again if
> > it's already frozen by the HBA driver. Keep a flag in the HBA driver, test
> > it to know if it's already frozen. This way the freeze count of the HBA
> > driver will always be 0 or 1, and you need to thaw it only with a count of 1.
> 
> Well, I don't much agree with all of this. It's not very practial, IMO. If you
> have to keep a flag or a count in the HBA driver, you might as well queue in
> the HBA driver.

It's much easier to set/clear a flag, than to manage a queue !

--
Manuel Bouyer, LIP6, Universite Paris VI.           Manuel.Bouyer@lip6.fr
--