| Message ID | 20210921082253.1859794-1-hch@lst.de (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | mm: don't include <linux/dax.h> in <linux/mempolicy.h> | expand |
On Tue, Sep 21, 2021 at 10:22:53AM +0200, Christoph Hellwig wrote: > Not required at all, and having this causes a huge kernel rebuild > as soon as something in dax.h changes. > > Signed-off-by: Christoph Hellwig <hch@lst.de> Looks good to me. Reviewed-by: Naoya Horiguchi <naoya.horiguchi@nec.com>
On Tue, Sep 21, 2021 at 1:23 AM Christoph Hellwig <hch@lst.de> wrote: > > Not required at all, and having this causes a huge kernel rebuild > as soon as something in dax.h changes. Looks good. Reviewed-by: Dan Williams <dan.j.williams@intel.com>
diff --git a/include/linux/mempolicy.h b/include/linux/mempolicy.h index 0aaf91b496e2fe..b4992a7e1abbd7 100644 --- a/include/linux/mempolicy.h +++ b/include/linux/mempolicy.h @@ -8,7 +8,6 @@ #include <linux/sched.h> #include <linux/mmzone.h> -#include <linux/dax.h> #include <linux/slab.h> #include <linux/rbtree.h> #include <linux/spinlock.h> diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 470400cc751363..adcd6d7b7233c9 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c @@ -39,6 +39,7 @@ #include <linux/kernel-page-flags.h> #include <linux/sched/signal.h> #include <linux/sched/task.h> +#include <linux/dax.h> #include <linux/ksm.h> #include <linux/rmap.h> #include <linux/export.h>
Not required at all, and having this causes a huge kernel rebuild as soon as something in dax.h changes. Signed-off-by: Christoph Hellwig <hch@lst.de> --- include/linux/mempolicy.h | 1 - mm/memory-failure.c | 1 + 2 files changed, 1 insertion(+), 1 deletion(-)