<div dir="ltr"><div>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.</div><div><br></div><div>Many thanks,</div><div><br></div><div>Satadru</div></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Mon, Jul 7, 2025 at 9:33 AM Thomas Zimmermann <<a href="mailto:tzimmermann@suse.de">tzimmermann@suse.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">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 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 DRM_FRAMEBUFFER_HAS_HANDLE_REF.<br>
>> +     */<br>
>> +    unsigned int internal_flags;<br>
>>      /**<br>
>>       * @filp_head: Placed on &drm_file.fbs, protected by &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>
</blockquote></div>