On 3/16/24 7:27 AM, Pavel Begunkov wrote: > On 3/16/24 11:52, Ming Lei wrote: >> On Fri, Mar 15, 2024 at 04:53:21PM -0600, Jens Axboe wrote: >>> >>> On Fri, 15 Mar 2024 15:29:50 +0000, Pavel Begunkov wrote: >>>> Patch 1 is a fix. >>>> >>>> Patches 2-7 are cleanups mainly dealing with issue_flags conversions, >>>> misundertsandings of the flags and of the tw state. It'd be great to have >>>> even without even w/o the rest. >>>> >>>> 8-11 mandate ctx locking for task_work and finally removes the CQE >>>> caches, instead we post directly into the CQ. Note that the cache is >>>> used by multishot auxiliary completions. >>>> >>>> [...] >>> >>> Applied, thanks! >>> >>> [02/11] io_uring/cmd: kill one issue_flags to tw conversion >>> commit: 31ab0342cf6434e1e2879d12f0526830ce97365d >>> [03/11] io_uring/cmd: fix tw <-> issue_flags conversion >>> commit: b48f3e29b89055894b3f50c657658c325b5b49fd >>> [04/11] io_uring/cmd: introduce io_uring_cmd_complete >>> commit: c5b4c92ca69215c0af17e4e9d8c84c8942f3257d >>> [05/11] ublk: don't hard code IO_URING_F_UNLOCKED >>> commit: c54cfb81fe1774231fca952eff928389bfc3b2e3 >>> [06/11] nvme/io_uring: don't hard code IO_URING_F_UNLOCKED >>> commit: 800a90681f3c3383660a8e3e2d279e0f056afaee >>> [07/11] io_uring/rw: avoid punting to io-wq directly >>> commit: 56d565d54373c17b7620fc605c899c41968e48d0 >>> [08/11] io_uring: force tw ctx locking >>> commit: f087cdd065af0418ffc8a9ed39eadc93347efdd5 >>> [09/11] io_uring: remove struct io_tw_state::locked >>> commit: 339f8d66e996ec52b47221448ff4b3534cc9a58d >>> [10/11] io_uring: refactor io_fill_cqe_req_aux >>> commit: 7b31c3964b769a6a16c4e414baa8094b441e498e >>> [11/11] io_uring: get rid of intermediate aux cqe caches >>> commit: 5a475a1f47412a44ed184aac04b9ff0aeaa31d65 >> >> Hi Jens and Pavel, > > Jens, I hope you already dropped the series for now, right? It's just sitting in a branch for now, it's not even in linux-next. I'll review and look at a v2 of the series. So it hasn't moved anywhere yet. -- Jens Axboe
