Re: [PATCH 2/5] perf tools: Allow to reset open files counter

From: Nilay Vaish
Date: Thu Jun 30 2016 - 09:28:40 EST


On 29 June 2016 at 11:05, Nilay Vaish <nilayvaish@xxxxxxxxx> wrote:
> On 28 June 2016 at 06:29, Jiri Olsa <jolsa@xxxxxxxxxx> wrote:
>> I hit a bug when running test suite without forking
>> each test (-F option):
>>
>> $ perf test -F dso
>> 8: Test dso data read : Ok
>> 9: Test dso data cache : FAILED!
>> 10: Test dso data reopen : FAILED!
>>
>> The reason the session file limit is set just once for
>> perf process so we need to reset it for each test,
>> otherwise wrong limit is taken into account.
>>
>> Link: http://lkml.kernel.org/n/tip-bqapag0ljaiwmb7hlkw09zk5@xxxxxxxxxxxxxx
>> Signed-off-by: Jiri Olsa <jolsa@xxxxxxxxxx>
>
>
> The patch seems fine to me. I confirmed that we see those failed
> tests without this patch and applying it fixes them.
>

Tested-by: Nilay Vaish <nilayvaish@xxxxxxxxx>