Re: [PATCH] New QStor SATA/RAID Driver for 2.6.9-rc2

Mark Lord
2004-09-14T18:12:45Z

>The SCSI LLD API really needs to -not- spinlock on the EH hooks,
>and instead simply guarantee that ->queuecommand and other hooks
>will not be called while the driver is in EH.
>
>ISTR James didn't disagree, so maybe a patch can be worked out...

It looks to me as if the eh code prevents further queuecommand()
calls while the LLD *_reset_handler() code is running.
I wonder if it also does so for the eh_strategy_handler() ?

Have to look at it, I guess.

>Of course, you could always just use ->eh_strategy_handler
>and do 100% of the error handling yourself.

Mmmm.. yes, that may be better, perhaps.

Whatever this driver does, it has to be reasonably portable
back to early 2.4.xx kernels, so it cannot depend too much
upon newly (or to-be) implemented semantics in 2.6.xx.

Cheers
--
Mark Lord
(hdparm keeper & the original "Linux IDE Guy")