Re: [PATCH] drm/prime: remove drm_prime_lookup_buf_by_handle

[Simona Vetter <simona.vetter@ffwll.ch>]

On Fri, Jun 13, 2025 at 12:11:39PM +0000, Saarinen, Jani wrote:
> Hi, 
> 
> > -----Original Message-----
> > From: dri-devel <dri-devel-bounces@lists.freedesktop.org> On Behalf Of Jani
> > Nikula
> > Sent: Friday, 13 June 2025 14.02
> > To: Tvrtko Ursulin <tursulin@ursulin.net>; Simona Vetter
> > <simona.vetter@ffwll.ch>; Christian König
> > <ckoenig.leichtzumerken@gmail.com>
> > Cc: tzimmermann@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 <tursulin@ursulin.net> wrote:
> > > On 13/06/2025 11:09, Jani Nikula wrote:
> > >> On Wed, 04 Jun 2025, Simona Vetter <simona.vetter@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