On Sat, Dec 07, 2024 at 04:14:30PM +0800, Junjie Fu wrote: > @@ -113,7 +113,7 @@ > #include <asm/tlbflush.h> > #include <asm/tlb.h> > #include <linux/uaccess.h> > - > +#include <linux/pagemap.h> > #include "internal.h" This is in the wrong place. The linux/ includes go before the asm/ includes. I like to stick to alphabetical order within that divide, but other people have other preferences.