<div><p style="font-size:50px;margin-top:0;margin-bottom:0">🙏</p><p style="margin-top:10px;margin-bottom:0">Satadru reacted via <a style="color:unset;text-decoration:underline" href="https://www.google.com/gmail/about/?utm_source=gmail-in-product&utm_medium=et&utm_campaign=emojireactionemail#app">Gmail</a></p></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, Jul 8, 2025, 03:38 Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de">tzimmermann@suse.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi<br>
<br>
Am 07.07.25 um 18:14 schrieb Satadru Pramanik:<br>
> Applying this patch to 6.16-rc5 resolves the sleep issue regression <br>
> from 6.16-rc4 I was having on my MacBookPro11,3 (Mid-2014 15" <br>
> MacBookPro), which has the NVIDIA GK107M GPU enabled via the Nouveau <br>
> driver.<br>
<br>
Thanks for testing. I think the sleep regression was just a side effect <br>
of the broken reference counting.<br>
<br>
Best regards<br>
Thomas<br>
<br>
><br>
> Many thanks,<br>
><br>
> Satadru<br>
><br>
> On Mon, Jul 7, 2025 at 9:33 AM Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de" target="_blank" rel="noreferrer">tzimmermann@suse.de</a>> <br>
> wrote:<br>
><br>
>     Hi<br>
><br>
>     Am 07.07.25 um 15:21 schrieb Christian König:<br>
><br>
>     >><br>
>     >> +#define DRM_FRAMEBUFFER_HAS_HANDLE_REF(_i)  BIT(0u + (_i))<br>
>     > Why the "0u + (_i)" here? An macro trick?<br>
><br>
>     You mean why not just BIT(_i)? internal_flags could possibly contain<br>
>     additional flags. Just using BIT(_i) would make it look as if it's<br>
>     only<br>
>     for those handle refs.<br>
><br>
>     Best regards<br>
>     Thomas<br>
><br>
>     ><br>
>     > Regards,<br>
>     > Christian.<br>
>     ><br>
>     >> +<br>
>     >>   /**<br>
>     >>    * struct drm_framebuffer - frame buffer object<br>
>     >>    *<br>
>     >> @@ -188,6 +191,10 @@ struct drm_framebuffer {<br>
>     >>       * DRM_MODE_FB_MODIFIERS.<br>
>     >>       */<br>
>     >>      int flags;<br>
>     >> +    /**<br>
>     >> +     * @internal_flags: Framebuffer flags like<br>
>     DRM_FRAMEBUFFER_HAS_HANDLE_REF.<br>
>     >> +     */<br>
>     >> +    unsigned int internal_flags;<br>
>     >>      /**<br>
>     >>       * @filp_head: Placed on &drm_file.fbs, protected by<br>
>     &drm_file.fbs_lock.<br>
>     >>       */<br>
><br>
>     -- <br>
>     --<br>
>     Thomas Zimmermann<br>
>     Graphics Driver Developer<br>
>     SUSE Software Solutions Germany GmbH<br>
>     Frankenstrasse 146, 90461 Nuernberg, Germany<br>
>     GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman<br>
>     HRB 36809 (AG Nuernberg)<br>
><br>
<br>
-- <br>
--<br>
Thomas Zimmermann<br>
Graphics Driver Developer<br>
SUSE Software Solutions Germany GmbH<br>
Frankenstrasse 146, 90461 Nuernberg, Germany<br>
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman<br>
HRB 36809 (AG Nuernberg)<br>
<br>
</blockquote></div>