lkml.org 
[lkml]   [2004]   [Oct]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] QStor SATA/RAID driver for 2.6.9-rc3
James Bottomley wrote:
>
> We use this already for other entities that require user context like
> domain validation. It seems to work as advertised.

Okay, done.

>>>scsi_done() doesn't require a lock
>>
>>Really? I wonder why the mid-layer is so religious about
>>doing the lock around every invocation of it today?
>
> It's not if you look at other drivers.

Well, I was actually looking at scsi.c, where this kind of thing is common:

spin_lock_irqsave(host->host_lock, flags);
scsi_done(cmd);
spin_unlock_irqrestore(host->host_lock, flags);

If those locks are not needed, the scsi.c maintainer really should
nuke'em. Meanwhile, I'll take them out of qstor.c as well, thanks.

> And you've tested this with things like SUSE's hwinfo utility which
> seems to send INQUIRIES on its own?

Not yet, but I have tested them with other scsiinfo-like tools.
Again, when it is proven to be an issue with something, it will get fixed.

> Really, I suppose, libata should provide the interfaces for doing this
> work for emulated commands.

Well, after this driver submission work is done with,
that's next on my list. Right now libata doesn't have
the right interface for easy sharing of such functions.

And the driver will need it's own copy in some versions anyway,
since this driver will be used on much older/earlier kernels
than just the ones with the latest libata stuff.

Cheers
--
Mark Lord
(hdparm keeper & the original "Linux IDE Guy")
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:06    [W:0.136 / U:0.352 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site