Re: [PATCH 3/3] perf/buildid-cache: Support --purge-all option

From: Ravi Bangoria
Date: Mon Apr 16 2018 - 05:40:58 EST


Hi Masami,

On 04/16/2018 02:57 PM, Masami Hiramatsu wrote:
> On Mon, 9 Apr 2018 16:36:33 +0530
> Ravi Bangoria <ravi.bangoria@xxxxxxxxxxxxxxxxxx> wrote:
>
>> User can remove files from cache using --remove/--purge options
>> but both needs list of files as an argument. It's not convenient
>> when you want to flush out entire cache. Add an option to purge
>> all files from cache.
>>
>> Ex,
>> # perf buildid-cache -l
>> /tmp/a.out (8a86ef73e44067bca52cc3f6cd3e5446c783391c)
>> /tmp/a.out.1 (ebe71fdcf4b366518cc154d570a33cd461a51c36)
>> # perf buildid-cache -P -v
>> Removing /tmp/a.out (8a86ef73e44067bca52cc3f6cd3e5446c783391c): Ok
>> Removing /tmp/a.out.1 (ebe71fdcf4b366518cc154d570a33cd461a51c36): Ok
>> Purged all: Ok
> Hmm, for purging all caches will be done by
>
> $ rm -rf ~/.debug
>
> Are there any difference?

No logical difference if you know it's ~/.debug where it goes. :)
I also used to do rm -rf earlier.

This option is for a perf users. But I'm fine if it's not really needed.
Will drop it.

Thanks,
Ravi