On Tue, Dec 06, 2016 at 03:50:52PM -0800, Laura Abbott wrote: > > __pa_symbol is technically the marcro that should be used for kernel > symbols. Switch to this as a pre-requisite for DEBUG_VIRTUAL which > will do bounds checking. > > Tested-by: James Morse <james.morse@xxxxxxx> > Signed-off-by: Laura Abbott <labbott@xxxxxxxxxx> This looks good to me; I have a (very minor) nit below, but either way: Reviewed-by: Mark Rutland <mark.rutland@xxxxxxx> Tested-by: Mark Rutland <mark.rutland@xxxxxxx> > diff --git a/arch/arm64/kernel/acpi_parking_protocol.c b/arch/arm64/kernel/acpi_parking_protocol.c > index a32b401..ca880ce 100644 > --- a/arch/arm64/kernel/acpi_parking_protocol.c > +++ b/arch/arm64/kernel/acpi_parking_protocol.c > @@ -18,6 +18,7 @@ > */ > #include <linux/acpi.h> > #include <linux/types.h> > +#include <linux/mm.h> Nit: please keep includes alphabetically ordered, at least where they're ordered today. Some files are already a mess, but it would be nice to keep the well-ordered ones ordered. Thanks, Mark. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>