All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@kernel.dk>
To: Pavel Begunkov <asml.silence@gmail.com>, io-uring@vger.kernel.org
Subject: Re: [zcrx-next 0/2] add support for synchronous refill
Date: Wed, 20 Aug 2025 13:36:07 -0600	[thread overview]
Message-ID: <46d0d30b-e9da-402d-9393-7877e9de117e@kernel.dk> (raw)
In-Reply-To: <20422c51-1227-42a8-8506-8f499eb53e89@gmail.com>

On 8/20/25 1:33 PM, Pavel Begunkov wrote:
> On 8/20/25 20:02, Jens Axboe wrote:
>> On 8/20/25 12:57 PM, Pavel Begunkov wrote:
>>> On 8/20/25 19:20, Jens Axboe wrote:
>>>>
>>>> On Sun, 17 Aug 2025 23:44:56 +0100, Pavel Begunkov wrote:
>>>>> Returning buffers via a ring is efficient but can cause problems
>>>>> when the ring doesn't have space. Add a way to return buffers
>>>>> synchronously via io_uring "register" syscall, which should serve
>>>>> as a slow fallback path.
>>>>>
>>>>> For a full branch with all relevant dependencies see
>>>>> https://github.com/isilence/linux.git zcrx/for-next
>>>>>
>>>>> [...]
>>>>
>>>> Applied, thanks!
>>>
>>> Leave these and other series out please. I sent them for
>>> review, but it'll be easier to keep in a branch and rebase
>>> it if necessary. I hoped tagging with zcrx-next would be
>>> good enough of a sign.
>>
>> Alright, but then please make it clear when you are sending
>> something out for merging.
> 
> That would be a pull request, should be obvious enough. Perhaps
> it'd make sense to tag for review only patches. Any preference
> what tag it should be or otherwise?

Please just send patches rather than a pull request. PRs get messed up
by people all the time, and there's no picking and choosing there, it's
all or nothing.

A review tag would be good in the subject, anything easily recognizable
should do it. Like for-review or whatever, doesn't matter to me.

-- 
Jens Axboe

      reply	other threads:[~2025-08-20 19:36 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-17 22:44 [zcrx-next 0/2] add support for synchronous refill Pavel Begunkov
2025-08-17 22:44 ` [zcrx-next 1/2] io_uring/zcrx: introduce io_parse_rqe() Pavel Begunkov
2025-08-17 22:44 ` [zcrx-next 2/2] io_uring/zcrx: allow synchronous buffer return Pavel Begunkov
2025-08-20 18:20 ` [zcrx-next 0/2] add support for synchronous refill Jens Axboe
2025-08-20 18:57   ` Pavel Begunkov
2025-08-20 19:02     ` Jens Axboe
2025-08-20 19:33       ` Pavel Begunkov
2025-08-20 19:36         ` Jens Axboe [this message]

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=46d0d30b-e9da-402d-9393-7877e9de117e@kernel.dk \
    --to=axboe@kernel.dk \
    --cc=asml.silence@gmail.com \
    --cc=io-uring@vger.kernel.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.