<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Sep 13, 2023, 7:40 AM Athira Rajeev <<a href="mailto:atrajeev@linux.vnet.ibm.com">atrajeev@linux.vnet.ibm.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
> On 08-Sep-2023, at 7:48 PM, Athira Rajeev <<a href="mailto:atrajeev@linux.vnet.ibm.com" target="_blank" rel="noreferrer">atrajeev@linux.vnet.ibm.com</a>> wrote:<br>
> <br>
> <br>
> <br>
>> On 08-Sep-2023, at 11:04 AM, Sachin Sant <<a href="mailto:sachinp@linux.ibm.com" target="_blank" rel="noreferrer">sachinp@linux.ibm.com</a>> wrote:<br>
>> <br>
>> <br>
>> <br>
>>> On 07-Sep-2023, at 10:29 PM, Athira Rajeev <<a href="mailto:atrajeev@linux.vnet.ibm.com" target="_blank" rel="noreferrer">atrajeev@linux.vnet.ibm.com</a>> wrote:<br>
>>> <br>
>>> Testcase "Parsing of all PMU events from sysfs" parse events for<br>
>>> all PMUs, and not just cpu. In case of powerpc, the PowerVM<br>
>>> environment supports events from hv_24x7 and hv_gpci PMU which<br>
>>> is of example format like below:<br>
>>> <br>
>>> - hv_24x7/CPM_ADJUNCT_INST,domain=?,core=?/<br>
>>> - hv_gpci/event,partition_id=?/<br>
>>> <br>
>>> The value for "?" needs to be filled in depending on system<br>
>>> configuration. It is better to skip these parametrized events<br>
>>> in this test as it is done in:<br>
>>> 'commit b50d691e50e6 ("perf test: Fix "all PMU test" to skip<br>
>>> parametrized events")' which handled a simialr instance with<br>
>>> "all PMU test".<br>
>>> <br>
>>> Fix parse-events test to skip parametrized events since<br>
>>> it needs proper setup of the parameters.<br>
>>> <br>
>>> Signed-off-by: Athira Rajeev <<a href="mailto:atrajeev@linux.vnet.ibm.com" target="_blank" rel="noreferrer">atrajeev@linux.vnet.ibm.com</a>><br>
>>> ---<br>
>>> Changelog:<br>
>>> v1 -> v2:<br>
>>> Addressed review comments from Ian. Updated size of<br>
>>> pmu event name variable and changed bool name which is<br>
>>> used to skip the test.<br>
>>> <br>
>> <br>
>> The patch fixes the reported issue.<br>
>> <br>
>> 6.2: Parsing of all PMU events from sysfs                          : Ok<br>
>> 6.3: Parsing of given PMU events from sysfs                        : Ok<br>
>> <br>
>> Tested-by: Sachin Sant <<a href="mailto:sachinp@linux.ibm.com" target="_blank" rel="noreferrer">sachinp@linux.ibm.com</a>><br>
>> <br>
>> - Sachin<br>
> <br>
> Hi Sachin, Ian<br>
> <br>
> Thanks for testing the patch<br>
<br>
Hi Arnaldo<br>
<br>
Can you please check and pull this if it looks good to go .<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Namhyung, can you please take a look?</div><div dir="auto"><br></div><div dir="auto">I'll be back home next week, now at Kernel Recipes in Paris.</div><div dir="auto"><br></div><div dir="auto">- Arnaldo </div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks<br>
Athira<br>
> <br>
> Athira<br>
> <br>
> <br>
<br>
</blockquote></div></div></div>