On 2/6/20 2:51 AM, Pavel Begunkov wrote: > On 2/6/2020 5:50 AM, Jens Axboe wrote: >> On 2/5/20 3:29 PM, Pavel Begunkov wrote: >>> On 05/02/2020 22:07, Pavel Begunkov wrote: >>>> This is the first series of shaving some overhead for wq-offloading. >>>> The 1st removes extra allocations, and the 3rd req->refs abusing. >>> >>> Rechecked a couple of assumptions, this patchset is messed up. >>> Drop it for now. >> >> OK, will do, haven't had time to look at it yet anyway. > > Sorry for the fuss. I'll return to it later. No worries >> Are you going to do the ->has_user removal? We should just do that >> separately first. > Yes. I've spotted a few bugs, so I'm going to patch them first with > merging/backporting in mind, and then deal with ->has_user. IMO, this > order makes more sense. I think it probably makes sense to do it in the opposite order, as the ->has_user cleanup/clarification should go into 5.6 whereas the other stuff is likely 5.7 material. -- Jens Axboe