On Mon, Feb 7, 2022 at 3:49 PM Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
>
> On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig <hch@xxxxxx> wrote:
> >
> > Move the check for the actual pgmap types that need the free at refcount
> > one behavior into the out of line helper, and thus avoid the need to
> > pull memremap.h into mm.h.
>
> Looks good to me assuming the compile bots agree.
>
> Reviewed-by: Dan Williams <dan.j.williams@xxxxxxxxx>
Yeah, same as Logan:
mm/memcontrol.c: In function ‘get_mctgt_type’:
mm/memcontrol.c:5724:29: error: implicit declaration of function
‘is_device_private_page’; did you mean
‘is_device_private_entry’? [-Werror=implicit-function-declaration]
5724 | if (is_device_private_page(page))
| ^~~~~~~~~~~~~~~~~~~~~~
| is_device_private_entry
...needs:
diff --git a/mm/memcontrol.c b/mm/memcontrol.c
index d1e97a54ae53..0ac7515c85f9 100644
--- a/mm/memcontrol.c
+++ b/mm/memcontrol.c
@@ -62,6 +62,7 @@
#include <linux/tracehook.h>
#include <linux/psi.h>
#include <linux/seq_buf.h>
+#include <linux/memremap.h>
#include "internal.h"
#include <net/sock.h>
#include <net/ip.h>
