Re: [PATCH resend] bsg: Add support for submitting requests at tail of queue

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 24 Mar 2009 13:14:38 +0100
Jens Axboe <jens.axboe@xxxxxxxxxx> wrote:

> On Tue, Mar 24 2009, FUJITA Tomonori wrote:
> > On Tue, 24 Mar 2009 10:41:32 +0200
> > Boaz Harrosh <bharrosh@xxxxxxxxxxx> wrote:
> > 
> > > Boaz Harrosh wrote:
> > > > Hi Tomo Jens
> > > > 
> > > > Tomo you never ack-by on this patch. I absolutely needs this for the
> > > > user-mode API of osd-initiator. Which is needed with up-coming exofs
> > > > utilities.
> > > > 
> > > > What do you want to do?
> > > > 
> > > > Thanks 
> > > > 
> > > 
> > > Hi Jens.
> > > 
> > > I absolutely need this patch for 2.6.30 merge window. It is totally
> > > un-dangerous since defaults are left unchanged.
> > 
> > The question is we really need this feature or not. Though I guess
> > that we need to address this starvation issue.
> 
> I'd argue that tail insertion should be the most used way to queue
> commands, head insertion should only be for the cases where you want
> immediate access (like error handling).

As I wrote, I think that we need to address this starvation issue.

But nobody has complained about this (with sg and bsg) for years
because we have not seen applications sending scsi commands that are
many enough to make a queue full. So it's worth knowing what unknown
applications do for what, I think.
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux