public inbox for [email protected]
 help / color / mirror / Atom feed
From: Hao Xu <[email protected]>
To: Jens Axboe <[email protected]>, io-uring <[email protected]>
Cc: Pavel Begunkov <[email protected]>, [email protected]
Subject: Re: [RFC] a new way to achieve asynchronous IO
Date: Tue, 21 Jun 2022 11:38:06 +0800	[thread overview]
Message-ID: <[email protected]> (raw)
In-Reply-To: <[email protected]>

On 6/20/22 21:41, Jens Axboe wrote:
> On 6/20/22 6:01 AM, Hao Xu wrote:
>> Hi,
>> I've some thought on the way of doing async IO. The current model is:
>> (given we are using SQPOLL mode)
>>
>> the sqthread does:
>> (a) Issue a request with nowait/nonblock flag.
>> (b) If it would block, reutrn -EAGAIN
>> (c) The io_uring layer captures this -EAGAIN and wake up/create
>> a io-worker to execute the request synchronously.
>> (d) Try to issue other requests in the above steps again.
>>
>> This implementation has two downsides:
>> (1) we have to find all the block point in the IO stack manually and
>> change them into "nowait/nonblock friendly".
>> (2) when we raise another io-worker to do the request, we submit the
>> request from the very beginning. This isn't a little bit inefficient.
>>
>>
>> While I think we can actually do it in a reverse way:
>> (given we are using SQPOLL mode)
>>
>> the sqthread1 does:
>> (a) Issue a request in the synchronous way
>> (b) If it is blocked/scheduled soon, raise another sqthread2
>> (c) sqthread2 tries to issue other requests in the same way.
>>
>> This solves problem (1), and may solve (2).
>> For (1), we just do the sqthread waken-up at the beginning of schedule()
>> just like what the io-worker and system-worker do. No need to find all
>> the block point.
>> For (2), we continue the blocked request from where it is blocked when
>> resource is satisfied.
>>
>> What we need to take care is making sure there is only one task
>> submitting the requests.
>>
>> To achieve this, we can maintain a pool of sqthread just like the iowq.
>>
>> I've done a very simple/ugly POC to demonstrate this:
>>
>> https://github.com/HowHsu/linux/commit/183be142493b5a816b58bd95ae4f0926227b587b
>>
>> I also wrote a simple test to test it, which submits two sqes, one
>> read(pipe), one nop request. The first one will be block since no data
>> in the pipe. Then a new sqthread was created/waken up to submit the
>> second one and then some data is written to the pipe(by a unrelated
>> user thread), soon the first sqthread is waken up and continues the
>> request.
>>
>> If the idea sounds no fatal issue I'll change the POC to real patches.
>> Any comments are welcome!
> 
> One thing I've always wanted to try out is kind of similar to this, but
> a superset of it. Basically io-wq isn't an explicit offload mechanism,
> it just happens automatically if the issue blocks. This applies to both
> SQPOLL and non-SQPOLL.
> 
> This takes a page out of the old syslet/threadlet that Ingo Molnar did
> way back in the day [1], but it never really went anywhere. But the
> pass-on-block primitive would apply very nice to io_uring.
> 
> That way it'd work is that any issue, SQPOLL or not, would just assume
> that it won't block. If it doesn't block, great, we can complete it
> inline. If it does block, an io-wq thread is grabbed and the context
> moved there. The io-wq takes over the blocking, and the original issue
> returns in some fashion that allows us to know it went implicitly async.

This sounds pretty good, I once thought about this but couldn't figure
it out clearly how to return to the desire place in the original
context.

> 
> This may be a bit more involved than what you suggest here, which in
> nature is similar in how we just hope for the best, and deal with the
> outcome if we did end up blocking.
> 
> Do you have any numbers from your approach?

Currently no, the POC is only to prove the idea works. Shouldn't be hard
to modify it to get some numbers.

> 
> [1] https://lore.kernel.org/all/[email protected]/T/
> 

I'll take a look and see what I can do.


  reply	other threads:[~2022-06-21  3:38 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-06-20 12:01 [RFC] a new way to achieve asynchronous IO Hao Xu
2022-06-20 12:03 ` Hao Xu
2022-06-20 13:41 ` Jens Axboe
2022-06-21  3:38   ` Hao Xu [this message]
2022-06-23 13:31   ` Hao Xu
2022-06-23 14:08     ` Jens Axboe
2022-06-27  7:11       ` Hao Xu
2022-06-28 13:33         ` Hao Xu
2022-07-12  7:11       ` Hao Xu

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 \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    [email protected] \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox