On Sun, Apr 19, 2020 at 12:51:20AM +0200, Guoqing Jiang wrote: > Some places can be replaced with __clear_page_buffers after the function > is exported. > > Cc: Chris Mason <clm@xxxxxx> > Cc: Josef Bacik <josef@xxxxxxxxxxxxxx> > Cc: David Sterba <dsterba@xxxxxxxx> > Cc: linux-btrfs@xxxxxxxxxxxxxxx > Signed-off-by: Guoqing Jiang <guoqing.jiang@xxxxxxxxxxxxxxx> > --- > fs/btrfs/disk-io.c | 5 ++--- > fs/btrfs/extent_io.c | 6 ++---- > fs/btrfs/inode.c | 14 ++++---------- > 3 files changed, 8 insertions(+), 17 deletions(-) > > diff --git a/fs/btrfs/disk-io.c b/fs/btrfs/disk-io.c > index a6cb5cbbdb9f..0f1e5690e8a4 100644 > --- a/fs/btrfs/disk-io.c > +++ b/fs/btrfs/disk-io.c > @@ -17,6 +17,7 @@ > #include <linux/error-injection.h> > #include <linux/crc32c.h> > #include <linux/sched/mm.h> > +#include <linux/buffer_head.h> I'm not really thrilled to see buffer_head.h being added back, we're on the track to remove buffer_head usage completely and adding it just for one helper does not seem great to me.
