Messages in this thread | | | Subject | Re: [PATCH 0/2][for-next] cleanup submission path | From | Jens Axboe <> | Date | Sun, 27 Oct 2019 10:32:04 -0600 |
| |
On 10/27/19 9:35 AM, Pavel Begunkov wrote: > A small cleanup of very similar but diverged io_submit_sqes() and > io_ring_submit() > > Pavel Begunkov (2): > io_uring: handle mm_fault outside of submission > io_uring: merge io_submit_sqes and io_ring_submit > > fs/io_uring.c | 116 ++++++++++++++------------------------------------ > 1 file changed, 33 insertions(+), 83 deletions(-)
I like the cleanups here, but one thing that seems off is the assumption that io_sq_thread() always needs to grab the mm. If the sqes processed are just READ/WRITE_FIXED, then it never needs to grab the mm.
-- Jens Axboe
| |