on 12/23/2022 1:53 PM, Christoph Hellwig wrote: > On Fri, Dec 23, 2022 at 08:52:22PM +0800, Kemeng Shi wrote: >> + blk_mq_request_bypass_insert(rq, false, list_empty(list)); > > Please try to avoid the overly long line here. Get it and I will fix this in next version. Thanks! > That beng said blk_mq_request_bypass_insert is simply a horrible > API. I think we should do something like this: I am not quite follow this. I guess this API is horrible for two possbile reasons: 1. It accepts two bool parameters which may be confusing betwwen them. 2. It adds additional checks for if we need to insert at head and if we need to run queue which is already checked by caller. Anyway, it seems another patch is needed for this, but I don't know proper way to send this patch. Add your patch to this patchset or you want to send a single one after this patchset. -- Best wishes Kemeng Shi
