[igt-dev] [PATCH] tests/kms_cursor_legacy: Reducing the IOCTL(drm_mode_cursor_ioctl) calls by reducing the timeout.

[Garg, Nemesa]

> -----Original Message-----
> From: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Sent: Friday, June 16, 2023 6:07 PM
> To: igt-dev at lists.freedesktop.org
> Cc: Garg, Nemesa <nemesa.garg at intel.com>; Garg at freedesktop.org; Juha-
> Pekka Heikkila <juhapekka.heikkila at gmail.com>
> Subject: Re: [igt-dev] [PATCH] tests/kms_cursor_legacy: Reducing the
> IOCTL(drm_mode_cursor_ioctl) calls by reducing the timeout.
> 
> Hi Garg,
> 
> tests/kms_cursor_legacy: Reducing the IOCTL(drm_mode_cursor_ioctl) calls
> by reducing the timeout.
> ----------------------------------------------------------------------------------------------- ^
> 
> Please do not write a dot at the end of Subject, so:
> 
> s/timeout./timeout/
> 
> Regards,
> Kamil
> 
Hi Kamil,

Thank you for your comment. I have made the change and sent a new patch.

Thanks,
Nemesa
> On 2023-06-15 at 18:06:54 +0530, Garg, Nemesa wrote:
> > Due to excessive IOCTL calls in a stress test we are getting disk
> > limit issue. Hence by reducing the timeout from 20 to 5 the IOCTL call
> > will get reduced.
> >
> > Checked with different values and found 5 as optimal value.
> >
> > Signed-off-by: Garg, Nemesa <nemesa.garg at intel.com>
> > ---
> >  tests/kms_cursor_legacy.c | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/kms_cursor_legacy.c b/tests/kms_cursor_legacy.c
> > index f1e55906..13918345 100644
> > --- a/tests/kms_cursor_legacy.c
> > +++ b/tests/kms_cursor_legacy.c
> > @@ -1562,12 +1562,12 @@ igt_main
> >  					errno = 0;
> >
> >  					igt_dynamic_f("pipe-%s",
> kmstest_pipe_name(n))
> > -						stress(&display, n,
> tests[i].ncpus, tests[i].flags, 20);
> > +						stress(&display, n,
> tests[i].ncpus, tests[i].flags, 5);
> >  				}
> >
> >  				errno = 0;
> >  				igt_dynamic("all-pipes")
> > -					stress(&display, -1, tests[i].ncpus,
> tests[i].flags, 20);
> > +					stress(&display, -1, tests[i].ncpus,
> tests[i].flags, 5);
> >  			}
> >  		}
> >  	}
> > --
> > 2.25.1
> >