Re: [PATCH 0/2] Update CascadelakeX and SkylakeX events list

From: "Jin, Yao" <yao.jin@linux.intel.com>
To: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Ian Rogers <irogers@google.com>, Jiri Olsa <jolsa@kernel.org>,
	Peter Zijlstra <peterz@infradead.org>,
	Ingo Molnar <mingo@redhat.com>,
	Alexander Shishkin <alexander.shishkin@linux.intel.com>,
	LKML <Linux-kernel@vger.kernel.org>,
	Andi Kleen <ak@linux.intel.com>,
	kan.liang@intel.com, "Jin, Yao" <yao.jin@intel.com>,
	Stephane Eranian <eranian@google.com>
Subject: Re: [PATCH 0/2] Update CascadelakeX and SkylakeX events list
Date: Fri, 18 Sep 2020 09:21:27 +0800	[thread overview]
Message-ID: <8ed53667-8241-1c58-ac2c-ecc9e9870afd@linux.intel.com> (raw)
In-Reply-To: <82271c45-4a7d-d892-eccd-ed042b86a333@linux.intel.com>

Hi Arnaldo,

On 9/7/2020 4:01 PM, Jin, Yao wrote:
> Hi Arnaldo,
> 
> On 7/17/2020 2:06 PM, Jin, Yao wrote:
>> Hi,
>>
>> On 6/17/2020 3:38 AM, Arnaldo Carvalho de Melo wrote:
>>> Em Tue, Jun 16, 2020 at 02:27:40PM +0800, Jin, Yao escreveu:
>>>> On 6/16/2020 2:16 PM, Ian Rogers wrote:
>>>>> On Mon, Jun 15, 2020 at 6:00 PM Jin, Yao <yao.jin@linux.intel.com> wrote:
>>>>>> Can I get an ACK for this patchset?
>>>>>> On 6/3/2020 10:18 AM, Jin Yao wrote:
>>>>>>> This patchset updates CascadelakeX events to v1.08 and
>>>>>>> updates SkylakeX events to v1.21.
>>>
>>>>>>> The events have been tested on CascadelakeX and SkylakeX
>>>>>>> servers with latest perf/core branch.
>>>
>>>>> could you rebase this on Arnaldo's tmp.perf/core tree?
>>>>> https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/log/?h=tmp.perf/core
>>>>> I tried using git am but get:
>>>>> Applying: perf vendor events: Update CascadelakeX events to v1.08
>>>>> error: patch fragment without header at line 279: @@ -213,14 +220,14 @@
>>>>> error: could not build fake ancestor
>>>>> Patch failed at 0001 perf vendor events: Update CascadelakeX events to v1.08
>>>>>
>>>>> Thanks,
>>>>> Ian
>>>>>
>>>>
>>>> The patchset are applied OK on perf/core branch.
>>>>
>>>> As far as I understand, the patch for posting should be based on perf/core branch, is it right?
>>>
>>> Sorry, I've been testing with using tmp.perf/core as a way to more
>>> quickly make available what I've processed, before I test it thoroughly
>>> both with container builds and with manual testing, so the bleeding edge
>>> is there, perf/core becoming a bit more stable, as I'll try not to
>>> rebase it like before.
>>>
>>> - Arnaldo
>>>
>>
>> Can this event update be accepted?
>>
>> Thanks
>> Jin Yao
> 
> Can this event list update be accepted?
> 
> Thanks
> Jin Yao

Ping again. :)

Can this event update patchset be accepted?

@Ian, I remembered you had tried this patchset and it's OK right?

Thanks
Jin Yao