Re: [PATCH v2 1/3] perf/sdt : Listing of SDT markers by perf

From: Masami Hiramatsu
Date: Sat Jul 19 2014 - 23:17:57 EST


(2014/07/19 2:50), Andi Kleen wrote:
> Hemant Kumar <hemant@xxxxxxxxxxxxxxxxxx> writes:
>
> First I should say supporting these probes is very useful. Thanks for
> working on this.
>
>> +
>> +#define SDT_CACHE_DIR "/var/cache/perf/"
>
> This requires running perf as root, right?
>
> It would be better to use the $HOME cache dir, like the recent JSON patches.

Even though, we have to enhance uprobe-tracer that allows non-root user
to add/remove events. :)
Anyway, I guess we can use ~/.debug for this purpose as buildid-cache does.

Thank you,


--
Masami HIRAMATSU
Software Platform Research Dept. Linux Technology Research Center
Hitachi, Ltd., Yokohama Research Laboratory
E-mail: masami.hiramatsu.pt@xxxxxxxxxxx


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/