Re: [PATCH] tracing/xen: Hide events that are not used when X86_PAE is not defined

From: Steven Rostedt
Date: Wed Oct 11 2017 - 21:05:50 EST


On Wed, 11 Oct 2017 19:39:20 -0500
Jeremy Linton <jeremy.linton@xxxxxxx> wrote:

> On 10/10/2017 06:12 PM, Steven Rostedt wrote:
> > From: Steven Rostedt (VMware) <rostedt@xxxxxxxxxxx>
> >
> > TRACE_EVENTS() take up memory. If they are defined but not used, then
> > they simply waste space. If their use case is behind a define, then the
> > trace events should be as well.
> >
> > The trace events xen_mmu_set_pte_atomic, xen_mmu_pte_clear, and
> > xen_mmu_pmd_clear are not used when CONFIG_X86_PAE is not defined.
>
> The only place i see them being used is mmu_pv.c, where they are wrapped
> in the stated #define.
>
> Looks good.
>
> Reviewed-by: Jeremy Linton <jeremy.linton@xxxxxxx>
>

Thanks Jeremy,

-- Steve