On 19.04.20 21:46, David Sterba wrote:
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.
Thanks for your reply, will drop this one.
Thanks,
Guoqing
