Re: [PATCH 1/2] xfs: handle bio split errors during gc

[Christoph Hellwig <hch@lst.de>]

On Mon, Oct 20, 2025 at 11:16:57AM -0400, Chris Mason wrote:
> On 10/20/25 10:57 AM, Christoph Hellwig wrote:
> > On Mon, Oct 20, 2025 at 08:54:46AM -0600, Keith Busch wrote:
> >>> Ugg, how?  If that actually happens we're toast, so we should find a
> >>> way to ensure it does not happen.
> >>
> >> You'd have to attempt sending an invalid bvec, like something that can't
> >> DMA map because you have a byte aligned offset, or the total size is
> >> smaller than the block device's.
> >>
> >> Not that you're doing anything like that here. This condition should
> >> never occur in this path because the bio vectors are all nicely aligned.
> >> It's just for completeness to ensure it doesn't go uncaught for every
> >> bio split caller.
> > 
> > So this is just from code inspection and you did not actually hit
> > such a case?
> 
> This is from me testing out AI reviews on linux-next, and they do love
> suggesting defensive programming a little too much.  I'd be happier
> with EIOs in this case but agree it's probably never going to happen.

It would be relaly useful to note in the patches if something was
found by code analys or testing and what tools are used.