On 24/09/2019 14:15, Jens Axboe wrote: > On 9/24/19 5:11 AM, Pavel Begunkov wrote: >> On 24/09/2019 13:34, Jens Axboe wrote: >>> On 9/24/19 4:13 AM, Jens Axboe wrote: >>>> On 9/24/19 3:49 AM, Peter Zijlstra wrote: >>>>> On Tue, Sep 24, 2019 at 10:36:28AM +0200, Jens Axboe wrote: >>>>> >>>>>> +struct io_wait_queue { >>>>>> + struct wait_queue_entry wq; >>>>>> + struct io_ring_ctx *ctx; >>>>>> + struct task_struct *task; >>>>> >>>>> wq.private is where the normal waitqueue stores the task pointer. >>>>> >>>>> (I'm going to rename that) >>>> >>>> If you do that, then we can just base the io_uring parts on that. >>> >>> Just took a quick look at it, and ran into block/kyber-iosched.c that >>> actually uses the private pointer for something that isn't a task >>> struct... >>> >> >> Let reuse autoremove_wake_function anyway. Changed a bit your patch: > > Yep that should do it, and saves 8 bytes of stack as well. > > BTW, did you test my patch, this one or the previous? Just curious if it > worked for you. > Not yet, going to do that tonight -- Yours sincerely, Pavel Begunkov
Attachment:
signature.asc
Description: OpenPGP digital signature