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

From: Nilay Vaish
Date: Wed Jun 29 2016 - 12:07:18 EST


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.

--
Nilay