On Tue, Aug 20, 2019 at 04:57:45PM +0100, Raphael Gault wrote: It would be worth having a body for the commit message like:
| in perf_event.c we use smp_processor_id(), but we haven't included | <linux/smp.h> where it is defined, and rely on this being pulled in | via a transitive include. Let's make this more robust by including | <linux.smp.h> explciitly. ... and with that, my Acked-by stands. Thanks, Mark. > Acked-by: Mark Rutland <[email protected]> > Signed-off-by: Raphael Gault <[email protected]> > --- > arch/arm64/kernel/perf_event.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/kernel/perf_event.c b/arch/arm64/kernel/perf_event.c > index 96e90e270042..24575c0a0065 100644 > --- a/arch/arm64/kernel/perf_event.c > +++ b/arch/arm64/kernel/perf_event.c > @@ -19,6 +19,7 @@ > #include <linux/of.h> > #include <linux/perf/arm_pmu.h> > #include <linux/platform_device.h> > +#include <linux/smp.h> > > /* ARMv8 Cortex-A53 specific event types. */ > #define ARMV8_A53_PERFCTR_PREF_LINEFILL 0xC2 > -- > 2.17.1 >
