lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <599637d9-0b78-9edb-d459-515f76aca9fe@huawei.com>
Date:   Mon, 5 Mar 2018 11:24:13 +0000
From:   John Garry <john.garry@...wei.com>
To:     William Cohen <wcohen@...hat.com>,
        Ganapatrao Kulkarni <gklkml16@...il.com>,
        Jiri Olsa <jolsa@...hat.com>
CC:     Ganapatrao Kulkarni <ganapatrao.kulkarni@...ium.com>,
        <ak@...ux.intel.com>, Peter Zijlstra <peterz@...radead.org>,
        Will Deacon <will.deacon@....com>,
        Linuxarm <linuxarm@...wei.com>,
        "Arnaldo Carvalho de Melo" <acme@...nel.org>,
        <linux-kernel@...r.kernel.org>,
        Zhangshaokun <zhangshaokun@...ilicon.com>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        Ingo Molnar <mingo@...hat.com>, <namhyung@...nel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        <Jayachandran.Nair@...ium.com>
Subject: Re: [PATCH v2 00/11] perf events patches for improved ARM64 support

>> I am seeing issue(log below) with this patchset on our platfrom.
>> i have tried using your v2 branch [1]
>>
>> root@...g-1>perf_acme>> ./perf --version
>> perf version 4.16.rc1.g087f7ca
>> root@...g-1>perf_acme>> ./perf stat -e bus_access_rd sleep 1
>>
>>  Performance counter stats for 'sleep 1':
>>
>>             23,099      bus_access_rd
>>
>>        1.000708516 seconds time elapsed
>>
>> root@...g-1>perf_acme>> cd -
>> /ganapat/perf/linux-hisi/tools/perf
>> root@...g-1>perf>> ./perf --version
>> perf version 4.16.rc1.gcb5a74
>> root@...g-1>perf>> ./perf stat -e bus_access_rd sleep 1
>>
>>  Performance counter stats for 'sleep 1':
>>
>>                  0      bus_access_rd
>>
>>        1.000709162 seconds time elapsed
>>
>> root@...g-1>perf>>
>>
>>
>> [1] https://github.com/hisilicon/linux-hisi.git
>>
> Hi,

Hi Will Cohen, Ganapatrao,

>
> I took a slightly different tack and compared the output of "perf list --details" to see if the event numbers numbers were the same.  For the common events borrowed from the armv8-recommended.json all the events are 0 for the cortex a53.  That would explain why the bus_access_rd above were counts were different if it was counting the SW_INCR event instead of bus_access_rd.
>

Thanks for this. I see the problem. At the end I missed bringing across 
something from v1. Easily fixed.

> Why not key the matching of common events off the "EventName" rather than "EventCode"? The EventName would be more descriptive. Then the json files could eliminate the "BriefDescription" and just have the EventName with nothing else for each event.  This would further reduce the amount of redundant fields between the specific implementations and the common set.
>

I did consider it and it could work. But it all depends on whether the 
"EventName" only is always readable/descriptive enough in the JSON, 
without requiring the "BriefDescription". The "EventName" can be rather 
succinct for any arch. And we need to guarantee that the "EventName" is 
unique.

Here's a sample:
[
     {
         "ArchStdEvent": "L1D_CACHE_RD",
     },
     {
         "ArchStdEvent": "L1D_CACHE_WR",
     },

vs.

[
     {
         "ArchStdEvent": "0x40",
         "BriefDescription": "L1D cache access, read"
     },
     {
         "ArchStdEvent": "0x41",
         "BriefDescription": "L1D cache access, write"
     },


Opinion?

John

> -Will
>
> .
>

thanks,
John


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ