Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: H+GurQ+Tx6hV5fBElpHvtfF6VYVpt/EfR2st4wC1hiH11gg+lO5BRZaq+58/3nxnRN5zi10Hbw
 lgeJMO3sslAQ==
IronPort-SDR: wB6qGLmIhUh+DA0ygVAsJn0Twt21K41cRG9dFAmomP8Vof/2fPqiSNDPinGDkGIRy9FJFJk09k
 zX/ycn8spXAQ==
Subject: Re: [PATCH V2 00/12] perf intel-pt: Add support for decoding FUP/TIP
 only
From:   Adrian Hunter <adrian.hunter@intel.com>
To:     Arnaldo Carvalho de Melo <acme@kernel.org>
Cc:     Jiri Olsa <jolsa@redhat.com>, Andi Kleen <ak@linux.intel.com>,
        linux-kernel@vger.kernel.org
References: <20200710151104.15137-1-adrian.hunter@intel.com>
Organization: Intel Finland Oy, Registered Address: PL 281, 00181 Helsinki,
 Business Identity Code: 0357606 - 4, Domiciled in Helsinki
Message-ID: <f0a4f89f-f642-f75f-a390-da16d0a058d0@intel.com>
Date:   Tue, 4 Aug 2020 16:34:36 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
MIME-Version: 1.0
In-Reply-To: <20200710151104.15137-1-adrian.hunter@intel.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On 10/07/20 6:10 pm, Adrian Hunter wrote:
> Hi
> 
> Here are some fixes and small improvements for Intel PT.

Andi added his "Reviewed-by" with 2 comments to tweak the
documentation.

The patches still apply, so do you want me to send a V3?

> 
> Changes in V2:
> 	For d/e flags, use +/- alphabetic options instead of numbers
> 	Update help text
> 	Improve documentation
> 
> 
> Adrian Hunter (12):
>       perf intel-pt: Fix FUP packet state
>       perf intel-pt: Fix duplicate branch after CBR
>       perf tools: Improve aux_output not supported error
>       perf auxtrace: Add missing itrace options to help text
>       perf auxtrace: Add optional error flags to the itrace 'e' option
>       perf intel-pt: Use itrace error flags to suppress some errors
>       perf auxtrace: Add optional log flags to the itrace 'd' option
>       perf intel-pt: Use itrace debug log flags to suppress some messages
>       perf intel-pt: Time filter logged perf events
>       perf auxtrace: Add itrace 'q' option for quicker, less detailed decoding
>       perf intel-pt: Add support for decoding FUP/TIP only
>       perf intel-pt: Add support for decoding PSB+ only
> 
>  tools/perf/Documentation/itrace.txt                |  14 ++
>  tools/perf/Documentation/perf-intel-pt.txt         |  63 +++++-
>  tools/perf/util/auxtrace.c                         |  50 +++++
>  tools/perf/util/auxtrace.h                         |  31 ++-
>  tools/perf/util/evsel.c                            |   4 +
>  .../perf/util/intel-pt-decoder/intel-pt-decoder.c  | 214 +++++++++++++++++++--
>  .../perf/util/intel-pt-decoder/intel-pt-decoder.h  |   1 +
>  tools/perf/util/intel-pt.c                         |  45 ++++-
>  8 files changed, 389 insertions(+), 33 deletions(-)
> 
> 
> Regards
> Adrian
>