[igt-dev] [PATCH v3 1/1] test/gem_create: limit set_pat only to Meteor Lake
Andi Shyti
andi.shyti at linux.intel.com
Wed Jul 12 08:18:47 UTC 2023
Hi,
On Wed, Jul 12, 2023 at 09:09:55AM +0100, Tvrtko Ursulin wrote:
>
> On 07/06/2023 09:57, Kamil Konieczny wrote:
> > On 2023-06-06 at 19:52:26 +0200, Andi Shyti wrote:
> > > From: Fei Yang <fei.yang at intel.com>
> > >
> > > For platforms other than Meteor Lake the set_pat extension should
> > > return -ENODEV.
> > >
> > > Signed-off-by: Fei Yang <fei.yang at intel.com>
> > > Signed-off-by: Andi Shyti <andi.shyti at linux.intel.com>
> > > ---
> > > tests/i915/gem_create.c | 4 ++++
> > > 1 file changed, 4 insertions(+)
> > >
> > > diff --git a/tests/i915/gem_create.c b/tests/i915/gem_create.c
> > > index e9e8216d2b..b7961d9ef2 100644
> > > --- a/tests/i915/gem_create.c
> > > +++ b/tests/i915/gem_create.c
> > > @@ -657,6 +657,7 @@ static void create_ext_set_pat(int fd)
> > > .base = { .name = I915_GEM_CREATE_EXT_SET_PAT },
> > > .pat_index = 65,
> > > };
> > > + uint32_t devid = intel_get_drm_devid(fd);
> > > struct drm_i915_gem_caching arg;
> > > uint64_t size;
> > > uint32_t handle;
> > > @@ -672,6 +673,9 @@ static void create_ext_set_pat(int fd)
> > > */
> > > if (ret == -EINVAL)
> > > igt_skip("I915_GEM_CREATE_EXT_SET_PAT is not supported\n");
> > > + else if (!IS_METEORLAKE(devid))
> > > + igt_assert_eq(ret, -ENODEV);
right, we should have returned from here, if we want to have
differentiated printouts...
> Hmm a sea of red..
>
> https://intel-gfx-ci.01.org/tree/drm-tip/index.html?testfilter=set-pat
>
> https://intel-gfx-ci.01.org/tree/drm-intel-fixes/CI_DIF_773/shard-snb7/igt@gem_create@create-ext-set-pat.html
>
> Starting subtest: create-ext-set-pat
> (gem_create:1095) CRITICAL: Test assertion failure function create_ext_set_pat, file ../../../usr/src/igt-gpu-tools/tests/i915/gem_create.c:679:
> (gem_create:1095) CRITICAL: Failed assertion: ret == 0
> Subtest create-ext-set-pat failed.
>
> Not sure why shards run did not flag this (?!), but AFAICT the logic does not work.
>
> It should be igt_skip on MTL, otherwise..
>
> > > +
> >
> > lgtm,
> >
> > Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> >
> > > igt_assert(ret == 0);
>
> .. test just fails here. So something like:
>
> igt_skip_on_f(ret == -EINVAL || (ret == -ENODEV && IS_METEORLAKE(devid)), "...")
But I guess this is also right!
I will fix it.
Thanks, Tvrtko!
Andi
> ?
>
> Regards,
>
> Tvrtko
>
> > > /*
> > > --
> > > 2.40.1
> > >
More information about the igt-dev
mailing list