[igt-dev] [PATCH i-g-t] igt/tests: kms_plane_stress: Add plane stress test

[Lisovskiy, Stanislav]

On Thu, 2019-03-21 at 14:06 +0100, Maarten Lankhorst wrote:
> Op 21-03-2019 om 13:59 schreef Lisovskiy, Stanislav:
> > On Thu, 2019-03-21 at 13:41 +0100, Maarten Lankhorst wrote:
> > > Op 21-03-2019 om 13:34 schreef Stanislav Lisovskiy:
> > > > This test attempts to utilize all connected
> > > > outputs at the same time, using maximum possible
> > > > resolution and amount of planes, to check whether
> > > > we are hiting any kind of bandwidth, watermark or
> > > > other limitations.
> > > > 
> > > > Signed-off-by: Stanislav Lisovskiy <
> > > > stanislav.lisovskiy at intel.com>
> > > > ---
> > > >  lib/igt_kms.h            |   7 ++
> > > >  tests/Makefile.sources   |   1 +
> > > >  tests/kms_plane_stress.c | 247
> > > > +++++++++++++++++++++++++++++++++++++++
> > > >  tests/meson.build        |   1 +
> > > >  tests/test.png           | Bin 0 -> 1068158 bytes
> > > 
> > > Remove the test png? You can use cairo calls to draw something,
> > > or
> > > igt_create_pattern_fb to get a default pattern.
> > 
> > Yes, can remove it. Just wanted some nice image here, however
> > probably
> > yeah will leave it for local testing.
> > 
> > > How is this different from kms_atomic_transition btw? It tries
> > > the
> > > same at least.
> > 
> > Well, I guess kms_atomic_transition doesn't utilize all pipe at the
> > same time, it goes just one by one. Here it tries to use all pipes
> > at the same time.
> > 
> 
> But it does have tests for all modes, see run_modeset_transition.
> Only thing missing is
> 
> that we don't enable all planes yet. But this should be easy to add
> with try commit.

In fact the idea behind this as we discussed with Martin, was to add
additional stressing factors here, like generating some heavy memory
usage at the same time + gpu load in order to see how system behaves in
this case.

The goal here is to detect all kinds of bandwidth issues, similar to
ones, which we had recently. That is why I had to put this to a
separate test case.

> 
> ~Maarten
>