Re: [PATCH] Stop printk printing non-printable chars

From: David Woodhouse
Date: Sat Jun 19 2004 - 06:19:28 EST


On Fri, 2004-06-18 at 21:53 +0100, matthew-lkml@xxxxxxxxxxxxxxxxxxxxx
wrote:
> The main problem seems to be in ACPI, but I don't see any reason for
> printk to even consider printing _any_ non-printable characters at all.
> It makes all characters out of the range 32..126 (except for newline)
> print as a '?'.

Please don't do that -- it makes printing UTF-8 impossible. While I'd
not argue that now is the time to start outputting UTF-8 all over the
place, I wouldn't accept that it's a good time to _prevent_ it either,
as your patch would do.

If you want to post-process printk output, don't do it in the kernel.

I'd suggest that in this instance you should be fixing the ACPI code
instead, so it doesn't print the characters to which you object.

--
dwmw2


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/