linux-kernel - Re: [PATCH] drm/fb-helper: Add force parameter to stub function

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <91edf43f-108b-45ab-af7d-097097ff6c24@suse.de>
Date: Wed, 14 Jan 2026 17:07:32 +0100
From: Thomas Zimmermann <tzimmermann@...e.de>
To: Matt Coster <matt.coster@...tec.com>,
 Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
 Maxime Ripard <mripard@...nel.org>, David Airlie <airlied@...il.com>,
 Simona Vetter <simona@...ll.ch>, Jocelyn Falempe <jfalempe@...hat.com>
Cc: Frank Binns <frank.binns@...tec.com>,
 Karolis Mituzas <karolis.mituzas@...tec.com>,
 dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/fb-helper: Add force parameter to stub function

Hi

Am 14.01.26 um 15:57 schrieb Matt Coster:
> The signature of drm_fb_helper_restore_fbdev_mode_unlocked() was modified
> in commit 943240d342f14 ("drm/client: Pass force parameter to client
> restore") to add a force parameter, but this was not added to the stub
> equivalent for when CONFIG_DRM_FBDEV_EMULATION is not enabled.

The only caller depends on CONFIG_DRM_FBDEV_EMULATION, so the stub 
should never be called. If possible, remove it entirely.

Most of these stub functions are left overs from when fbdev emulation 
was less well isolated from the rest of DRM.

Best regards
Thomas

>
> Suggested-by: Karolis Mituzas <karolis.mituzas@...tec.com>
> Fixes: 943240d342f14 ("drm/client: Pass force parameter to client restore")
> Signed-off-by: Matt Coster <matt.coster@...tec.com>
> ---
>   include/drm/drm_fb_helper.h | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/drm/drm_fb_helper.h b/include/drm/drm_fb_helper.h
> index dd9a18f8de5a7..eebb36a099698 100644
> --- a/include/drm/drm_fb_helper.h
> +++ b/include/drm/drm_fb_helper.h
> @@ -333,7 +333,8 @@ static inline int drm_fb_helper_check_var(struct fb_var_screeninfo *var,
>   }
>   
>   static inline int
> -drm_fb_helper_restore_fbdev_mode_unlocked(struct drm_fb_helper *fb_helper)
> +drm_fb_helper_restore_fbdev_mode_unlocked(struct drm_fb_helper *fb_helper,
> +					  bool force)
>   {
>   	return 0;
>   }
>
> ---
> base-commit: b91a565ed14fcf900b4d95e86882b4b763860986
> change-id: 20260114-drm-fb-helper-stub-fix-03f07bd2be61
>

-- 
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstr. 146, 90461 Nürnberg, Germany, www.suse.com
GF: Jochen Jaser, Andrew McDonald, Werner Knoblich, (HRB 36809, AG Nürnberg)