* [PATCH] fbcon: Remove obsolete reference to initmem_freed
@ 2022-06-08 15:41 ` Geert Uytterhoeven
0 siblings, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2022-06-08 15:41 UTC (permalink / raw)
To: Helge Deller; +Cc: linux-fbdev, dri-devel, Geert Uytterhoeven
initmem_freed was removed in v2.1.124, and the underlying issue was
fixed for good in commit 92b004d1aa9f367c ("video/logo: prevent use of
logos after they have been freed").
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/video/fbdev/core/fbcon.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c
index 594650452afd14d8..1be8aa9f80747987 100644
--- a/drivers/video/fbdev/core/fbcon.c
+++ b/drivers/video/fbdev/core/fbcon.c
@@ -2180,7 +2180,6 @@ static int fbcon_switch(struct vc_data *vc)
if (logo_shown == FBCON_LOGO_DRAW) {
logo_shown = fg_console;
- /* This is protected above by initmem_freed */
fb_show_logo(info, ops->rotate);
update_region(vc,
vc->vc_origin + vc->vc_size_row * vc->vc_top,
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] fbcon: Remove obsolete reference to initmem_freed
@ 2022-06-08 15:41 ` Geert Uytterhoeven
0 siblings, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2022-06-08 15:41 UTC (permalink / raw)
To: Helge Deller; +Cc: linux-fbdev, Geert Uytterhoeven, dri-devel
initmem_freed was removed in v2.1.124, and the underlying issue was
fixed for good in commit 92b004d1aa9f367c ("video/logo: prevent use of
logos after they have been freed").
Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
drivers/video/fbdev/core/fbcon.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c
index 594650452afd14d8..1be8aa9f80747987 100644
--- a/drivers/video/fbdev/core/fbcon.c
+++ b/drivers/video/fbdev/core/fbcon.c
@@ -2180,7 +2180,6 @@ static int fbcon_switch(struct vc_data *vc)
if (logo_shown == FBCON_LOGO_DRAW) {
logo_shown = fg_console;
- /* This is protected above by initmem_freed */
fb_show_logo(info, ops->rotate);
update_region(vc,
vc->vc_origin + vc->vc_size_row * vc->vc_top,
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] fbcon: Remove obsolete reference to initmem_freed
2022-06-08 15:41 ` Geert Uytterhoeven
(?)
@ 2022-06-08 16:40 ` Daniel Vetter
-1 siblings, 0 replies; 3+ messages in thread
From: Daniel Vetter @ 2022-06-08 16:40 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: Helge Deller, linux-fbdev, dri-devel
On Wed, Jun 08, 2022 at 05:41:16PM +0200, Geert Uytterhoeven wrote:
> initmem_freed was removed in v2.1.124, and the underlying issue was
> fixed for good in commit 92b004d1aa9f367c ("video/logo: prevent use of
> logos after they have been freed").
>
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
So I got totally lost in late_initcall_sync trying to figure out whether
there's any fun races against driver load, and I guess as long as you
don't manage to hotplug an usb display link at just the right time this
works.
But also that's all just the existing logo.c code, so pure distraction.
Thanks for your patch, pushed to drm-misc-next.
-Daniel
> ---
> drivers/video/fbdev/core/fbcon.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c
> index 594650452afd14d8..1be8aa9f80747987 100644
> --- a/drivers/video/fbdev/core/fbcon.c
> +++ b/drivers/video/fbdev/core/fbcon.c
> @@ -2180,7 +2180,6 @@ static int fbcon_switch(struct vc_data *vc)
> if (logo_shown == FBCON_LOGO_DRAW) {
>
> logo_shown = fg_console;
> - /* This is protected above by initmem_freed */
> fb_show_logo(info, ops->rotate);
> update_region(vc,
> vc->vc_origin + vc->vc_size_row * vc->vc_top,
> --
> 2.25.1
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-06-08 16:41 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-08 15:41 [PATCH] fbcon: Remove obsolete reference to initmem_freed Geert Uytterhoeven
2022-06-08 15:41 ` Geert Uytterhoeven
2022-06-08 16:40 ` Daniel Vetter
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.