linux-kernel - Re: [PATCH 05/33] perf record: Allow the user to disable time stamps

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140805143159.GI13375@kernel.org>
Date:	Tue, 5 Aug 2014 11:31:59 -0300
From:	Arnaldo Carvalho de Melo <acme@...nel.org>
To:	Andi Kleen <ak@...ux.intel.com>
Cc:	Ingo Molnar <mingo@...nel.org>, linux-kernel@...r.kernel.org,
	Andi Kleen <andi@...stfloor.org>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>,
	Stephane Eranian <eranian@...gle.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Jiri Olsa <jolsa@...hat.com>
Subject: Re: [PATCH 05/33] perf record: Allow the user to disable time stamps

Em Tue, Aug 05, 2014 at 07:17:33AM -0700, Andi Kleen escreveu:
> > > Time stamps are always implicitely enabled for record currently.  The
> > > old --time/-T option is a nop.

> > > Allow the user to disable timestamps by using --no-time

> > > This can cause some minor misaccounting (by missing mmaps), but
> > > significantly lowers the size of perf.data

> > I'm not any big change in size:

> >  -rw------- 1 mingo mingo 384768 Aug  5 08:01 perf.data.timestamps
> >  -rw------- 1 mingo mingo 336952 Aug  5 08:00 perf.data.notimestamps

> It will depend on your workload. What period did you use
> (or did it automatically use) and what kind of 
> workload was it?

> The smaller the period, the higher the benefit.

> There are some classes of workloads where using a smaller
> period is especially beneficial, essentially anything
> with lots of small events, instead of long loops.

> You also get a higher benefit if you use -c or -F, because
> without that each sample is smaller (no period reported)

> You also get higher benefit for longer traces, and traces
> that do not start a lot of programs, as those
> tend to be dominated by MMAP events and other overhead.

I guess it would have been great if you had put the above paragraphs in
the documentation for 'perf record' :-\

- Arnaldo
 
> > So either remove the --time option altogether, or fix its 
> > 'misaccounting' so that the profile can be relied on.

> I don't know how to fix it. Do you?

> I guess one possible way to mitigate would be to lower the perf
> buffer sizes, then the worst case out of ordering
> would be less (I believe any potential problem
> just comes from out of order events). However that may
> impact performance.

Mentioning that OOO is the main problem here, i.e. that some samples may
be misaccounted, should've also been added to the documentation.

- Arnaldo
--
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/