[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20091120083053.GB19778@elte.hu>
Date: Fri, 20 Nov 2009 09:30:53 +0100
From: Ingo Molnar <mingo@...e.hu>
To: Pekka Enberg <penberg@...helsinki.fi>
Cc: Li Zefan <lizf@...fujitsu.com>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Frederic Weisbecker <fweisbec@...il.com>,
Steven Rostedt <rostedt@...dmis.org>,
Peter Zijlstra <peterz@...radead.org>,
Eduard - Gabriel Munteanu <eduard.munteanu@...ux360.ro>,
LKML <linux-kernel@...r.kernel.org>,
"linux-mm@...ck.org" <linux-mm@...ck.org>
Subject: Re: [RFC][PATCH 1/2] perf: Add 'perf kmem' tool
* Pekka Enberg <penberg@...helsinki.fi> wrote:
> On Fri, Nov 20, 2009 at 10:14 AM, Ingo Molnar <mingo@...e.hu> wrote:
> > Pekka, Eduard and the other slab hackers might have ideas about what
> > other stats they generally like to see to judge the health of a workload
> > (or system).
>
> kmalloc()/kfree() CPU ping-pong call-sites (i.e. alloc and free
> happening on different CPUs) is one interesting metric we haven't
> implemented yet. Valgrind massif tool type of output graph would be
> helpful as well:
>
> http://valgrind.org/docs/manual/ms-manual.html
>
> On Fri, Nov 20, 2009 at 10:14 AM, Ingo Molnar <mingo@...e.hu> wrote:
> > If this iteration looks good to the slab folks then i can apply it as-is
> > and we can do the other changes relative to that. It looks good to me as
> > a first step, and it's functional already.
>
> Yeah, looks OK to me as the first step. Patch 2 looks premature,
> though, looking at the output of "perf kmem" from patch 1.
>
> Acked-by: Pekka Enberg <penberg@...helsinki.fi>
Great - thanks for the quick ack!
Regarding patch 2 - can we set some definitive benchmark threshold for
that? I.e. a list of must-have features in 'perf kmem' before we can do
it? 100% information and analysis equivalency with kmemtrace-user tool?
Eduard, what do you think?
Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists