Re: [PATCH] perf test: Increase buffer size for Coresight basic tests

From: Arnaldo Carvalho de Melo
Date: Wed Apr 17 2024 - 12:15:45 EST


On Mon, Apr 15, 2024 at 01:55:25PM +0530, Anshuman Khandual wrote:
>
>
> On 4/12/24 13:52, James Clark wrote:
> >
> >
> > On 12/04/2024 08:04, Anshuman Khandual wrote:
> >>
> >>
> >> On 3/26/24 17:07, James Clark wrote:
> >>> These tests record in a mode that includes kernel trace but look for
> >>> samples of a userspace process. This makes them sensitive to any kernel
> >>> compilation options that increase the amount of time spent in the
> >>> kernel. If the trace buffer is completely filled before userspace is
> >>> reached then the test will fail. Double the buffer size to fix this.
> >>
> >> This is a valid concern to address, but just wondering how did we arrive
> >> at the conclusion that doubling the buffer size i.e making that 8M will
> >> solve the problem positively for vast number of kerne build scenarios ?
> >>
> >
> > Nobody else has reported anything yet, if it happens again we can always
> > increase it again if that is what the issue is. I had most of the kernel
> > debugging stuff turned on like memory debugging etc, which is probably
> > why I ran into it and 8MB fixed it for me. So I'm not sure if there is
> > much more that could be added.
>
> Makes sense,
>
> Reviewed-by: Anshuman Khandual <anshuman.khandual@xxxxxxx>

Thanks, applied to perf-tools-next,

- Arnaldo