[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20131115060643.GA9237@gmail.com>
Date: Fri, 15 Nov 2013 07:06:43 +0100
From: Ingo Molnar <mingo@...nel.org>
To: Arnaldo Carvalho de Melo <acme@...radead.org>
Cc: linux-kernel@...r.kernel.org,
Adrian Hunter <adrian.hunter@...el.com>,
David Ahern <dsahern@...il.com>,
Frederic Weisbecker <fweisbec@...il.com>,
Ingo Molnar <mingo@...hat.com>, Jiri Olsa <jolsa@...hat.com>,
Mike Galbraith <efault@....de>,
Namhyung Kim <namhyung@...il.com>,
Paul Mackerras <paulus@...ba.org>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Stephane Eranian <eranian@...gle.com>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Namhyung Kim <namhyung@...nel.org>
Subject: Re: [PATCH 10/10] perf record: Add an option to force per-cpu mmaps
* Arnaldo Carvalho de Melo <acme@...radead.org> wrote:
> +--force-per-cpu::
> + Force the use of per-cpu mmaps. By default, when tasks are specified (i.e. -p,
> + -t or -u options) per-thread mmaps are created. This option overrides that and
> + forces per-cpu mmaps. A side-effect of that is that inheritance is
> + automatically enabled. Add the -i option also to disable inheritance.
So I still haven't seen an explanation why it's called 'force'
anything. AFAICS nothing is 'forced' really, this is simply another
trace-ringbuffer setup method, right?
And I also raised why this shouldn't be the default event tracing
method instead of a weird config option. Per-cpu tracing is cache
compact, it is easier to size properly and in general it is pretty
easy to think about. (It also has less of the TSC timestamp ordering
problems as per thread tracing, at least in theory.)
Is there something that makes per cpu tracing undesirable as the
default?
Thanks,
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