2022-10-25 20:06:30

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: perf amd: PERF_MEM_LVLNUM_CXL -> PERF_MEM_LVLNUM_EXTN_MEM

Ravi,

I'm updating the tools header copies and noticed that a previous
sync brought PERF_MEM_LVLNUM_CXL but now this got renamed to
PERF_MEM_LVLNUM_EXTN_MEM, so I had to add this change to the sync of
tools/uapi/linux/perf_event.h, please ack.

Thanks,

- Arnaldo

⬢[acme@toolbox perf-urgent]$ git diff tools/perf/util/mem-events.c
diff --git a/tools/perf/util/mem-events.c b/tools/perf/util/mem-events.c
index b3a91093069a5715..c80fce18025051ae 100644
--- a/tools/perf/util/mem-events.c
+++ b/tools/perf/util/mem-events.c
@@ -295,7 +295,7 @@ static const char * const mem_lvl[] = {
};

static const char * const mem_lvlnum[] = {
- [PERF_MEM_LVLNUM_CXL] = "CXL",
+ [PERF_MEM_LVLNUM_EXTN_MEM] = "Extension mem",
[PERF_MEM_LVLNUM_IO] = "I/O",
[PERF_MEM_LVLNUM_ANY_CACHE] = "Any cache",
[PERF_MEM_LVLNUM_LFB] = "LFB/MAB",
⬢[acme@toolbox perf-urgent]$

--

- Arnaldo


2022-10-26 06:30:58

by Ravi Bangoria

[permalink] [raw]
Subject: Re: perf amd: PERF_MEM_LVLNUM_CXL -> PERF_MEM_LVLNUM_EXTN_MEM

Arnaldo,

On 26-Oct-22 1:25 AM, Arnaldo Carvalho de Melo wrote:
> Ravi,
>
> I'm updating the tools header copies and noticed that a previous
> sync brought PERF_MEM_LVLNUM_CXL but now this got renamed to
> PERF_MEM_LVLNUM_EXTN_MEM, so I had to add this change to the sync of
> tools/uapi/linux/perf_event.h, please ack.

Kernel header needs to be fixed as discussed here:
https://lore.kernel.org/lkml/[email protected]

Thanks,
Ravi

s/[email protected]/[email protected]/

2022-10-26 14:13:12

by Ravi Bangoria

[permalink] [raw]
Subject: Re: perf amd: PERF_MEM_LVLNUM_CXL -> PERF_MEM_LVLNUM_EXTN_MEM

On 26-Oct-22 7:10 PM, Arnaldo Carvalho de Melo wrote:
> Em Wed, Oct 26, 2022 at 11:16:24AM +0530, Ravi Bangoria escreveu:
>> Arnaldo,
>>
>> On 26-Oct-22 1:25 AM, Arnaldo Carvalho de Melo wrote:
>>> Ravi,
>>>
>>> I'm updating the tools header copies and noticed that a previous
>>> sync brought PERF_MEM_LVLNUM_CXL but now this got renamed to
>>> PERF_MEM_LVLNUM_EXTN_MEM, so I had to add this change to the sync of
>>> tools/uapi/linux/perf_event.h, please ack.
>>
>> Kernel header needs to be fixed as discussed here:
>> https://lore.kernel.org/lkml/[email protected]

Peter, can you please pick up this patch.

>
> Gotcha, will leave the tools/ copy with CXL then.
>
> - Arnaldo

2022-10-26 14:36:59

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: Re: perf amd: PERF_MEM_LVLNUM_CXL -> PERF_MEM_LVLNUM_EXTN_MEM

Em Wed, Oct 26, 2022 at 11:16:24AM +0530, Ravi Bangoria escreveu:
> Arnaldo,
>
> On 26-Oct-22 1:25 AM, Arnaldo Carvalho de Melo wrote:
> > Ravi,
> >
> > I'm updating the tools header copies and noticed that a previous
> > sync brought PERF_MEM_LVLNUM_CXL but now this got renamed to
> > PERF_MEM_LVLNUM_EXTN_MEM, so I had to add this change to the sync of
> > tools/uapi/linux/perf_event.h, please ack.
>
> Kernel header needs to be fixed as discussed here:
> https://lore.kernel.org/lkml/[email protected]

Gotcha, will leave the tools/ copy with CXL then.

- Arnaldo