DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 4F13621722
Date: Fri, 12 Jan 2018 11:14:24 -0300
From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Adrian Hunter <adrian.hunter@intel.com>
Cc: Kim Phillips <kim.phillips@arm.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Will Deacon <will.deacon@arm.com>, robh@kernel.org,
        mathieu.poirier@linaro.org, pawel.moll@arm.com,
        suzuki.poulose@arm.com, marc.zyngier@arm.com,
        linux-kernel@vger.kernel.org, alexander.shishkin@linux.intel.com,
        peterz@infradead.org, mingo@redhat.com, tglx@linutronix.de,
        linux-arm-kernel@lists.infradead.org, Jiri Olsa <jolsa@kernel.org>,
        Andi Kleen <ak@linux.intel.com>, Wang Nan <wangnan0@huawei.com>
Subject: Re: [PATCH v4] perf tools: Add ARM Statistical Profiling Extensions
 (SPE) support
Message-ID: <20180112141423.GA9655@kernel.org>
References: <20171121173302.bead17a1178ed4583e68014e@arm.com>
 <0a07a10c-fc0c-3f97-ac01-59429aab0937@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <0a07a10c-fc0c-3f97-ac01-59429aab0937@intel.com>
User-Agent: Mutt/1.9.1 (2017-09-22)
Sender: linux-kernel-owner@vger.kernel.org

Em Thu, Jan 11, 2018 at 04:17:51PM +0200, Adrian Hunter escreveu:
> On 22/11/17 01:33, Kim Phillips wrote:
> > - technically both cs-etm and spe can be used simultaneously, however
> >   disabled for simplicity in this release

> > Signed-off-by: Kim Phillips <kim.phillips@arm.com>
 
> For what is there now, it looks fine from the auxtrace point of view.  There
> are a couple of minor points below but nevertheless:
> 
> Acked-by: Adrian Hunter <adrian.hunter@intel.com>

Hi Kim,

	Please address the comments made by Adrian so that we can get
this merged,

	Adrian, thanks a lot for taking the time to review this, much
appreciated!

Thanks,

- Arnaldo