[igt-dev] [PATCH i-g-t v2 0/1] tests/i915/perf: Add stress / race exercises

[Kamil Konieczny]

Hi,

On 2023-02-10 at 08:53:12 +0100, Janusz Krzysztofik wrote:
> Hi,
> 
> On Thursday, 9 February 2023 12:50:38 CET Janusz Krzysztofik wrote:
> > Users reported oopses on list corruptions when using i915 perf with a
> > number of concurrently running graphics applications.  That indicates we
> > are currently missing some important tests for such scenarios.  Cover
> > that gap.
> > 
> > v2: drop open-race subtest for now, not capable of triggering the user
> >     reported bug, but triggering other bugs which I can't see any fixes
> >     for queued yet,
> >   - move the other new subtest out of tests/i915/perf.c (Ashutosh).
> > 
> > Janusz Krzysztofik (1):
> >   tests/gem_ctx_exec: Exercise barrier race
> 
> While still waiting for CI results (BAT results don't cover the new subtest) 
> I've collected results from a forced execution of the subtest in BAT scope on 
> trybot: https://patchwork.freedesktop.org/series/113608/#rev2
> 
> While working as expected on most platforms, the test failed on some ancient 
> ones instead of skipping.  I've fixed this issue and tested the fix 
> successfully on trybot: https://patchwork.freedesktop.org/series/113608/#rev3
> 
> I'm still waiting for your comments, if any, before I submit the fixed 
> version.

Patch looks good but as you already noticed it is blacklisted
and do not cause noticeable fail. Proposed solution is to move
it to other test or to create new one, imho one you proposed

igt at gem_barrier_race@remote-request

looks good.

Regards,
Kamil

> 
> Thanks,
> Janusz
> 
> > 
> >  tests/i915/gem_ctx_exec.c | 123 ++++++++++++++++++++++++++++++++++++++
> >  tests/meson.build         |   9 ++-
> >  2 files changed, 131 insertions(+), 1 deletion(-)
> > 
> > 
> 
> 
> 
>