Re: [Bug][xfstests xfs/556] inconsistent {HARDIRQ-ON-W} -> {IN-HARDIRQ-W} usage

["Darrick J. Wong" <djwong@kernel.org>]

On Sun, Mar 22, 2026 at 11:17:58PM -0700, Christoph Hellwig wrote:
> On Fri, Mar 20, 2026 at 09:34:44AM -0700, Darrick J. Wong wrote:
> > --- a/fs/iomap/bio.c
> > +++ b/fs/iomap/bio.c
> > @@ -8,7 +8,15 @@
> >  #include "internal.h"
> >  #include "trace.h"
> >  
> > -static void iomap_read_end_io(struct bio *bio)
> > +struct iomap_failed_bio {
> > +	struct list_head io_list;
> > +	struct bio *bio;
> > +};
> 
> Between this and the blockdev dontcache work I really wish we could
> have a generic offload using a percpu lit and workqueue.

<shrug> For IO errors I don't think it's urgent, but yes it would be
very nice to have a single mechanism for "kick IO completion to a
workqueue" type things.

> Also I don't think we need the separate list here, a linked list using
> bi_next should be sufficient and avoid the allocation in the failure
> path.

For now, I'll change this patch to use a bio_list.

--D