diff options
| author | Amit Shah <amit.shah@redhat.com> | 2013-04-15 12:00:15 +0930 |
|---|---|---|
| committer | Rusty Russell <rusty@rustcorp.com.au> | 2013-04-15 15:17:39 +0930 |
| commit | 74ff582cd65ad01c45f1971feac28f23b7eb2687 (patch) | |
| tree | d3a4222a562e7ee16b5ac053461fb5132a3b63f6 /drivers/char/virtio_console.c | |
| parent | 285e71ea6f3583a85e27cb2b9a7d8c35d4c0d558 (diff) | |
| download | op-kernel-dev-74ff582cd65ad01c45f1971feac28f23b7eb2687.zip op-kernel-dev-74ff582cd65ad01c45f1971feac28f23b7eb2687.tar.gz | |
virtio: console: replace EMFILE with EBUSY for already-open port
Returning EMFILE (process has too many open files) is incorrect to
indicate a port is already open by another process. Use EBUSY for that.
This does change what we report to userspace, but I believe userspace
can look at it this way: it gets EBUSY, a new error code, instead of
EMFILE. It's still an error, and that's not changing.
Reported-by: Mateusz Guzik <mguzik@redhat.com>
Signed-off-by: Amit Shah <amit.shah@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'drivers/char/virtio_console.c')
| -rw-r--r-- | drivers/char/virtio_console.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c index f4f31fe..5ee7765 100644 --- a/drivers/char/virtio_console.c +++ b/drivers/char/virtio_console.c @@ -1036,7 +1036,7 @@ static int port_fops_open(struct inode *inode, struct file *filp) spin_lock_irq(&port->inbuf_lock); if (port->guest_connected) { spin_unlock_irq(&port->inbuf_lock); - ret = -EMFILE; + ret = -EBUSY; goto out; } |
