Re: [PATCH] Cute feature: colored printk output

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Ingo Molnar
Date: Saturday, October 6, 2007 - 4:08 am

* Jan Engelhardt <jengelh@computergmbh.de> wrote:


looks really good to me!

Reviewed-by: Ingo Molnar <mingo@elte.hu>

small nit:

+       vc->vc_color = printk_color;
+       update_attr(vc);

that should be in a set_vc_color() function and the new code should do:

+	set_vc_color(vc, vc->vc_color);

(and same at the other places that call update_attr() as well)

feature request: would be interesting to have a color table (defined in 
the .config) dependent on message loglevel. That way KERN_CRIT messages 
could be red, KERN_INFO ones white, etc.

	Ingo
-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 12:13 pm)
Re: [PATCH] Cute feature: colored printk output, Lennart Sorensen, (Fri Oct 5, 12:19 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 12:21 pm)
Re: [PATCH] Cute feature: colored printk output, Lennart Sorensen, (Fri Oct 5, 12:24 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 12:32 pm)
Re: [PATCH] Cute feature: colored printk output, Lennart Sorensen, (Fri Oct 5, 12:43 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 12:55 pm)
RE: [PATCH] Cute feature: colored printk output, Medve Emilian-EMMEDVE1, (Fri Oct 5, 1:23 pm)
Re: [PATCH] Cute feature: colored printk output, Krzysztof Halasa, (Fri Oct 5, 4:22 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 4:47 pm)
Re: [PATCH] Cute feature: colored printk output, Krzysztof Halasa, (Fri Oct 5, 5:10 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 5:23 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Fri Oct 5, 5:26 pm)
Re: [PATCH] Cute feature: colored printk output, Ingo Molnar, (Sat Oct 6, 4:08 am)
Re: [PATCH] Cute feature: colored printk output, Krzysztof Halasa, (Sat Oct 6, 4:58 am)
Re: [PATCH] Cute feature: colored printk output, Dave Jones, (Sat Oct 6, 10:16 am)
Re: [PATCH] Cute feature: colored printk output, Oleg Verych, (Sat Oct 6, 11:09 am)
Re: [PATCH] Cute feature: colored printk output, Bill Davidsen, (Sat Oct 6, 12:53 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Sat Oct 6, 1:01 pm)
Re: [PATCH] Cute feature: colored printk output, Miguel Botón, (Sat Oct 6, 2:19 pm)
Re: [PATCH] Cute feature: colored printk output, Bill Davidsen, (Sat Oct 6, 5:50 pm)
Re: [PATCH] Cute feature: colored printk output, Jan Engelhardt, (Sun Oct 7, 1:42 am)