[systemd-devel] [PATCH 1/2] vconsole: match on vtcon events, not fbcon ones

[Zbigniew Jędrzejewski-Szmek]

On Tue, Feb 24, 2015 at 05:49:02PM +0100, Jan Engelhardt wrote:
> I observe that upon loading of framebuffer drivers, I do not get the
> desired system font, but the kernel-level defaults (usually
> lib/fonts/font_8x16.c, but your mileage may vary depending on kernel
> config and boot options).
> 
> The fbcon driver may be loaded at a time way before the first
> framebuffer device is active, such that the vconsole setup helper
> runs too early.
> 
> The existing rule is non-fitting. The going live of the fbcon kernel
> component does not indicate the proper time at which to load the
> visuals, which really ought to be done when a new vtcon object comes
> into existence. (The font table is a per-vtcon property.)
OK, but what happens when fbcon is loaded afterwards? Don't we want to
keep both rules to be safe? (Sorry if I'm not making any sense. I don't
really understand how it all fits together.)

Zbyszek

> ---
>  src/vconsole/90-vconsole.rules.in | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/src/vconsole/90-vconsole.rules.in b/src/vconsole/90-vconsole.rules.in
> index 0620096..35b9ad5 100644
> --- a/src/vconsole/90-vconsole.rules.in
> +++ b/src/vconsole/90-vconsole.rules.in
> @@ -5,7 +5,6 @@
>  #  the Free Software Foundation; either version 2.1 of the License, or
>  #  (at your option) any later version.
>  
> -# Kernel resets vconsole state when changing console drivers so run
> -# systemd-vconsole-setup when fbcon loads
> -
> -ACTION=="add", SUBSYSTEM=="graphics", KERNEL=="fbcon", RUN+="@rootlibexecdir@/systemd-vconsole-setup"
> +# Each vtcon keeps its own state of fonts.
> +#
> +ACTION=="add", SUBSYSTEM=="vtconsole", KERNEL=="vtcon*", RUN+="@rootlibexecdir@/systemd-vconsole-setup"
> -- 
> 2.1.4
> 
> _______________________________________________
> systemd-devel mailing list
> systemd-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/systemd-devel