> On Nov 13, 2015, at 16:41, Hillf Danton <hillf...@alibaba-inc.com> wrote: > >> >> Instead of the condition, we could have: >> >> __entry->pfn = page ? page_to_pfn(page) : -1; >> >> >> But if there's no reason to do the tracepoint if page is NULL, then >> this patch is fine. I'm just throwing out this idea. >> > we trace only if page is valid > > --- linux-next/mm/huge_memory.c Fri Nov 13 16:00:22 2015 > +++ b/mm/huge_memory.c Fri Nov 13 16:26:19 2015 > @@ -1987,7 +1987,8 @@ static int __collapse_huge_page_isolate( > > out: > release_pte_pages(pte, _pte); > - trace_mm_collapse_huge_page_isolate(page_to_pfn(page), none_or_zero, > + if (page) > + trace_mm_collapse_huge_page_isolate(page_to_pfn(page), > none_or_zero, > referenced, writable, result); > return 0; > } > — > my V4 patch move if (!page) into trace function, so that we don’t need call page_to_fn() if the trace if disabled . more efficient . Thanks
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
