[PATCHv5 4/9] blk-integrity: consider entire bio list for merging

Christoph Hellwig hch at lst.de
Sat Sep 14 00:30:12 PDT 2024


On Fri, Sep 13, 2024 at 11:28:49AM -0700, Keith Busch wrote:
> From: Keith Busch <kbusch at kernel.org>
> 
> If a bio is merged to a request, the entire bio list is merged, so don't
> temporarily detach it from its list when counting segments. In most
> cases, bi_next will already be NULL, so detaching is usually a no-op.
> But if the bio does have a list, the current code is miscounting the
> segments for the resulting merge.

As I explained in detail last round this is still wrong.  There is
no bio list here ever.




More information about the Linux-nvme mailing list