[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160727112411.6e60c186@gandalf.local.home>
Date: Wed, 27 Jul 2016 11:24:11 -0400
From: Steven Rostedt <rostedt@...dmis.org>
To: Janani Ravichandran <janani.rvchndrn@...il.com>
Cc: linux-mm@...ck.org, linux-kernel@...r.kernel.org, riel@...riel.com,
akpm@...ux-foundation.org, hannes@...pxchg.org,
vdavydov@...tuozzo.com, mhocko@...e.com, vbabka@...e.cz,
mgorman@...hsingularity.net, kirill.shutemov@...ux.intel.com,
bywxiaobai@....com
Subject: Re: [PATCH 2/2] mm: compaction.c: Add/Modify direct compaction
tracepoints
On Wed, 27 Jul 2016 10:51:03 -0400
Janani Ravichandran <janani.rvchndrn@...il.com> wrote:
> Add zone information to an existing tracepoint in compact_zone(). Also,
> add a new tracepoint at the end of the compaction code so that latency
> information can be derived.
>
> Signed-off-by: Janani Ravichandran <janani.rvchndrn@...il.com>
> ---
>
> + trace_mm_compaction_try_to_compact_pages_end(rc, *contended);
> +
Again, I'm not to thrilled about tracepoints just being added to track
the length of function calls. We have function graph tracer for that.
-- Steve
> return rc;
> }
>
Powered by blists - more mailing lists