[igt-dev] [i-g-t] tests/gem_userptr_blits: Remove vma-merge subtest
Surendrakumar Upadhyay, TejaskumarX
tejaskumarx.surendrakumar.upadhyay at intel.com
Fri Sep 24 05:23:46 UTC 2021
> -----Original Message-----
> From: Surendrakumar Upadhyay, TejaskumarX
> Sent: 07 September 2021 21:31
> To: Daniel Vetter <daniel.vetter at ffwll.ch>
> Cc: IGT development <igt-dev at lists.freedesktop.org>; Lankhorst, Maarten
> <maarten.lankhorst at intel.com>; Ursulin, Tvrtko <tvrtko.ursulin at intel.com>
> Subject: RE: [igt-dev] [i-g-t] tests/gem_userptr_blits: Remove vma-merge
> subtest
>
>
>
> > -----Original Message-----
> > From: Daniel Vetter <daniel.vetter at ffwll.ch>
> > Sent: 31 August 2021 17:07
> > To: Surendrakumar Upadhyay, TejaskumarX
> > <tejaskumarx.surendrakumar.upadhyay at intel.com>
> > Cc: IGT development <igt-dev at lists.freedesktop.org>; Lankhorst,
> > Maarten <maarten.lankhorst at intel.com>; Ursulin, Tvrtko
> > <tvrtko.ursulin at intel.com>
> > Subject: Re: [igt-dev] [i-g-t] tests/gem_userptr_blits: Remove
> > vma-merge subtest
> >
> > On Tue, Aug 31, 2021 at 7:47 AM Surendrakumar Upadhyay, TejaskumarX
> > <tejaskumarx.surendrakumar.upadhyay at intel.com> wrote:
> > >
> > >
> > >
> > > > -----Original Message-----
> > > > From: Daniel Vetter <daniel.vetter at ffwll.ch>
> > > > Sent: 30 August 2021 21:31
> > > > To: Surendrakumar Upadhyay, TejaskumarX
> > > > <tejaskumarx.surendrakumar.upadhyay at intel.com>
> > > > Cc: IGT development <igt-dev at lists.freedesktop.org>; Lankhorst,
> > > > Maarten <maarten.lankhorst at intel.com>; Ursulin, Tvrtko
> > > > <tvrtko.ursulin at intel.com>
> > > > Subject: Re: [igt-dev] [i-g-t] tests/gem_userptr_blits: Remove
> > > > vma-merge subtest
> > > >
> > > > On Tue, Aug 24, 2021 at 11:17 AM Tejas Upadhyay
> > > > <tejaskumarx.surendrakumar.upadhyay at intel.com> wrote:
> > > > >
> > > > > mmap requests for huge page size in vma-merge subtest are
> > > > > cancelled from kernel using i915_request_cancel(rq, -EINTR),
> > > > > otherwise it results in system hang. It is known/expected
> > > > > failure thus it is decided to remove the subtest.
> > > > >
> > > > > Signed-off-by: Tejas Upadhyay
> > > > > <tejaskumarx.surendrakumar.upadhyay at intel.com>
> > > >
> > > > Bugfixes need to link to the gitlab issue, otherwise it's not
> > > > really a bugfix for a CI issue.
> > >
> > > Tejas : Ok.
> > >
> > > >
> > > > Furthermore this is wrong:
> > > > - In DII this might happen. If this is a patch upstream then
> > > > please mark up the DII patch as NOT_UPSTREAM. Dropping it is
> > > > blocked on backporting hangcheck. Your manager should be aware of
> > > > the Jira that tracks hangcheck backporting.
> > >
> > > Tejas : This is only for upstream.
> > >
> > > > - In upstream hangcheck should get us out of this situation. If
> > > > not, then this is a kernel bug.
> > >
> > >
> > > Tejas : Hangcheck watchdog is implemented in kernel by Tvrtko. Hence
> > > now
> > test does not hang, it fails.
> >
> > Ok, but then pls fix the invalid assumption from the testcase, not
> > just delete it. With explanation for why that assumption is invalid,
> > and what should be done instead.
> > -Daniel
>
> Assumption from testcase is not invalid, its just that test scenario is not
> supported with the new userptr invalidation scheme implemented in i915
> commit ed29c2691188 ("drm/i915: Fix userptr so we do not have to worry
> about obj->mm.lock, v7.")". Now please suggest what should be our next
> step?
Anyone has any suggestion here as what should be our next step?
Thanks,
Tejas
>
> Thanks,
> Tejas
>
> >
> > >
> > > >
> > > > Cheers, Daniel
> > > > > ---
> > > > > tests/i915/gem_userptr_blits.c | 40
> > > > > ----------------------------------
> > > > > 1 file changed, 40 deletions(-)
> > > > >
> > > > > diff --git a/tests/i915/gem_userptr_blits.c
> > > > > b/tests/i915/gem_userptr_blits.c index 756bd6e4..11bce6d7 100644
> > > > > --- a/tests/i915/gem_userptr_blits.c
> > > > > +++ b/tests/i915/gem_userptr_blits.c
> > > > > @@ -702,43 +702,6 @@ static size_t hugepagesize(void) #undef
> > > > > LINE }
> > > > >
> > > > > -static void test_vma_merge(int i915) -{
> > > > > - const size_t sz = 2 * hugepagesize();
> > > > > - igt_spin_t *spin;
> > > > > - uint32_t handle;
> > > > > - void *addr;
> > > > > - uint64_t ahnd = get_reloc_ahnd(i915, 0);
> > > > > -
> > > > > - addr = mmap(NULL, sz, PROT_READ | PROT_WRITE,
> > > > > - MAP_SHARED | MAP_ANONYMOUS, -1, 0);
> > > > > -
> > > > > - gem_userptr(i915, addr + sz / 2, 4096, 0, userptr_flags, &handle);
> > > > > -
> > > > > - spin = igt_spin_new(i915, .ahnd = ahnd, .dependency = handle,
> > > > > - .flags = IGT_SPIN_FENCE_OUT);
> > > > > - igt_assert(gem_bo_busy(i915, handle));
> > > > > -
> > > > > - for (size_t x = 0; x < sz; x += 4096) {
> > > > > - if (x == sz / 2)
> > > > > - continue;
> > > > > -
> > > > > - igt_assert(mmap(addr + x, 4096, PROT_READ | PROT_WRITE,
> > > > > - MAP_FIXED | MAP_SHARED | MAP_ANON, -1, 0) !=
> > > > > - MAP_FAILED);
> > > > > - }
> > > > > -
> > > > > - igt_spin_end(spin);
> > > > > - gem_close(i915, handle);
> > > > > -
> > > > > - munmap(addr, sz);
> > > > > -
> > > > > - gem_sync(i915, spin->handle);
> > > > > - igt_assert_eq(sync_fence_status(spin->out_fence), 1);
> > > > > - igt_spin_free(i915, spin);
> > > > > - put_ahnd(ahnd);
> > > > > -}
> > > > > -
> > > > > static void test_huge_split(int i915) {
> > > > > const size_t sz = 2 * hugepagesize(); @@ -2614,9 +2577,6
> > > > > @@ igt_main_args("c:", NULL, help_str, opt_handler, NULL)
> > > > > igt_subtest("nohangcheck")
> > > > > test_nohangcheck_hostile(fd);
> > > > >
> > > > > - igt_subtest("vma-merge")
> > > > > - test_vma_merge(fd);
> > > > > -
> > > > > igt_subtest("huge-split")
> > > > > test_huge_split(fd);
> > > > > }
> > > > > --
> > > > > 2.31.1
> > > > >
> > > >
> > > >
> > > > --
> > > > Daniel Vetter
> > > > Software Engineer, Intel Corporation http://blog.ffwll.ch
> >
> >
> >
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > http://blog.ffwll.ch
More information about the igt-dev
mailing list