On Wed, 11 Oct 2017 19:39:20 -0500
Jeremy Linton <[email protected]> wrote:

> On 10/10/2017 06:12 PM, Steven Rostedt wrote:
> > From: Steven Rostedt (VMware) <[email protected]>
> > 
> > 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 <[email protected]>
> 

Thanks Jeremy,

-- Steve

Reply via email to