On Fri, Jun 13, 2025 at 12:11:39PM +0000, Saarinen, Jani wrote: > Hi, > > > -----Original Message----- > > From: dri-devel <dri-devel-boun...@lists.freedesktop.org> On Behalf Of Jani > > Nikula > > Sent: Friday, 13 June 2025 14.02 > > To: Tvrtko Ursulin <tursu...@ursulin.net>; Simona Vetter > > <simona.vet...@ffwll.ch>; Christian König > > <ckoenig.leichtzumer...@gmail.com> > > Cc: tzimmerm...@suse.de; dri-devel@lists.freedesktop.org > > Subject: Re: [PATCH] drm/prime: remove drm_prime_lookup_buf_by_handle > > > > On Fri, 13 Jun 2025, Tvrtko Ursulin <tursu...@ursulin.net> wrote: > > > On 13/06/2025 11:09, Jani Nikula wrote: > > >> On Wed, 04 Jun 2025, Simona Vetter <simona.vet...@ffwll.ch> wrote: > > >>> On Wed, Jun 04, 2025 at 05:36:22PM +0200, Simona Vetter wrote: > > >> This regressed one of our CI IGT tests [1]. > > >> > > >> BR, > > >> Jani. > > >> > > >> > > >> [1] https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/14463 > > > > > > It also explodes even more trivially when logging into a KDE Wayland > > > session: > > > > Smells like a revert, and back to the drawing board, perhaps? > I would say so. Looks like this on our CI > https://intel-gfx-ci.01.org/tree/drm-tip/igt@prime_self_import@basic-with_one_bo.html > > And systems stop testing anything after (see eg > https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16692/bat-twl-1/igt_runner0.txt > ) when aborts happens.
Oops, sorry, I should have asked for a CI run to make sure, because I did review igt_prime test coverage a bit while looking at this patch to make sure we're not screwing up too badly. -Sima > > > > > > > BR, > > Jani. > > Br > Other Jani > > > > > > -- > > Jani Nikula, Intel -- Simona Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch