All of lore.kernel.org
 help / color / mirror / Atom feed
From: Walker, Benjamin <benjamin.walker at intel.com>
To: spdk@lists.01.org
Subject: Re: [SPDK] Problem with Blobstore when write 65MB continously
Date: Wed, 10 Jan 2018 17:17:16 +0000	[thread overview]
Message-ID: <1515604634.6063.43.camel@intel.com> (raw)
In-Reply-To: CANvN+e=0WFUQPchg+KebVNT+4hrcee7yq9DzeRUc6i7d9VnNiA@mail.gmail.com

[-- Attachment #1: Type: text/plain, Size: 1221 bytes --]

On Wed, 2018-01-10 at 17:00 +0000, Andrey Kuzmin wrote:
> 
> 
> On Wed, Jan 10, 2018, 19:47 Luse, Paul E <paul.e.luse(a)intel.com> wrote:
> > So I'll take the easy way out and just ask.. what's the most efficient way
> > for an app, designed just this one for whatever reason, essentially
> > relinquish control to the event reactor - is there a single call he can slip
> > into the loop to give other pending events, if any a chance to run and if
> > not continue on submitting?
> 
> It appears quite logical to start submission with a check for pending
> completions, doesn't it? Or check for completions if downstream bdev returns
> busy status. That would definitely meet app expectations whatever the request
> pool size is.

We've considered checking for completions inside the submission path if we would
otherwise return ENOMEM. So far, we've decided not to go that direction for two
reasons.

1) Even if we do this, there are still cases where we'll return ENOMEM. For
instance, if there are no completions to reap yet.
2) This would result in completion callbacks in response to a submit call.
Today, the expectations are set that completions are called in response to a
poll call only.

             reply	other threads:[~2018-01-10 17:17 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-10 17:17 Walker, Benjamin [this message]
  -- strict thread matches above, loose matches on Subject: below --
2018-01-11  4:08 [SPDK] Problem with Blobstore when write 65MB continously Zhengyu Zhang
2018-01-10 20:53 Walker, Benjamin
2018-01-10 19:28 Andrey Kuzmin
2018-01-10 17:11 Walker, Benjamin
2018-01-10 17:02 Luse, Paul E
2018-01-10 17:00 Andrey Kuzmin
2018-01-10 16:58 Walker, Benjamin
2018-01-10 16:47 Luse, Paul E
2018-01-10 16:32 Walker, Benjamin
2018-01-10 16:21 Harris, James R
2018-01-10 16:03 Luse, Paul E
2018-01-10 15:59 Zhengyu Zhang
2018-01-10 15:18 Luse, Paul E
2018-01-10 14:03 Luse, Paul E
2018-01-10  3:15 Zhengyu Zhang

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=1515604634.6063.43.camel@intel.com \
    --to=spdk@lists.01.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.