[igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip tests if selected fetch is not available

[Srinivas, Vidya]

> -----Original Message-----
> From: Mark Yacoub <markyacoub at chromium.org>
> Sent: Thursday, October 7, 2021 8:54 PM
> To: Srinivas, Vidya <vidya.srinivas at intel.com>
> Cc: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>;
> Kattamanchi, JaswanthX <jaswanthx.kattamanchi at intel.com>; B, Jeevan
> <jeevan.b at intel.com>; igt-dev at lists.freedesktop.org; Modem,
> Bhanuprakash <bhanuprakash.modem at intel.com>;
> markyacoub at google.com; Almahallawy, Khaled
> <khaled.almahallawy at intel.com>; Lin, Charlton <charlton.lin at intel.com>
> Subject: Re: [igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip tests if selected
> fetch is not available
> 
> Reviewed and applied. Thanks Vidya!

Thank you so much Mark.

Regards
Vidya
> 
> On Thu, Oct 7, 2021 at 3:51 AM Srinivas, Vidya <vidya.srinivas at intel.com>
> wrote:
> >
> > Hello,
> >
> > I think we can take Rev3 for final merge?
> > https://patchwork.freedesktop.org/series/94288/#rev3 CI is PASS.
> >
> > Regards
> > Vidya
> >
> > > -----Original Message-----
> > > From: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> > > Sent: Wednesday, October 6, 2021 10:55 PM
> > > To: Mark Yacoub <markyacoub at chromium.org>
> > > Cc: Kattamanchi, JaswanthX <jaswanthx.kattamanchi at intel.com>; B,
> > > Jeevan <jeevan.b at intel.com>; igt-dev at lists.freedesktop.org; Modem,
> > > Bhanuprakash <bhanuprakash.modem at intel.com>; Srinivas, Vidya
> > > <vidya.srinivas at intel.com>; markyacoub at google.com
> > > Subject: RE: [igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip tests
> > > if selected fetch is not available
> > >
> > > We have an issue for this failure. Reported the results.
> > > https://gitlab.freedesktop.org/drm/intel/-/issues/4262
> > >
> > > Lakshmi.
> > >
> > > -----Original Message-----
> > > From: Mark Yacoub <markyacoub at chromium.org>
> > > Sent: Wednesday, October 6, 2021 9:27 AM
> > > To: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>
> > > Cc: Kattamanchi, JaswanthX <jaswanthx.kattamanchi at intel.com>; B,
> > > Jeevan <jeevan.b at intel.com>; igt-dev at lists.freedesktop.org; Modem,
> > > Bhanuprakash <bhanuprakash.modem at intel.com>; Srinivas, Vidya
> > > <vidya.srinivas at intel.com>; markyacoub at google.com
> > > Subject: Re: [igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip tests
> > > if selected fetch is not available
> > >
> > > Hey - I'm looking at the errors and they look unrelated. Can we get
> > > a new run
> > > :) Thanks!
> > >
> > > On Thu, Sep 30, 2021 at 6:36 PM Vudum, Lakshminarayana
> > > <lakshminarayana.vudum at intel.com> wrote:
> > > >
> > > > Re-reported.
> > > >
> > > >
> > > > -----Original Message-----
> > > > From: Mark Yacoub <markyacoub at chromium.org>
> > > > Sent: Thursday, September 30, 2021 7:29 AM
> > > > To: Vudum, Lakshminarayana <lakshminarayana.vudum at intel.com>;
> > > > Kattamanchi, JaswanthX <jaswanthx.kattamanchi at intel.com>
> > > > Cc: B, Jeevan <jeevan.b at intel.com>; igt-dev at lists.freedesktop.org;
> > > > Modem, Bhanuprakash <bhanuprakash.modem at intel.com>; Srinivas,
> > > Vidya
> > > > <vidya.srinivas at intel.com>; markyacoub at google.com
> > > > Subject: Re: [igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip tests
> > > > if selected fetch is not available
> > > >
> > > > Hi Lakshmi/Jaswanth
> > > >
> > > > I checked the patch
> > > > https://patchwork.freedesktop.org/series/94288/#rev2 and it looks
> > > > like a
> > > false positive, can we re-report?
> > > >
> > > > Thanks,
> > > > Mark
> > > >
> > > > On Thu, Sep 30, 2021 at 8:10 AM Srinivas, Vidya
> > > > <vidya.srinivas at intel.com>
> > > wrote:
> > > > >
> > > > >
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: B, Jeevan <jeevan.b at intel.com>
> > > > > > Sent: Thursday, September 30, 2021 3:26 PM
> > > > > > To: Srinivas, Vidya <vidya.srinivas at intel.com>;
> > > > > > igt-dev at lists.freedesktop.org
> > > > > > Cc: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>;
> > > > > > markyacoub at google.com; Srinivas, Vidya
> > > > > > <vidya.srinivas at intel.com>
> > > > > > Subject: RE: [igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip
> > > > > > tests if selected fetch is not available
> > > > > >
> > > > > > Looks good to me.
> > > > >
> > > > > Thank you very much Jeevan.
> > > > >
> > > > > Regards
> > > > > Vidya
> > > > > >
> > > > > > >-----Original Message-----
> > > > > > >From: igt-dev <igt-dev-bounces at lists.freedesktop.org> On
> > > > > > >Behalf Of Vidya Srinivas
> > > > > > >Sent: Thursday, September 16, 2021 7:30 PM
> > > > > > >To: igt-dev at lists.freedesktop.org
> > > > > > >Cc: Modem, Bhanuprakash <bhanuprakash.modem at intel.com>;
> > > > > > >markyacoub at google.com; Srinivas, Vidya
> > > > > > ><vidya.srinivas at intel.com>
> > > > > > >Subject: [igt-dev] [PATCH i-g-t] tests/kms_psr2_sf: Skip
> > > > > > >tests if selected fetch is not available
> > > > > > >
> > > > > > >Currently the tests are failing on systems where selected
> > > > > > >fetch feature is not availble. Kernel returns -EINVAL when
> > > > > > >the test executes reporting "Invalid debug mask 4".
> > > > > > >Patch skips if selected fetch feature is not available in the kernel.
> > > > > > >
> > > > > > >v2: Incorportated Mark Yacoub's comments.
> > > > > > >Changed igt_skip to igt_require_f
> > > > > > >
> > > > > > >Signed-off-by: Vidya Srinivas <vidya.srinivas at intel.com>
> > > > > > Reviewed-by: Jeevan B<jeevan.b at intel.com>
> > > > > > >---
> > > > > > > lib/igt_psr.c | 2 +-
> > > > > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > > > >
> > > > > > >diff --git a/lib/igt_psr.c b/lib/igt_psr.c index
> > > > > > >68240f9a33b4..0af503fb796e 100644
> > > > > > >--- a/lib/igt_psr.c
> > > > > > >+++ b/lib/igt_psr.c
> > > > > > >@@ -167,7 +167,7 @@ static bool psr_set(int device, int
> > > > > > >debugfs_fd, int
> > > > > > >mode)  }
> > > > > > >
> > > > > > > ret = psr_write(debugfs_fd, debug_val); -igt_assert(ret >
> > > > > > > 0);
> > > > > > >+igt_require_f(ret == 0, "PSR2 SF feature not available\n");
> > > > > > > }
> > > > > > >
> > > > > > > /* Restore original value on exit */
> > > > > > >--
> > > > > > >2.33.0
> > > > > >
> > > > >
> >