On 11/21/19 8:23 AM, Pavel Begunkov wrote: > On 11/21/2019 4:53 PM, Jens Axboe wrote: >> On 11/21/19 7:28 AM, Pavel Begunkov wrote: >>>>> >>>>> Was that a reviewed-by from you? It'd be nice to get these more >>>>> formally so I can add the attributes. I'll drop the other patch. >>>> >>>> I want to do more tests. There is no test machine at this time, at least >>>> until tomorrow morning. >>>> >>> BTW, aside from the locking problem, that it fixes another one. If >>> allocation for a shadow req is failed, io_submit_sqes() just continues >>> without it ignoring draining. >> >> Indeed. BTW, your commit message is way too weak for this patch. It >> doesn't explain why we're making this change at all. I'm going to fix >> it up. >> > Yeah, I were going to do that today. This one is of quick-before-leaving > kind, I didn't even looked up into the problem discussion properly. > > It needs 2 problems statements + reported-by Here's what I have: https://git.kernel.dk/cgit/linux-block/commit/?h=for-5.5/io_uring-post&id=b1417baddedff723b22a4817753625350cfa895a -- Jens Axboe