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: <8e33080d-d2a1-cc57-3ec7-179ea7af1577@huawei.com>
Date:   Thu, 10 Jun 2021 10:02:11 +0800
From:   Yang Jihong <yangjihong1@...wei.com>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
CC:     <peterz@...radead.org>, <mingo@...hat.com>, <mark.rutland@....com>,
        <alexander.shishkin@...ux.intel.com>, <jolsa@...hat.com>,
        <namhyung@...nel.org>, <linux-perf-users@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v2] perf annotate: Add itrace options support

Hell Arnaldo,

On 2021/6/9 1:10, Arnaldo Carvalho de Melo wrote:
> Em Mon, Jun 07, 2021 at 10:59:18AM +0800, Yang Jihong escreveu:
>> The "auxtrace_info" and "auxtrace" functions are not set in "tool" member of
>> "annotate". As a result, perf annotate does not support parsing itrace data.
>>
>> A simple example is as follows:
>>
>>    # perf record -e arm_spe_0/branch_filter=1/ -a sleep 1
>>    [ perf record: Woken up 9 times to write data ]
>>    [ perf record: Captured and wrote 20.874 MB perf.data ]
>>    # perf annotate --stdio
>>    Error:
>>    The perf.data data has no samples!
> 
> Can I get an Acked-by/Reviewed-by for this? I don't have a machine where
> I can test this.
> 

I have tested the patch on a machine that supports ARM SPE.
The test result is as follows:
# perf annotate --stdio
# perf record --all-user -e arm_spe_0/branch_filter=1/ ls
Couldn't synthesize bpf events.
perf.data  perf.data.old
[ perf record: Woken up 1 times to write data ]
[ perf record: Captured and wrote 0.010 MB perf.data ]
# perf annotate --stdio
  Percent |      Source code & Disassembly of libc-2.28.so for 
branch-miss (1 samples, percent: local period)
------------------------------------------------------------------------------------------------------------
          :
          :
          :
          :           Disassembly of section .text:
          :
          :           0000000000066180 <__getdelim@@GLIBC_2.17>:
     0.00 :   66180:  stp     x29, x30, [sp, #-96]!
     0.00 :   66184:  cmp     x0, #0x0
     0.00 :   66188:  ccmp    x1, #0x0, #0x4, ne  // ne = any
     0.00 :   6618c:  mov     x29, sp
     0.00 :   66190:  stp     x24, x25, [sp, #56]
     0.00 :   66194:  stp     x26, x27, [sp, #72]
     0.00 :   66198:  str     x28, [sp, #88]
     0.00 :   6619c:  b.eq    66450 <__getdelim@@GLIBC_2.17+0x2d0>  // 
b.none
     0.00 :   661a0:  stp     x22, x23, [x29, #40]
     0.00 :   661a4:  mov     x22, x1
     0.00 :   661a8:  ldr     w1, [x3]
     0.00 :   661ac:  mov     w23, w2
     0.00 :   661b0:  stp     x20, x21, [x29, #24]
     0.00 :   661b4:  mov     x20, x3
     0.00 :   661b8:  mov     x21, x0
     0.00 :   661bc:  tbnz    w1, #15, 66360 <__getdelim@@GLIBC_2.17+0x1e0>
     0.00 :   661c0:  ldr     x0, [x3, #136]
     0.00 :   661c4:  ldr     x2, [x0, #8]
     0.00 :   661c8:  str     x19, [x29, #16]
     0.00 :   661cc:  mrs     x19, tpidr_el0
     0.00 :   661d0:  sub     x19, x19, #0x700
     0.00 :   661d4:  cmp     x2, x19
     0.00 :   661d8:  b.eq    663f0 <__getdelim@@GLIBC_2.17+0x270>  // 
b.none
     0.00 :   661dc:  mov     w1, #0x1                        // #1
     0.00 :   661e0:  ldaxr   w2, [x0]
     0.00 :   661e4:  cmp     w2, #0x0
     0.00 :   661e8:  b.ne    661f4 <__getdelim@@GLIBC_2.17+0x74>  // b.any
     0.00 :   661ec:  stxr    w3, w1, [x0]
     0.00 :   661f0:  cbnz    w3, 661e0 <__getdelim@@GLIBC_2.17+0x60>
     0.00 :   661f4:  b.ne    66448 <__getdelim@@GLIBC_2.17+0x2c8>  // b.any
     0.00 :   661f8:  ldr     x0, [x20, #136]
     0.00 :   661fc:  ldr     w1, [x20]
     0.00 :   66200:  ldr     w2, [x0, #4]
     0.00 :   66204:  str     x19, [x0, #8]
     0.00 :   66208:  add     w2, w2, #0x1
     0.00 :   6620c:  str     w2, [x0, #4]
     0.00 :   66210:  tbnz    w1, #5, 66388 <__getdelim@@GLIBC_2.17+0x208>
     0.00 :   66214:  ldr     x19, [x29, #16]
     0.00 :   66218:  ldr     x0, [x21]
     0.00 :   6621c:  cbz     x0, 66228 <__getdelim@@GLIBC_2.17+0xa8>
     0.00 :   66220:  ldr     x0, [x22]
     0.00 :   66224:  cbnz    x0, 6623c <__getdelim@@GLIBC_2.17+0xbc>
     0.00 :   66228:  mov     x0, #0x78                       // #120
     0.00 :   6622c:  str     x0, [x22]
     0.00 :   66230:  bl      20710 <malloc@plt>
     0.00 :   66234:  str     x0, [x21]
     0.00 :   66238:  cbz     x0, 66428 <__getdelim@@GLIBC_2.17+0x2a8>
     0.00 :   6623c:  ldr     x27, [x20, #8]
     0.00 :   66240:  str     x19, [x29, #16]
     0.00 :   66244:  ldr     x19, [x20, #16]
     0.00 :   66248:  sub     x19, x19, x27
     0.00 :   6624c:  cmp     x19, #0x0
     0.00 :   66250:  b.le    66398 <__getdelim@@GLIBC_2.17+0x218>
     0.00 :   66254:  mov     x25, #0x0                       // #0
     0.00 :   66258:  b       662d8 <__getdelim@@GLIBC_2.17+0x158>
     0.00 :   6625c:  nop
     0.00 :   66260:  add     x24, x19, x25
     0.00 :   66264:  ldr     x3, [x22]
     0.00 :   66268:  add     x26, x24, #0x1
     0.00 :   6626c:  ldr     x0, [x21]
     0.00 :   66270:  cmp     x3, x26
     0.00 :   66274:  b.cs    6629c <__getdelim@@GLIBC_2.17+0x11c>  // 
b.hs, b.nlast
     0.00 :   66278:  lsl     x3, x3, #1
     0.00 :   6627c:  cmp     x3, x26
     0.00 :   66280:  csel    x26, x3, x26, cs  // cs = hs, nlast
     0.00 :   66284:  mov     x1, x26
     0.00 :   66288:  bl      206f0 <realloc@plt>
     0.00 :   6628c:  cbz     x0, 66438 <__getdelim@@GLIBC_2.17+0x2b8>
     0.00 :   66290:  str     x0, [x21]
     0.00 :   66294:  ldr     x27, [x20, #8]
     0.00 :   66298:  str     x26, [x22]
     0.00 :   6629c:  mov     x2, x19
     0.00 :   662a0:  mov     x1, x27
     0.00 :   662a4:  add     x0, x0, x25
     0.00 :   662a8:  bl      87390 <explicit_bzero@@GLIBC_2.25+0x50>
     0.00 :   662ac:  ldr     x0, [x20, #8]
     0.00 :   662b0:  add     x19, x0, x19
     0.00 :   662b4:  str     x19, [x20, #8]
     0.00 :   662b8:  cbnz    x28, 66410 <__getdelim@@GLIBC_2.17+0x290>
     0.00 :   662bc:  mov     x0, x20
     0.00 :   662c0:  bl      73b80 <__underflow@@GLIBC_2.17>
     0.00 :   662c4:  cmn     w0, #0x1
     0.00 :   662c8:  b.eq    66410 <__getdelim@@GLIBC_2.17+0x290>  // 
b.none
     0.00 :   662cc:  ldp     x27, x19, [x20, #8]
     0.00 :   662d0:  mov     x25, x24
     0.00 :   662d4:  sub     x19, x19, x27
     0.00 :   662d8:  mov     x2, x19
     0.00 :   662dc:  mov     w1, w23
     0.00 :   662e0:  mov     x0, x27
     0.00 :   662e4:  bl      807b0 <memchr@@GLIBC_2.17>
     0.00 :   662e8:  cmp     x0, #0x0
     0.00 :   662ec:  mov     x28, x0
     0.00 :   662f0:  sub     x0, x0, x27
     0.00 :   662f4:  csinc   x19, x19, x0, eq  // eq = none
     0.00 :   662f8:  mov     x0, #0x7fffffffffffffff         // 
#9223372036854775807
     0.00 :   662fc:  sub     x0, x0, x25
     0.00 :   66300:  cmp     x19, x0
     0.00 :   66304:  b.lt    66260 <__getdelim@@GLIBC_2.17+0xe0>  // 
b.tstop
     0.00 :   66308:  adrp    x0, 17f000 <sys_sigabbrev@@GLIBC_2.17+0x320>
     0.00 :   6630c:  ldr     x0, [x0, #3624]
     0.00 :   66310:  mrs     x2, tpidr_el0
     0.00 :   66314:  ldr     x19, [x29, #16]
     0.00 :   66318:  mov     w3, #0x4b                       // #75
     0.00 :   6631c:  ldr     w1, [x20]
     0.00 :   66320:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   66324:  str     w3, [x2, x0]
     0.00 :   66328:  tbnz    w1, #15, 66340 <__getdelim@@GLIBC_2.17+0x1c0>
     0.00 :   6632c:  ldr     x0, [x20, #136]
     0.00 :   66330:  ldr     w1, [x0, #4]
     0.00 :   66334:  sub     w1, w1, #0x1
     0.00 :   66338:  str     w1, [x0, #4]
     0.00 :   6633c:  cbz     w1, 663b8 <__getdelim@@GLIBC_2.17+0x238>
     0.00 :   66340:  mov     x0, x24
     0.00 :   66344:  ldr     x28, [sp, #88]
     0.00 :   66348:  ldp     x20, x21, [x29, #24]
     0.00 :   6634c:  ldp     x22, x23, [x29, #40]
     0.00 :   66350:  ldp     x24, x25, [sp, #56]
     0.00 :   66354:  ldp     x26, x27, [sp, #72]
     0.00 :   66358:  ldp     x29, x30, [sp], #96
     0.00 :   6635c:  ret
   100.00 :   66360:  tbz     w1, #5, 66218 <__getdelim@@GLIBC_2.17+0x98>
     0.00 :   66364:  ldp     x20, x21, [x29, #24]
     0.00 :   66368:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   6636c:  ldp     x22, x23, [x29, #40]
     0.00 :   66370:  mov     x0, x24
     0.00 :   66374:  ldp     x24, x25, [sp, #56]
     0.00 :   66378:  ldp     x26, x27, [sp, #72]
     0.00 :   6637c:  ldr     x28, [sp, #88]
     0.00 :   66380:  ldp     x29, x30, [sp], #96
     0.00 :   66384:  ret
     0.00 :   66388:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   6638c:  ldr     x19, [x29, #16]
     0.00 :   66390:  b       66328 <__getdelim@@GLIBC_2.17+0x1a8>
     0.00 :   66394:  nop
     0.00 :   66398:  mov     x0, x20
     0.00 :   6639c:  bl      73b80 <__underflow@@GLIBC_2.17>
     0.00 :   663a0:  cmn     w0, #0x1
     0.00 :   663a4:  b.eq    66438 <__getdelim@@GLIBC_2.17+0x2b8>  // 
b.none
     0.00 :   663a8:  ldp     x27, x19, [x20, #8]
     0.00 :   663ac:  sub     x19, x19, x27
     0.00 :   663b0:  b       66254 <__getdelim@@GLIBC_2.17+0xd4>
     0.00 :   663b4:  nop
     0.00 :   663b8:  str     xzr, [x0, #8]
     0.00 :   663bc:  ldxr    w2, [x0]
     0.00 :   663c0:  stlxr   w3, w1, [x0]
     0.00 :   663c4:  cbnz    w3, 663bc <__getdelim@@GLIBC_2.17+0x23c>
     0.00 :   663c8:  cmp     w2, #0x1
     0.00 :   663cc:  b.le    66340 <__getdelim@@GLIBC_2.17+0x1c0>
     0.00 :   663d0:  mov     x1, #0x81                       // #129
     0.00 :   663d4:  mov     x2, #0x1                        // #1
     0.00 :   663d8:  mov     x3, #0x0                        // #0
     0.00 :   663dc:  mov     x8, #0x62                       // #98
     0.00 :   663e0:  svc     #0x0
     0.00 :   663e4:  ldp     x20, x21, [x29, #24]
     0.00 :   663e8:  ldp     x22, x23, [x29, #40]
     0.00 :   663ec:  b       66370 <__getdelim@@GLIBC_2.17+0x1f0>
     0.00 :   663f0:  ldr     w2, [x0, #4]
     0.00 :   663f4:  add     w2, w2, #0x1
     0.00 :   663f8:  str     w2, [x0, #4]
     0.00 :   663fc:  tbz     w1, #5, 66214 <__getdelim@@GLIBC_2.17+0x94>
     0.00 :   66400:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   66404:  ldr     x19, [x29, #16]
     0.00 :   66408:  b       66330 <__getdelim@@GLIBC_2.17+0x1b0>
     0.00 :   6640c:  nop
     0.00 :   66410:  ldr     x0, [x21]
     0.00 :   66414:  strb    wzr, [x0, x24]
     0.00 :   66418:  ldr     w1, [x20]
     0.00 :   6641c:  ldr     x19, [x29, #16]
     0.00 :   66420:  b       66328 <__getdelim@@GLIBC_2.17+0x1a8>
     0.00 :   66424:  nop
     0.00 :   66428:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   6642c:  ldr     w1, [x20]
     0.00 :   66430:  b       66328 <__getdelim@@GLIBC_2.17+0x1a8>
     0.00 :   66434:  nop
     0.00 :   66438:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   6643c:  ldr     w1, [x20]
     0.00 :   66440:  ldr     x19, [x29, #16]
     0.00 :   66444:  b       66328 <__getdelim@@GLIBC_2.17+0x1a8>
     0.00 :   66448:  bl      e3ba0 <pthread_setcanceltype@@GLIBC_2.17+0x30>
     0.00 :   6644c:  b       661f8 <__getdelim@@GLIBC_2.17+0x78>
     0.00 :   66450:  adrp    x0, 17f000 <sys_sigabbrev@@GLIBC_2.17+0x320>
     0.00 :   66454:  ldr     x0, [x0, #3624]
     0.00 :   66458:  mrs     x1, tpidr_el0
     0.00 :   6645c:  mov     w2, #0x16                       // #22
     0.00 :   66460:  mov     x24, #0xffffffffffffffff        // #-1
     0.00 :   66464:  str     w2, [x1, x0]
     0.00 :   66468:  b       66370 <__getdelim@@GLIBC_2.17+0x1f0>
     0.00 :   6646c:  ldr     w1, [x20]
     0.00 :   66470:  mov     x4, x0
     0.00 :   66474:  tbnz    w1, #15, 6648c <__getdelim@@GLIBC_2.17+0x30c>
     0.00 :   66478:  ldr     x0, [x20, #136]
     0.00 :   6647c:  ldr     w1, [x0, #4]
     0.00 :   66480:  sub     w1, w1, #0x1
     0.00 :   66484:  str     w1, [x0, #4]
     0.00 :   66488:  cbz     w1, 66494 <__getdelim@@GLIBC_2.17+0x314>
     0.00 :   6648c:  mov     x0, x4
     0.00 :   66490:  bl      20e40 <gnu_get_libc_version@@GLIBC_2.17+0x130>
     0.00 :   66494:  str     xzr, [x0, #8]
     0.00 :   66498:  ldxr    w2, [x0]
     0.00 :   6649c:  stlxr   w3, w1, [x0]
     0.00 :   664a0:  cbnz    w3, 66498 <__getdelim@@GLIBC_2.17+0x318>
     0.00 :   664a4:  cmp     w2, #0x1
     0.00 :   664a8:  b.le    6648c <__getdelim@@GLIBC_2.17+0x30c>
     0.00 :   664ac:  mov     x1, #0x81                       // #129
     0.00 :   664b0:  mov     x2, #0x1                        // #1
     0.00 :   664b4:  mov     x3, #0x0                        // #0
     0.00 :   664b8:  mov     x8, #0x62                       // #98
     0.00 :   664bc:  svc     #0x0
     0.00 :   664c0:  b       6648c <__getdelim@@GLIBC_2.17+0x30c>

Intel PT and ARM SPE share auxtrace functions of itrace framework. 
Therefore, this patch can be verified on computers that support Intel PT.

Because I do not have a machine that supports Intel PT, I cannot verify 
the Intel PT scenario. :)

Thanks,
Yang

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ