On Thu, 12 Jun 2025 07:06:47 -0700
Dave Hansen <dave.han...@intel.com> wrote:
> On 6/12/25 07:03, Steven Rostedt wrote:
> > Remove the events/tlb.h from mm/rmap.c and add the define
> > CREATE_TRACE_POINTS back in the x86 code.
> >
> > Fixes: 4cc79b3303f22 ("mm/migration: add trace events for base page and
> > HugeTLB migrations")
>
> Is this a big enough problem that it deserves a cc:stable@?
That's questionable. Without this patch, some memory is simply wasted.
So it matters how much you care about wasted memory? On memory tight
systems it may make a difference. But it's just one trace event, which
would free up around 5K of memory.
I put in the fixes tag here because it was an obvious mistake that the
header file wasn't removed from rmap.c when the call to the tracepoint
was. Hmm, I think I put in the wrong fixes tag. I should probably be
the other commit:
Fixes: e73ad5ff2f76 ("mm, x86/mm: Make the batched unmap TLB flush API more
generic")
-- Steve
