diff mbox series

block: Include mm_types.h instead of mm.h

Message ID 20210621122407.3116975-1-willy@infradead.org (mailing list archive)
State New, archived
Headers show
Series block: Include mm_types.h instead of mm.h | expand

Commit Message

Matthew Wilcox (Oracle) June 21, 2021, 12:24 p.m. UTC
There's no need to include all of mm.h in bvec.h.  It only needs a few
things like the definition of struct page, PAGE_SIZE, PAGE_MASK and so
on, all of which are provided by mm_types.h.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
---
 include/linux/bvec.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Bart Van Assche June 21, 2021, 3:52 p.m. UTC | #1
On 6/21/21 5:24 AM, Matthew Wilcox (Oracle) wrote:
> There's no need to include all of mm.h in bvec.h.  It only needs a few
> things like the definition of struct page, PAGE_SIZE, PAGE_MASK and so
> on, all of which are provided by mm_types.h.

Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Jens Axboe June 21, 2021, 3:55 p.m. UTC | #2
On 6/21/21 6:24 AM, Matthew Wilcox (Oracle) wrote:
> There's no need to include all of mm.h in bvec.h.  It only needs a few
> things like the definition of struct page, PAGE_SIZE, PAGE_MASK and so
> on, all of which are provided by mm_types.h.

Applied, thanks.
diff mbox series

Patch

diff --git a/include/linux/bvec.h b/include/linux/bvec.h
index ff832e698efb..a0f3ccc09aba 100644
--- a/include/linux/bvec.h
+++ b/include/linux/bvec.h
@@ -11,7 +11,7 @@ 
 #include <linux/errno.h>
 #include <linux/limits.h>
 #include <linux/minmax.h>
-#include <linux/mm.h>
+#include <linux/mm_types.h>
 #include <linux/types.h>
 
 struct page;