Re: [PATCH] block: split out a new blk_plug.h helper

[Johannes Thumshirn]

On 7/6/26 8:45 AM, Christoph Hellwig wrote:
> On Mon, Jul 06, 2026 at 08:38:12AM +0200, Johannes Thumshirn wrote:
> > On 7/6/26 6:11 AM, Christoph Hellwig wrote:
> > > diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h
> > > index 9213a5716f95..20cb8ed7d987 100644
> > > --- a/include/linux/blkdev.h
> > > +++ b/include/linux/blkdev.h
> > > @@ -7,6 +7,7 @@
> > >      #include <linux/types.h>
> > >    #include <linux/blk_types.h>
> > > +#include <linux/blk_plug.h>
> > >    #include <linux/device.h>
> > >    #include <linux/list.h>
> > >    #include <linux/llist.h>
> > I know it's a lot of cross subsystem churn, but wouldn't it be cleaner to
> > not include blk_plug.h in blkdev.h, but patch the update the consumers? A
> > quick grep shows 68 files that would need updating and some you already
> > have updated.
> Right now blkdev.h needs the rq_list from it.  So we'd need to move
> that to linux/types.h or something first, which feels a bit iffy.
>
> And no, including blk_types.h in blk_plug.h is not a solution,
> as that is still touched far too often.

Ah sh*t I did miss rq_list. Fair enough.


Reviewed-by: Johannes Thumshirn <johannes.thumshirn@xxxxxxx>