lkml.org 
[lkml]   [2016]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v3 3/7 UPDATE] perf tools: Add option for the path of buildid dsos under symfs
From
Date
On 5/17/16 7:47 PM, Hekuang wrote:
>
>
> 在 2016/5/16 10:50, David Ahern 写道:
>> On 5/15/16 7:30 PM, Hekuang wrote:
>>> In previous patch, I use 'perf buildid-cache -a' to add vdso
>>> binary into the HOST buildid dir.
>>
>> So 'perf buildid-cache' needs the symfs option?
>>
>>
> With this patch 'PATCH v3 3/7 UPDATE', the tree of symfs dir is
> like this:
>
> ├── debug($(dso-prefix))
> │ ├── .build-id
> │ │ ├── 3a
> │ │ │ └── e5ba6d4e532ad529e43ccf1ce1ddf8a64a4fdd ->
> ../../[kernel.kallsyms]/3ae5ba6d4e532ad529e43ccf1ce1ddf8a64a4fdd
> │ │ └── 84
> │ │ └── dbd75729adba57cc42f5544b25de571c0c8731 ->
> ../../[vdso32]/84dbd75729adba57cc42f5544b25de571c0c8731
> │ ├── [kernel.kallsyms]
> │ │ └── 3ae5ba6d4e532ad529e43ccf1ce1ddf8a64a4fdd
> │ ├── [vdso]
> │ │ └── 84dbd75729adba57cc42f5544b25de571c0c8731
> │ └── [vdso32]
> │ └── 84dbd75729adba57cc42f5544b25de571c0c8731
> ├── lib
> │ ├── ld-2.22.so
> │ └── libc-2.22.so
> ├── tmp
> │ └── hello
> └── xxx
>
> So all binaries we need are included in the symfs dir. I think
> this is consistent with your idea explained in previous mails.
>
> With this symfs, we do not need buildid dir anymore and what's
> your idea on 'perf buildid-cache' needs symfs option? after all,
> that only effects on buildid dir.

I don't understand why dso-prefix option is needed? Why make me type yet
more options to the analysis command? Why can't the directory be located
under the symfs tree in a known location and populated the same way it
is without symfs?

\
 
 \ /
  Last update: 2016-05-18 04:01    [W:0.195 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site