| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354 |
- From 53b5c5f559cc95fc676cb4cc9e5a8d9f0cb8f58c Mon Sep 17 00:00:00 2001
- Message-Id: <53b5c5f559cc95fc676cb4cc9e5a8d9f0cb8f58c.1659815468.git.chunkeey@gmail.com>
- From: Christian Lamparter <[email protected]>
- Date: Sat, 6 Aug 2022 21:22:01 +0200
- Subject: [PATCH v1 1/2] um: fix default console kernel parameter
- To: [email protected]
- Cc: Richard Weinberger <[email protected]>,
- Anton Ivanov <[email protected]>,
- Johannes Berg <[email protected]>
- OpenWrt's UML with 5.15 was producing odd errors/warnings during preinit
- part of the early userspace portion:
- |[ 0.000000] Kernel command line: ubd0=root.img root=98:0 console=tty
- |[...]
- |[ 0.440000] random: jshn: uninitialized urandom read (4 bytes read)
- |[ 0.460000] random: jshn: uninitialized urandom read (4 bytes read)
- |/etc/preinit: line 47: can't create /dev/tty: No such device or address
- |/etc/preinit: line 48: can't create /dev/tty: No such device or address
- |/etc/preinit: line 58: can't open /dev/tty: No such device or address
- |[...] repeated many times
- That "/dev/tty" came from the command line (which is automatically
- added if no console= parameter was specified for the uml binary).
- The TLDP project tells the following about the /dev/tty:
- <https://tldp.org/HOWTO/Text-Terminal-HOWTO-7.html#ss7.3>
- | /dev/tty stands for the controlling terminal (if any) for the current
- | process.[...]
- | /dev/tty is something like a link to the actually terminal device[..]
- The "(if any)" is important here, since it's possible for processes to
- not have a controlling terminal.
- I think this was a simple typo and the author wanted tty0 there.
- CC: Thomas Meyer <[email protected]>
- Fixes: d7ffac33631b ("um: stdio_console: Make preferred console")
- Signed-off-by: Christian Lamparter <[email protected]>
- ---
- arch/um/kernel/um_arch.c | 2 +-
- 1 file changed, 1 insertion(+), 1 deletion(-)
- --- a/arch/um/kernel/um_arch.c
- +++ b/arch/um/kernel/um_arch.c
- @@ -31,7 +31,7 @@
- #include <os.h>
-
- #define DEFAULT_COMMAND_LINE_ROOT "root=98:0"
- -#define DEFAULT_COMMAND_LINE_CONSOLE "console=tty"
- +#define DEFAULT_COMMAND_LINE_CONSOLE "console=tty0"
-
- /* Changed in add_arg and setup_arch, which run before SMP is started */
- static char __initdata command_line[COMMAND_LINE_SIZE] = { 0 };
|