Re: [PATCH v15 10/10] perf test: Add direct off-cpu test

[Ian Rogers]

On Thu, Feb 13, 2025 at 3:04 PM Howard Chu <howardchu95@xxxxxxxxx> wrote:
>
> Hello,
>
> On Thu, Feb 13, 2025 at 3:00 PM Howard Chu <howardchu95@xxxxxxxxx> wrote:
> >
> > Why is there a --off-cpu-thresh 2000?
> >
> > We collect an off-cpu period __ONLY ONCE__, either in direct sample form,
> > or in accumulated form (in BPF stack trace map).
> >
> > If I don't add --off-cpu-thresh 2000, the sample in the original test
> > goes into the ring buffer instead of the BPF stack trace map.
> >
> > Additionally, when using -e dummy, the ring buffer is not open, causing
> > us to lose a sample.
>
> Just noticed that this commit message is wrong, should be:
> """
> Add tests for direct off-cpu samples and --off-cpu-thresh option.
> """

Tested-by: Ian Rogers <irogers@xxxxxxxxxx>
```
121: perf record offcpu profiling tests                              : Ok
```
I'd be tempted to keep the comments about why 2000 next to the actual
code rather than in the commit message. In the code the value is 12000
and not 2000 though?

Thanks,
Ian