diff options
author | Jeff Dike <jdike@addtoit.com> | 2007-07-16 02:38:54 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-16 12:05:38 -0400 |
commit | d14ad81f800a57d3f21f8e98556c728968883e9a (patch) | |
tree | cbe17039283dd51654aaa5a9ac4fae9233285d96 /arch/um/drivers/stdio_console.c | |
parent | 75886f21e3a53a298b097103fd52e83184a89c22 (diff) |
uml: handle errors on opening host side of consoles
If the host side of a console can't be opened, this will now produce visible
error messages.
enable_chan now returns a status and this is passed up to con_open and
ssl_open, which will complain if anything went wrong.
The default host device for the serial line driver is now a pts device rather
than a pty device since lots of hosts have LEGACY_PTYS disabled. This had
always been failing on such hosts, but silently.
Signed-off-by: Jeff Dike <jdike@linux.intel.com>
Cc: Paolo 'Blaisorblade' Giarrusso <blaisorblade@yahoo.it>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/um/drivers/stdio_console.c')
-rw-r--r-- | arch/um/drivers/stdio_console.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/arch/um/drivers/stdio_console.c b/arch/um/drivers/stdio_console.c index e312488a7a99..656036e90b19 100644 --- a/arch/um/drivers/stdio_console.c +++ b/arch/um/drivers/stdio_console.c | |||
@@ -99,7 +99,12 @@ static int con_remove(int n, char **error_out) | |||
99 | 99 | ||
100 | static int con_open(struct tty_struct *tty, struct file *filp) | 100 | static int con_open(struct tty_struct *tty, struct file *filp) |
101 | { | 101 | { |
102 | return line_open(vts, tty); | 102 | int err = line_open(vts, tty); |
103 | if (err) | ||
104 | printk(KERN_ERR "Failed to open console %d, err = %d\n", | ||
105 | tty->index, err); | ||
106 | |||
107 | return err; | ||
103 | } | 108 | } |
104 | 109 | ||
105 | /* Set in an initcall, checked in an exitcall */ | 110 | /* Set in an initcall, checked in an exitcall */ |