[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c42bfeb5-32ed-ae67-e09e-90968b71b74a@linux.intel.com>
Date: Wed, 20 Feb 2019 17:13:40 +0300
From: Alexey Budankov <alexey.budankov@...ux.intel.com>
To: Jiri Olsa <jolsa@...hat.com>
Cc: Arnaldo Carvalho de Melo <acme@...nel.org>,
Ingo Molnar <mingo@...hat.com>,
Peter Zijlstra <peterz@...radead.org>,
Namhyung Kim <namhyung@...nel.org>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Andi Kleen <ak@...ux.intel.com>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2 2/4] perf record: implement -z=<level> and
--mmap-flush=<thres> options
On 12.02.2019 16:08, Jiri Olsa wrote:
> On Mon, Feb 11, 2019 at 11:22:38PM +0300, Alexey Budankov wrote:
>
> SNIP
>
>> diff --git a/tools/perf/util/session.h b/tools/perf/util/session.h
>> index d96eccd7d27f..0e14884f28b2 100644
>> --- a/tools/perf/util/session.h
>> +++ b/tools/perf/util/session.h
>> @@ -35,6 +35,8 @@ struct perf_session {
>> struct ordered_events ordered_events;
>> struct perf_data *data;
>> struct perf_tool *tool;
>> + u64 bytes_transferred;
>> + u64 bytes_compressed;
>
> please add those in separate patch, so we can see how they are
> incremented and displayed easily
Moved declaration and printing of the values into a separate patch.
Thanks,
Alexey
>
> thanks,
> jirka
>
Powered by blists - more mailing lists