All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <jens.axboe@oracle.com>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: bharrosh@panasas.com, linux-scsi@vger.kernel.org,
	osd-dev@open-osd.org, dgilbert@interlog.com
Subject: Re: [PATCH resend] bsg: Add support for submitting requests at tail of queue
Date: Tue, 24 Mar 2009 13:26:09 +0100	[thread overview]
Message-ID: <20090324122608.GR27476@kernel.dk> (raw)
In-Reply-To: <20090324212248A.fujita.tomonori@lab.ntt.co.jp>

On Tue, Mar 24 2009, FUJITA Tomonori wrote:
> On Tue, 24 Mar 2009 13:14:38 +0100
> Jens Axboe <jens.axboe@oracle.com> wrote:
> 
> > On Tue, Mar 24 2009, FUJITA Tomonori wrote:
> > > On Tue, 24 Mar 2009 10:41:32 +0200
> > > Boaz Harrosh <bharrosh@panasas.com> 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.

Not sure it exists, just saying that if I were to write an app that uses
bsg for doing "normal" IO, I would certainly use tail insertion.

-- 
Jens Axboe


  reply	other threads:[~2009-03-24 12:26 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-19  9:13 [PATCH resend] bsg: Add support for submitting requests at tail of queue Boaz Harrosh
2009-03-24  8:41 ` Boaz Harrosh
2009-03-24 11:23   ` Jens Axboe
2009-03-24 12:05   ` FUJITA Tomonori
2009-03-24 12:14     ` Jens Axboe
2009-03-24 12:22       ` FUJITA Tomonori
2009-03-24 12:26         ` Jens Axboe [this message]
2009-03-24 12:53     ` Boaz Harrosh
2009-03-25  4:06       ` FUJITA Tomonori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20090324122608.GR27476@kernel.dk \
    --to=jens.axboe@oracle.com \
    --cc=bharrosh@panasas.com \
    --cc=dgilbert@interlog.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-scsi@vger.kernel.org \
    --cc=osd-dev@open-osd.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.