[PATCH v3] drm/framebuffer: Acquire internal references on GEM handles

Satadru Pramanik satadru at gmail.com
Mon Jul 7 16:14:45 UTC 2025


Applying this patch to 6.16-rc5 resolves the sleep issue regression from
6.16-rc4 I was having on my MacBookPro11,3 (Mid-2014 15" MacBookPro), which
has the NVIDIA GK107M GPU enabled via the Nouveau driver.

Many thanks,

Satadru

On Mon, Jul 7, 2025 at 9:33 AM Thomas Zimmermann <tzimmermann at suse.de>
wrote:

> Hi
>
> Am 07.07.25 um 15:21 schrieb Christian König:
>
> >>
> >> +#define DRM_FRAMEBUFFER_HAS_HANDLE_REF(_i)  BIT(0u + (_i))
> > Why the "0u + (_i)" here? An macro trick?
>
> You mean why not just BIT(_i)? internal_flags could possibly contain
> additional flags. Just using BIT(_i) would make it look as if it's only
> for those handle refs.
>
> Best regards
> Thomas
>
> >
> > Regards,
> > Christian.
> >
> >> +
> >>   /**
> >>    * struct drm_framebuffer - frame buffer object
> >>    *
> >> @@ -188,6 +191,10 @@ struct drm_framebuffer {
> >>       * DRM_MODE_FB_MODIFIERS.
> >>       */
> >>      int flags;
> >> +    /**
> >> +     * @internal_flags: Framebuffer flags like
> DRM_FRAMEBUFFER_HAS_HANDLE_REF.
> >> +     */
> >> +    unsigned int internal_flags;
> >>      /**
> >>       * @filp_head: Placed on &drm_file.fbs, protected by
> &drm_file.fbs_lock.
> >>       */
>
> --
> --
> Thomas Zimmermann
> Graphics Driver Developer
> SUSE Software Solutions Germany GmbH
> Frankenstrasse 146, 90461 Nuernberg, Germany
> GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
> HRB 36809 (AG Nuernberg)
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20250707/57af4bc6/attachment.htm>


More information about the dri-devel mailing list