sharkcz / rpms / kernel

Forked from rpms/kernel 6 years ago
Clone
Jesse Keating 3494df0
diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c
Jesse Keating 3494df0
index 48ce834..8c99bf1 100644
Jesse Keating 3494df0
--- a/drivers/char/virtio_console.c
Jesse Keating 3494df0
+++ b/drivers/char/virtio_console.c
Jesse Keating 3494df0
@@ -33,35 +33,6 @@
Jesse Keating 3494df0
 #include <linux/workqueue.h>
Jesse Keating 3494df0
 #include "hvc_console.h"
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-/* Moved here from .h file in order to disable MULTIPORT. */
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_F_MULTIPORT 1	/* Does host provide multiple ports? */
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-struct virtio_console_multiport_conf {
Jesse Keating 3494df0
-	struct virtio_console_config config;
Jesse Keating 3494df0
-	/* max. number of ports this device can hold */
Jesse Keating 3494df0
-	__u32 max_nr_ports;
Jesse Keating 3494df0
-	/* number of ports added so far */
Jesse Keating 3494df0
-	__u32 nr_ports;
Jesse Keating 3494df0
-} __attribute__((packed));
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-/*
Jesse Keating 3494df0
- * A message that's passed between the Host and the Guest for a
Jesse Keating 3494df0
- * particular port.
Jesse Keating 3494df0
- */
Jesse Keating 3494df0
-struct virtio_console_control {
Jesse Keating 3494df0
-	__u32 id;		/* Port number */
Jesse Keating 3494df0
-	__u16 event;		/* The kind of control event (see below) */
Jesse Keating 3494df0
-	__u16 value;		/* Extra information for the key */
Jesse Keating 3494df0
-};
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-/* Some events for control messages */
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_PORT_READY	0
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_CONSOLE_PORT	1
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_RESIZE		2
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_PORT_OPEN	3
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_PORT_NAME	4
Jesse Keating 3494df0
-#define VIRTIO_CONSOLE_PORT_REMOVE	5
Jesse Keating 3494df0
-
Jesse Keating 3494df0
 /*
Jesse Keating 3494df0
  * This is a global struct for storing common data for all the devices
Jesse Keating 3494df0
  * this driver handles.
Jesse Keating 3494df0
@@ -107,6 +78,9 @@ struct console {
Jesse Keating 3494df0
 	/* The hvc device associated with this console port */
Jesse Keating 3494df0
 	struct hvc_struct *hvc;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	/* The size of the console */
Jesse Keating 3494df0
+	struct winsize ws;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	/*
Jesse Keating 3494df0
 	 * This number identifies the number that we used to register
Jesse Keating 3494df0
 	 * with hvc in hvc_instantiate() and hvc_alloc(); this is the
Jesse Keating 3494df0
@@ -139,7 +113,6 @@ struct ports_device {
Jesse Keating 3494df0
 	 * notification
Jesse Keating 3494df0
 	 */
Jesse Keating 3494df0
 	struct work_struct control_work;
Jesse Keating 3494df0
-	struct work_struct config_work;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	struct list_head ports;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -150,7 +123,7 @@ struct ports_device {
Jesse Keating 3494df0
 	spinlock_t cvq_lock;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	/* The current config space is stored here */
Jesse Keating 3494df0
-	struct virtio_console_multiport_conf config;
Jesse Keating 3494df0
+	struct virtio_console_config config;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	/* The virtio device we're associated with */
Jesse Keating 3494df0
 	struct virtio_device *vdev;
Jesse Keating 3494df0
@@ -189,6 +162,9 @@ struct port {
Jesse Keating 3494df0
 	 */
Jesse Keating 3494df0
 	spinlock_t inbuf_lock;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	/* Protect the operations on the out_vq. */
Jesse Keating 3494df0
+	spinlock_t outvq_lock;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	/* The IO vqs for this port */
Jesse Keating 3494df0
 	struct virtqueue *in_vq, *out_vq;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -214,6 +190,8 @@ struct port {
Jesse Keating 3494df0
 	/* The 'id' to identify the port with the Host */
Jesse Keating 3494df0
 	u32 id;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	bool outvq_full;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	/* Is the host device open */
Jesse Keating 3494df0
 	bool host_connected;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -403,22 +381,22 @@ out:
Jesse Keating 3494df0
 	return ret;
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-static ssize_t send_control_msg(struct port *port, unsigned int event,
Jesse Keating 3494df0
-				unsigned int value)
Jesse Keating 3494df0
+static ssize_t __send_control_msg(struct ports_device *portdev, u32 port_id,
Jesse Keating 3494df0
+				  unsigned int event, unsigned int value)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
 	struct scatterlist sg[1];
Jesse Keating 3494df0
 	struct virtio_console_control cpkt;
Jesse Keating 3494df0
 	struct virtqueue *vq;
Jesse Keating 3494df0
 	unsigned int len;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	if (!use_multiport(port->portdev))
Jesse Keating 3494df0
+	if (!use_multiport(portdev))
Jesse Keating 3494df0
 		return 0;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	cpkt.id = port->id;
Jesse Keating 3494df0
+	cpkt.id = port_id;
Jesse Keating 3494df0
 	cpkt.event = event;
Jesse Keating 3494df0
 	cpkt.value = value;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	vq = port->portdev->c_ovq;
Jesse Keating 3494df0
+	vq = portdev->c_ovq;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	sg_init_one(sg, &cpkt, sizeof(cpkt));
Jesse Keating 3494df0
 	if (virtqueue_add_buf(vq, sg, 1, 0, &cpkt) >= 0) {
Jesse Keating 3494df0
@@ -429,15 +407,39 @@ static ssize_t send_control_msg(struct port *port, unsigned int event,
Jesse Keating 3494df0
 	return 0;
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-static ssize_t send_buf(struct port *port, void *in_buf, size_t in_count)
Jesse Keating 3494df0
+static ssize_t send_control_msg(struct port *port, unsigned int event,
Jesse Keating 3494df0
+				unsigned int value)
Jesse Keating 3494df0
+{
Jesse Keating 3494df0
+	return __send_control_msg(port->portdev, port->id, event, value);
Jesse Keating 3494df0
+}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+/* Callers must take the port->outvq_lock */
Jesse Keating 3494df0
+static void reclaim_consumed_buffers(struct port *port)
Jesse Keating 3494df0
+{
Jesse Keating 3494df0
+	void *buf;
Jesse Keating 3494df0
+	unsigned int len;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	while ((buf = virtqueue_get_buf(port->out_vq, &len))) {
Jesse Keating 3494df0
+		kfree(buf);
Jesse Keating 3494df0
+		port->outvq_full = false;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+static ssize_t send_buf(struct port *port, void *in_buf, size_t in_count,
Jesse Keating 3494df0
+			bool nonblock)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
 	struct scatterlist sg[1];
Jesse Keating 3494df0
 	struct virtqueue *out_vq;
Jesse Keating 3494df0
 	ssize_t ret;
Jesse Keating 3494df0
+	unsigned long flags;
Jesse Keating 3494df0
 	unsigned int len;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	out_vq = port->out_vq;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	spin_lock_irqsave(&port->outvq_lock, flags);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	reclaim_consumed_buffers(port);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	sg_init_one(sg, in_buf, in_count);
Jesse Keating 3494df0
 	ret = virtqueue_add_buf(out_vq, sg, 1, 0, in_buf);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -446,14 +448,29 @@ static ssize_t send_buf(struct port *port, void *in_buf, size_t in_count)
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	if (ret < 0) {
Jesse Keating 3494df0
 		in_count = 0;
Jesse Keating 3494df0
-		goto fail;
Jesse Keating 3494df0
+		goto done;
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	/* Wait till the host acknowledges it pushed out the data we sent. */
Jesse Keating 3494df0
+	if (ret == 0)
Jesse Keating 3494df0
+		port->outvq_full = true;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	if (nonblock)
Jesse Keating 3494df0
+		goto done;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * Wait till the host acknowledges it pushed out the data we
Jesse Keating 3494df0
+	 * sent.  This is done for ports in blocking mode or for data
Jesse Keating 3494df0
+	 * from the hvc_console; the tty operations are performed with
Jesse Keating 3494df0
+	 * spinlocks held so we can't sleep here.
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
 	while (!virtqueue_get_buf(out_vq, &len))
Jesse Keating 3494df0
 		cpu_relax();
Jesse Keating 3494df0
-fail:
Jesse Keating 3494df0
-	/* We're expected to return the amount of data we wrote */
Jesse Keating 3494df0
+done:
Jesse Keating 3494df0
+	spin_unlock_irqrestore(&port->outvq_lock, flags);
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * We're expected to return the amount of data we wrote -- all
Jesse Keating 3494df0
+	 * of it
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
 	return in_count;
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -503,9 +520,28 @@ static ssize_t fill_readbuf(struct port *port, char *out_buf, size_t out_count,
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 /* The condition that must be true for polling to end */
Jesse Keating 3494df0
-static bool wait_is_over(struct port *port)
Jesse Keating 3494df0
+static bool will_read_block(struct port *port)
Jesse Keating 3494df0
+{
Jesse Keating 3494df0
+	return !port_has_data(port) && port->host_connected;
Jesse Keating 3494df0
+}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+static bool will_write_block(struct port *port)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
-	return port_has_data(port) || !port->host_connected;
Jesse Keating 3494df0
+	bool ret;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	if (!port->host_connected)
Jesse Keating 3494df0
+		return true;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	spin_lock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * Check if the Host has consumed any buffers since we last
Jesse Keating 3494df0
+	 * sent data (this is only applicable for nonblocking ports).
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
+	reclaim_consumed_buffers(port);
Jesse Keating 3494df0
+	ret = port->outvq_full;
Jesse Keating 3494df0
+	spin_unlock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	return ret;
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 static ssize_t port_fops_read(struct file *filp, char __user *ubuf,
Jesse Keating 3494df0
@@ -528,7 +564,7 @@ static ssize_t port_fops_read(struct file *filp, char __user *ubuf,
Jesse Keating 3494df0
 			return -EAGAIN;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 		ret = wait_event_interruptible(port->waitqueue,
Jesse Keating 3494df0
-					       wait_is_over(port));
Jesse Keating 3494df0
+					       !will_read_block(port));
Jesse Keating 3494df0
 		if (ret < 0)
Jesse Keating 3494df0
 			return ret;
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
@@ -554,9 +590,22 @@ static ssize_t port_fops_write(struct file *filp, const char __user *ubuf,
Jesse Keating 3494df0
 	struct port *port;
Jesse Keating 3494df0
 	char *buf;
Jesse Keating 3494df0
 	ssize_t ret;
Jesse Keating 3494df0
+	bool nonblock;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	port = filp->private_data;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	nonblock = filp->f_flags & O_NONBLOCK;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	if (will_write_block(port)) {
Jesse Keating 3494df0
+		if (nonblock)
Jesse Keating 3494df0
+			return -EAGAIN;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+		ret = wait_event_interruptible(port->waitqueue,
Jesse Keating 3494df0
+					       !will_write_block(port));
Jesse Keating 3494df0
+		if (ret < 0)
Jesse Keating 3494df0
+			return ret;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	count = min((size_t)(32 * 1024), count);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	buf = kmalloc(count, GFP_KERNEL);
Jesse Keating 3494df0
@@ -569,9 +618,14 @@ static ssize_t port_fops_write(struct file *filp, const char __user *ubuf,
Jesse Keating 3494df0
 		goto free_buf;
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	ret = send_buf(port, buf, count);
Jesse Keating 3494df0
+	ret = send_buf(port, buf, count, nonblock);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	if (nonblock && ret > 0)
Jesse Keating 3494df0
+		goto out;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 free_buf:
Jesse Keating 3494df0
 	kfree(buf);
Jesse Keating 3494df0
+out:
Jesse Keating 3494df0
 	return ret;
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -586,7 +640,7 @@ static unsigned int port_fops_poll(struct file *filp, poll_table *wait)
Jesse Keating 3494df0
 	ret = 0;
Jesse Keating 3494df0
 	if (port->inbuf)
Jesse Keating 3494df0
 		ret |= POLLIN | POLLRDNORM;
Jesse Keating 3494df0
-	if (port->host_connected)
Jesse Keating 3494df0
+	if (!will_write_block(port))
Jesse Keating 3494df0
 		ret |= POLLOUT;
Jesse Keating 3494df0
 	if (!port->host_connected)
Jesse Keating 3494df0
 		ret |= POLLHUP;
Jesse Keating 3494df0
@@ -610,6 +664,10 @@ static int port_fops_release(struct inode *inode, struct file *filp)
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	spin_unlock_irq(&port->inbuf_lock);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	spin_lock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+	reclaim_consumed_buffers(port);
Jesse Keating 3494df0
+	spin_unlock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	return 0;
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -638,6 +696,15 @@ static int port_fops_open(struct inode *inode, struct file *filp)
Jesse Keating 3494df0
 	port->guest_connected = true;
Jesse Keating 3494df0
 	spin_unlock_irq(&port->inbuf_lock);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	spin_lock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * There might be a chance that we missed reclaiming a few
Jesse Keating 3494df0
+	 * buffers in the window of the port getting previously closed
Jesse Keating 3494df0
+	 * and opening now.
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
+	reclaim_consumed_buffers(port);
Jesse Keating 3494df0
+	spin_unlock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	/* Notify host of port being opened */
Jesse Keating 3494df0
 	send_control_msg(filp->private_data, VIRTIO_CONSOLE_PORT_OPEN, 1);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -676,9 +743,9 @@ static int put_chars(u32 vtermno, const char *buf, int count)
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	port = find_port_by_vtermno(vtermno);
Jesse Keating 3494df0
 	if (!port)
Jesse Keating 3494df0
-		return 0;
Jesse Keating 3494df0
+		return -EPIPE;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	return send_buf(port, (void *)buf, count);
Jesse Keating 3494df0
+	return send_buf(port, (void *)buf, count, false);
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 /*
Jesse Keating 3494df0
@@ -692,9 +759,13 @@ static int get_chars(u32 vtermno, char *buf, int count)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
 	struct port *port;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	/* If we've not set up the port yet, we have no input to give. */
Jesse Keating 3494df0
+	if (unlikely(early_put_chars))
Jesse Keating 3494df0
+		return 0;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	port = find_port_by_vtermno(vtermno);
Jesse Keating 3494df0
 	if (!port)
Jesse Keating 3494df0
-		return 0;
Jesse Keating 3494df0
+		return -EPIPE;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	/* If we don't have an input queue yet, we can't get input. */
Jesse Keating 3494df0
 	BUG_ON(!port->in_vq);
Jesse Keating 3494df0
@@ -705,22 +776,14 @@ static int get_chars(u32 vtermno, char *buf, int count)
Jesse Keating 3494df0
 static void resize_console(struct port *port)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
 	struct virtio_device *vdev;
Jesse Keating 3494df0
-	struct winsize ws;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	/* The port could have been hot-unplugged */
Jesse Keating 3494df0
-	if (!port)
Jesse Keating 3494df0
+	if (!port || !is_console_port(port))
Jesse Keating 3494df0
 		return;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	vdev = port->portdev->vdev;
Jesse Keating 3494df0
-	if (virtio_has_feature(vdev, VIRTIO_CONSOLE_F_SIZE)) {
Jesse Keating 3494df0
-		vdev->config->get(vdev,
Jesse Keating 3494df0
-				  offsetof(struct virtio_console_config, cols),
Jesse Keating 3494df0
-				  &ws.ws_col, sizeof(u16));
Jesse Keating 3494df0
-		vdev->config->get(vdev,
Jesse Keating 3494df0
-				  offsetof(struct virtio_console_config, rows),
Jesse Keating 3494df0
-				  &ws.ws_row, sizeof(u16));
Jesse Keating 3494df0
-		hvc_resize(port->cons.hvc, ws);
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
+	if (virtio_has_feature(vdev, VIRTIO_CONSOLE_F_SIZE))
Jesse Keating 3494df0
+		hvc_resize(port->cons.hvc, port->cons.ws);
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 /* We set the configuration at this point, since we now have a tty */
Jesse Keating 3494df0
@@ -804,6 +867,13 @@ int init_port_console(struct port *port)
Jesse Keating 3494df0
 	spin_unlock_irq(&pdrvdata_lock);
Jesse Keating 3494df0
 	port->guest_connected = true;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * Start using the new console output if this is the first
Jesse Keating 3494df0
+	 * console to come up.
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
+	if (early_put_chars)
Jesse Keating 3494df0
+		early_put_chars = NULL;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	/* Notify host of port being opened */
Jesse Keating 3494df0
 	send_control_msg(port, VIRTIO_CONSOLE_PORT_OPEN, 1);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -859,6 +929,8 @@ static ssize_t debugfs_read(struct file *filp, char __user *ubuf,
Jesse Keating 3494df0
 	out_offset += snprintf(buf + out_offset, out_count - out_offset,
Jesse Keating 3494df0
 			       "host_connected: %d\n", port->host_connected);
Jesse Keating 3494df0
 	out_offset += snprintf(buf + out_offset, out_count - out_offset,
Jesse Keating 3494df0
+			       "outvq_full: %d\n", port->outvq_full);
Jesse Keating 3494df0
+	out_offset += snprintf(buf + out_offset, out_count - out_offset,
Jesse Keating 3494df0
 			       "is_console: %s\n",
Jesse Keating 3494df0
 			       is_console_port(port) ? "yes" : "no");
Jesse Keating 3494df0
 	out_offset += snprintf(buf + out_offset, out_count - out_offset,
Jesse Keating 3494df0
@@ -875,6 +947,153 @@ static const struct file_operations port_debugfs_ops = {
Jesse Keating 3494df0
 	.read  = debugfs_read,
Jesse Keating 3494df0
 };
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+static void set_console_size(struct port *port, u16 rows, u16 cols)
Jesse Keating 3494df0
+{
Jesse Keating 3494df0
+	if (!port || !is_console_port(port))
Jesse Keating 3494df0
+		return;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->cons.ws.ws_row = rows;
Jesse Keating 3494df0
+	port->cons.ws.ws_col = cols;
Jesse Keating 3494df0
+}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+static unsigned int fill_queue(struct virtqueue *vq, spinlock_t *lock)
Jesse Keating 3494df0
+{
Jesse Keating 3494df0
+	struct port_buffer *buf;
Jesse Keating 3494df0
+	unsigned int nr_added_bufs;
Jesse Keating 3494df0
+	int ret;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	nr_added_bufs = 0;
Jesse Keating 3494df0
+	do {
Jesse Keating 3494df0
+		buf = alloc_buf(PAGE_SIZE);
Jesse Keating 3494df0
+		if (!buf)
Jesse Keating 3494df0
+			break;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+		spin_lock_irq(lock);
Jesse Keating 3494df0
+		ret = add_inbuf(vq, buf);
Jesse Keating 3494df0
+		if (ret < 0) {
Jesse Keating 3494df0
+			spin_unlock_irq(lock);
Jesse Keating 3494df0
+			free_buf(buf);
Jesse Keating 3494df0
+			break;
Jesse Keating 3494df0
+		}
Jesse Keating 3494df0
+		nr_added_bufs++;
Jesse Keating 3494df0
+		spin_unlock_irq(lock);
Jesse Keating 3494df0
+	} while (ret > 0);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	return nr_added_bufs;
Jesse Keating 3494df0
+}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+static int add_port(struct ports_device *portdev, u32 id)
Jesse Keating 3494df0
+{
Jesse Keating 3494df0
+	char debugfs_name[16];
Jesse Keating 3494df0
+	struct port *port;
Jesse Keating 3494df0
+	struct port_buffer *buf;
Jesse Keating 3494df0
+	dev_t devt;
Jesse Keating 3494df0
+	unsigned int nr_added_bufs;
Jesse Keating 3494df0
+	int err;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port = kmalloc(sizeof(*port), GFP_KERNEL);
Jesse Keating 3494df0
+	if (!port) {
Jesse Keating 3494df0
+		err = -ENOMEM;
Jesse Keating 3494df0
+		goto fail;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->portdev = portdev;
Jesse Keating 3494df0
+	port->id = id;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->name = NULL;
Jesse Keating 3494df0
+	port->inbuf = NULL;
Jesse Keating 3494df0
+	port->cons.hvc = NULL;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->cons.ws.ws_row = port->cons.ws.ws_col = 0;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->host_connected = port->guest_connected = false;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->outvq_full = false;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	port->in_vq = portdev->in_vqs[port->id];
Jesse Keating 3494df0
+	port->out_vq = portdev->out_vqs[port->id];
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	cdev_init(&port->cdev, &port_fops);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	devt = MKDEV(portdev->chr_major, id);
Jesse Keating 3494df0
+	err = cdev_add(&port->cdev, devt, 1);
Jesse Keating 3494df0
+	if (err < 0) {
Jesse Keating 3494df0
+		dev_err(&port->portdev->vdev->dev,
Jesse Keating 3494df0
+			"Error %d adding cdev for port %u\n", err, id);
Jesse Keating 3494df0
+		goto free_port;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+	port->dev = device_create(pdrvdata.class, &port->portdev->vdev->dev,
Jesse Keating 3494df0
+				  devt, port, "vport%up%u",
Jesse Keating 3494df0
+				  port->portdev->drv_index, id);
Jesse Keating 3494df0
+	if (IS_ERR(port->dev)) {
Jesse Keating 3494df0
+		err = PTR_ERR(port->dev);
Jesse Keating 3494df0
+		dev_err(&port->portdev->vdev->dev,
Jesse Keating 3494df0
+			"Error %d creating device for port %u\n",
Jesse Keating 3494df0
+			err, id);
Jesse Keating 3494df0
+		goto free_cdev;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	spin_lock_init(&port->inbuf_lock);
Jesse Keating 3494df0
+	spin_lock_init(&port->outvq_lock);
Jesse Keating 3494df0
+	init_waitqueue_head(&port->waitqueue);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	/* Fill the in_vq with buffers so the host can send us data. */
Jesse Keating 3494df0
+	nr_added_bufs = fill_queue(port->in_vq, &port->inbuf_lock);
Jesse Keating 3494df0
+	if (!nr_added_bufs) {
Jesse Keating 3494df0
+		dev_err(port->dev, "Error allocating inbufs\n");
Jesse Keating 3494df0
+		err = -ENOMEM;
Jesse Keating 3494df0
+		goto free_device;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * If we're not using multiport support, this has to be a console port
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
+	if (!use_multiport(port->portdev)) {
Jesse Keating 3494df0
+		err = init_port_console(port);
Jesse Keating 3494df0
+		if (err)
Jesse Keating 3494df0
+			goto free_inbufs;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	spin_lock_irq(&portdev->ports_lock);
Jesse Keating 3494df0
+	list_add_tail(&port->list, &port->portdev->ports);
Jesse Keating 3494df0
+	spin_unlock_irq(&portdev->ports_lock);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	/*
Jesse Keating 3494df0
+	 * Tell the Host we're set so that it can send us various
Jesse Keating 3494df0
+	 * configuration parameters for this port (eg, port name,
Jesse Keating 3494df0
+	 * caching, whether this is a console port, etc.)
Jesse Keating 3494df0
+	 */
Jesse Keating 3494df0
+	send_control_msg(port, VIRTIO_CONSOLE_PORT_READY, 1);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+	if (pdrvdata.debugfs_dir) {
Jesse Keating 3494df0
+		/*
Jesse Keating 3494df0
+		 * Finally, create the debugfs file that we can use to
Jesse Keating 3494df0
+		 * inspect a port's state at any time
Jesse Keating 3494df0
+		 */
Jesse Keating 3494df0
+		sprintf(debugfs_name, "vport%up%u",
Jesse Keating 3494df0
+			port->portdev->drv_index, id);
Jesse Keating 3494df0
+		port->debugfs_file = debugfs_create_file(debugfs_name, 0444,
Jesse Keating 3494df0
+							 pdrvdata.debugfs_dir,
Jesse Keating 3494df0
+							 port,
Jesse Keating 3494df0
+							 &port_debugfs_ops);
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
+	return 0;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+free_inbufs:
Jesse Keating 3494df0
+	while ((buf = virtqueue_detach_unused_buf(port->in_vq)))
Jesse Keating 3494df0
+		free_buf(buf);
Jesse Keating 3494df0
+free_device:
Jesse Keating 3494df0
+	device_destroy(pdrvdata.class, port->dev->devt);
Jesse Keating 3494df0
+free_cdev:
Jesse Keating 3494df0
+	cdev_del(&port->cdev);
Jesse Keating 3494df0
+free_port:
Jesse Keating 3494df0
+	kfree(port);
Jesse Keating 3494df0
+fail:
Jesse Keating 3494df0
+	/* The host might want to notify management sw about port add failure */
Jesse Keating 3494df0
+	__send_control_msg(portdev, id, VIRTIO_CONSOLE_PORT_READY, 0);
Jesse Keating 3494df0
+	return err;
Jesse Keating 3494df0
+}
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 /* Remove all port-specific data. */
Jesse Keating 3494df0
 static int remove_port(struct port *port)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
@@ -888,7 +1107,18 @@ static int remove_port(struct port *port)
Jesse Keating 3494df0
 		spin_lock_irq(&pdrvdata_lock);
Jesse Keating 3494df0
 		list_del(&port->cons.list);
Jesse Keating 3494df0
 		spin_unlock_irq(&pdrvdata_lock);
Jesse Keating 3494df0
+#if 0
Jesse Keating 3494df0
+		/*
Jesse Keating 3494df0
+		 * hvc_remove() not called as removing one hvc port
Jesse Keating 3494df0
+		 * results in other hvc ports getting frozen.
Jesse Keating 3494df0
+		 *
Jesse Keating 3494df0
+		 * Once this is resolved in hvc, this functionality
Jesse Keating 3494df0
+		 * will be enabled.  Till that is done, the -EPIPE
Jesse Keating 3494df0
+		 * return from get_chars() above will help
Jesse Keating 3494df0
+		 * hvc_console.c to clean up on ports we remove here.
Jesse Keating 3494df0
+		 */
Jesse Keating 3494df0
 		hvc_remove(port->cons.hvc);
Jesse Keating 3494df0
+#endif
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 	if (port->guest_connected)
Jesse Keating 3494df0
 		send_control_msg(port, VIRTIO_CONSOLE_PORT_OPEN, 0);
Jesse Keating 3494df0
@@ -900,6 +1130,8 @@ static int remove_port(struct port *port)
Jesse Keating 3494df0
 	/* Remove unused data this port might have received. */
Jesse Keating 3494df0
 	discard_port_data(port);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+	reclaim_consumed_buffers(port);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 	/* Remove buffers we queued up for the Host to send us data in. */
Jesse Keating 3494df0
 	while ((buf = virtqueue_detach_unused_buf(port->in_vq)))
Jesse Keating 3494df0
 		free_buf(buf);
Jesse Keating 3494df0
@@ -924,7 +1156,7 @@ static void handle_control_message(struct ports_device *portdev,
Jesse Keating 3494df0
 	cpkt = (struct virtio_console_control *)(buf->buf + buf->offset);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	port = find_port_by_id(portdev, cpkt->id);
Jesse Keating 3494df0
-	if (!port) {
Jesse Keating 3494df0
+	if (!port && cpkt->event != VIRTIO_CONSOLE_PORT_ADD) {
Jesse Keating 3494df0
 		/* No valid header at start of buffer.  Drop it. */
Jesse Keating 3494df0
 		dev_dbg(&portdev->vdev->dev,
Jesse Keating 3494df0
 			"Invalid index %u in control packet\n", cpkt->id);
Jesse Keating 3494df0
@@ -932,6 +1164,24 @@ static void handle_control_message(struct ports_device *portdev,
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	switch (cpkt->event) {
Jesse Keating 3494df0
+	case VIRTIO_CONSOLE_PORT_ADD:
Jesse Keating 3494df0
+		if (port) {
Jesse Keating 3494df0
+			dev_dbg(&portdev->vdev->dev,
Jesse Keating 3494df0
+				"Port %u already added\n", port->id);
Jesse Keating 3494df0
+			send_control_msg(port, VIRTIO_CONSOLE_PORT_READY, 1);
Jesse Keating 3494df0
+			break;
Jesse Keating 3494df0
+		}
Jesse Keating 3494df0
+		if (cpkt->id >= portdev->config.max_nr_ports) {
Jesse Keating 3494df0
+			dev_warn(&portdev->vdev->dev,
Jesse Keating 3494df0
+				"Request for adding port with out-of-bound id %u, max. supported id: %u\n",
Jesse Keating 3494df0
+				cpkt->id, portdev->config.max_nr_ports - 1);
Jesse Keating 3494df0
+			break;
Jesse Keating 3494df0
+		}
Jesse Keating 3494df0
+		add_port(portdev, cpkt->id);
Jesse Keating 3494df0
+		break;
Jesse Keating 3494df0
+	case VIRTIO_CONSOLE_PORT_REMOVE:
Jesse Keating 3494df0
+		remove_port(port);
Jesse Keating 3494df0
+		break;
Jesse Keating 3494df0
 	case VIRTIO_CONSOLE_CONSOLE_PORT:
Jesse Keating 3494df0
 		if (!cpkt->value)
Jesse Keating 3494df0
 			break;
Jesse Keating 3494df0
@@ -944,15 +1194,34 @@ static void handle_control_message(struct ports_device *portdev,
Jesse Keating 3494df0
 		 * have to notify the host first.
Jesse Keating 3494df0
 		 */
Jesse Keating 3494df0
 		break;
Jesse Keating 3494df0
-	case VIRTIO_CONSOLE_RESIZE:
Jesse Keating 3494df0
+	case VIRTIO_CONSOLE_RESIZE: {
Jesse Keating 3494df0
+		struct {
Jesse Keating 3494df0
+			__u16 rows;
Jesse Keating 3494df0
+			__u16 cols;
Jesse Keating 3494df0
+		} size;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 		if (!is_console_port(port))
Jesse Keating 3494df0
 			break;
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+		memcpy(&size, buf->buf + buf->offset + sizeof(*cpkt),
Jesse Keating 3494df0
+		       sizeof(size));
Jesse Keating 3494df0
+		set_console_size(port, size.rows, size.cols);
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 		port->cons.hvc->irq_requested = 1;
Jesse Keating 3494df0
 		resize_console(port);
Jesse Keating 3494df0
 		break;
Jesse Keating 3494df0
+	}
Jesse Keating 3494df0
 	case VIRTIO_CONSOLE_PORT_OPEN:
Jesse Keating 3494df0
 		port->host_connected = cpkt->value;
Jesse Keating 3494df0
 		wake_up_interruptible(&port->waitqueue);
Jesse Keating 3494df0
+		/*
Jesse Keating 3494df0
+		 * If the host port got closed and the host had any
Jesse Keating 3494df0
+		 * unconsumed buffers, we'll be able to reclaim them
Jesse Keating 3494df0
+		 * now.
Jesse Keating 3494df0
+		 */
Jesse Keating 3494df0
+		spin_lock_irq(&port->outvq_lock);
Jesse Keating 3494df0
+		reclaim_consumed_buffers(port);
Jesse Keating 3494df0
+		spin_unlock_irq(&port->outvq_lock);
Jesse Keating 3494df0
 		break;
Jesse Keating 3494df0
 	case VIRTIO_CONSOLE_PORT_NAME:
Jesse Keating 3494df0
 		/*
Jesse Keating 3494df0
@@ -990,32 +1259,6 @@ static void handle_control_message(struct ports_device *portdev,
Jesse Keating 3494df0
 			kobject_uevent(&port->dev->kobj, KOBJ_CHANGE);
Jesse Keating 3494df0
 		}
Jesse Keating 3494df0
 		break;
Jesse Keating 3494df0
-	case VIRTIO_CONSOLE_PORT_REMOVE:
Jesse Keating 3494df0
-		/*
Jesse Keating 3494df0
-		 * Hot unplug the port.  We don't decrement nr_ports
Jesse Keating 3494df0
-		 * since we don't want to deal with extra complexities
Jesse Keating 3494df0
-		 * of using the lowest-available port id: We can just
Jesse Keating 3494df0
-		 * pick up the nr_ports number as the id and not have
Jesse Keating 3494df0
-		 * userspace send it to us.  This helps us in two
Jesse Keating 3494df0
-		 * ways:
Jesse Keating 3494df0
-		 *
Jesse Keating 3494df0
-		 * - We don't need to have a 'port_id' field in the
Jesse Keating 3494df0
-		 *   config space when a port is hot-added.  This is a
Jesse Keating 3494df0
-		 *   good thing as we might queue up multiple hotplug
Jesse Keating 3494df0
-		 *   requests issued in our workqueue.
Jesse Keating 3494df0
-		 *
Jesse Keating 3494df0
-		 * - Another way to deal with this would have been to
Jesse Keating 3494df0
-		 *   use a bitmap of the active ports and select the
Jesse Keating 3494df0
-		 *   lowest non-active port from that map.  That
Jesse Keating 3494df0
-		 *   bloats the already tight config space and we
Jesse Keating 3494df0
-		 *   would end up artificially limiting the
Jesse Keating 3494df0
-		 *   max. number of ports to sizeof(bitmap).  Right
Jesse Keating 3494df0
-		 *   now we can support 2^32 ports (as the port id is
Jesse Keating 3494df0
-		 *   stored in a u32 type).
Jesse Keating 3494df0
-		 *
Jesse Keating 3494df0
-		 */
Jesse Keating 3494df0
-		remove_port(port);
Jesse Keating 3494df0
-		break;
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -1092,204 +1335,29 @@ static void config_intr(struct virtio_device *vdev)
Jesse Keating 3494df0
 	struct ports_device *portdev;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	portdev = vdev->priv;
Jesse Keating 3494df0
-	if (use_multiport(portdev)) {
Jesse Keating 3494df0
-		/* Handle port hot-add */
Jesse Keating 3494df0
-		schedule_work(&portdev->config_work);
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-	/*
Jesse Keating 3494df0
-	 * We'll use this way of resizing only for legacy support.
Jesse Keating 3494df0
-	 * For newer userspace (VIRTIO_CONSOLE_F_MULTPORT+), use
Jesse Keating 3494df0
-	 * control messages to indicate console size changes so that
Jesse Keating 3494df0
-	 * it can be done per-port
Jesse Keating 3494df0
-	 */
Jesse Keating 3494df0
-	resize_console(find_port_by_id(portdev, 0));
Jesse Keating 3494df0
-}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-static unsigned int fill_queue(struct virtqueue *vq, spinlock_t *lock)
Jesse Keating 3494df0
-{
Jesse Keating 3494df0
-	struct port_buffer *buf;
Jesse Keating 3494df0
-	unsigned int nr_added_bufs;
Jesse Keating 3494df0
-	int ret;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	nr_added_bufs = 0;
Jesse Keating 3494df0
-	do {
Jesse Keating 3494df0
-		buf = alloc_buf(PAGE_SIZE);
Jesse Keating 3494df0
-		if (!buf)
Jesse Keating 3494df0
-			break;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-		spin_lock_irq(lock);
Jesse Keating 3494df0
-		ret = add_inbuf(vq, buf);
Jesse Keating 3494df0
-		if (ret < 0) {
Jesse Keating 3494df0
-			spin_unlock_irq(lock);
Jesse Keating 3494df0
-			free_buf(buf);
Jesse Keating 3494df0
-			break;
Jesse Keating 3494df0
-		}
Jesse Keating 3494df0
-		nr_added_bufs++;
Jesse Keating 3494df0
-		spin_unlock_irq(lock);
Jesse Keating 3494df0
-	} while (ret > 0);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	return nr_added_bufs;
Jesse Keating 3494df0
-}
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-static int add_port(struct ports_device *portdev, u32 id)
Jesse Keating 3494df0
-{
Jesse Keating 3494df0
-	char debugfs_name[16];
Jesse Keating 3494df0
-	struct port *port;
Jesse Keating 3494df0
-	struct port_buffer *buf;
Jesse Keating 3494df0
-	dev_t devt;
Jesse Keating 3494df0
-	unsigned int nr_added_bufs;
Jesse Keating 3494df0
-	int err;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	port = kmalloc(sizeof(*port), GFP_KERNEL);
Jesse Keating 3494df0
-	if (!port) {
Jesse Keating 3494df0
-		err = -ENOMEM;
Jesse Keating 3494df0
-		goto fail;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	port->portdev = portdev;
Jesse Keating 3494df0
-	port->id = id;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	port->name = NULL;
Jesse Keating 3494df0
-	port->inbuf = NULL;
Jesse Keating 3494df0
-	port->cons.hvc = NULL;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	port->host_connected = port->guest_connected = false;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	port->in_vq = portdev->in_vqs[port->id];
Jesse Keating 3494df0
-	port->out_vq = portdev->out_vqs[port->id];
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	cdev_init(&port->cdev, &port_fops);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	devt = MKDEV(portdev->chr_major, id);
Jesse Keating 3494df0
-	err = cdev_add(&port->cdev, devt, 1);
Jesse Keating 3494df0
-	if (err < 0) {
Jesse Keating 3494df0
-		dev_err(&port->portdev->vdev->dev,
Jesse Keating 3494df0
-			"Error %d adding cdev for port %u\n", err, id);
Jesse Keating 3494df0
-		goto free_port;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-	port->dev = device_create(pdrvdata.class, &port->portdev->vdev->dev,
Jesse Keating 3494df0
-				  devt, port, "vport%up%u",
Jesse Keating 3494df0
-				  port->portdev->drv_index, id);
Jesse Keating 3494df0
-	if (IS_ERR(port->dev)) {
Jesse Keating 3494df0
-		err = PTR_ERR(port->dev);
Jesse Keating 3494df0
-		dev_err(&port->portdev->vdev->dev,
Jesse Keating 3494df0
-			"Error %d creating device for port %u\n",
Jesse Keating 3494df0
-			err, id);
Jesse Keating 3494df0
-		goto free_cdev;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	spin_lock_init(&port->inbuf_lock);
Jesse Keating 3494df0
-	init_waitqueue_head(&port->waitqueue);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	/* Fill the in_vq with buffers so the host can send us data. */
Jesse Keating 3494df0
-	nr_added_bufs = fill_queue(port->in_vq, &port->inbuf_lock);
Jesse Keating 3494df0
-	if (!nr_added_bufs) {
Jesse Keating 3494df0
-		dev_err(port->dev, "Error allocating inbufs\n");
Jesse Keating 3494df0
-		err = -ENOMEM;
Jesse Keating 3494df0
-		goto free_device;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	/*
Jesse Keating 3494df0
-	 * If we're not using multiport support, this has to be a console port
Jesse Keating 3494df0
-	 */
Jesse Keating 3494df0
-	if (!use_multiport(port->portdev)) {
Jesse Keating 3494df0
-		err = init_port_console(port);
Jesse Keating 3494df0
-		if (err)
Jesse Keating 3494df0
-			goto free_inbufs;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	spin_lock_irq(&portdev->ports_lock);
Jesse Keating 3494df0
-	list_add_tail(&port->list, &port->portdev->ports);
Jesse Keating 3494df0
-	spin_unlock_irq(&portdev->ports_lock);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	/*
Jesse Keating 3494df0
-	 * Tell the Host we're set so that it can send us various
Jesse Keating 3494df0
-	 * configuration parameters for this port (eg, port name,
Jesse Keating 3494df0
-	 * caching, whether this is a console port, etc.)
Jesse Keating 3494df0
-	 */
Jesse Keating 3494df0
-	send_control_msg(port, VIRTIO_CONSOLE_PORT_READY, 1);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	if (pdrvdata.debugfs_dir) {
Jesse Keating 3494df0
-		/*
Jesse Keating 3494df0
-		 * Finally, create the debugfs file that we can use to
Jesse Keating 3494df0
-		 * inspect a port's state at any time
Jesse Keating 3494df0
-		 */
Jesse Keating 3494df0
-		sprintf(debugfs_name, "vport%up%u",
Jesse Keating 3494df0
-			port->portdev->drv_index, id);
Jesse Keating 3494df0
-		port->debugfs_file = debugfs_create_file(debugfs_name, 0444,
Jesse Keating 3494df0
-							 pdrvdata.debugfs_dir,
Jesse Keating 3494df0
-							 port,
Jesse Keating 3494df0
-							 &port_debugfs_ops);
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-	return 0;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-free_inbufs:
Jesse Keating 3494df0
-	while ((buf = virtqueue_detach_unused_buf(port->in_vq)))
Jesse Keating 3494df0
-		free_buf(buf);
Jesse Keating 3494df0
-free_device:
Jesse Keating 3494df0
-	device_destroy(pdrvdata.class, port->dev->devt);
Jesse Keating 3494df0
-free_cdev:
Jesse Keating 3494df0
-	cdev_del(&port->cdev);
Jesse Keating 3494df0
-free_port:
Jesse Keating 3494df0
-	kfree(port);
Jesse Keating 3494df0
-fail:
Jesse Keating 3494df0
-	return err;
Jesse Keating 3494df0
-}
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-/*
Jesse Keating 3494df0
- * The workhandler for config-space updates.
Jesse Keating 3494df0
- *
Jesse Keating 3494df0
- * This is called when ports are hot-added.
Jesse Keating 3494df0
- */
Jesse Keating 3494df0
-static void config_work_handler(struct work_struct *work)
Jesse Keating 3494df0
-{
Jesse Keating 3494df0
-	struct virtio_console_multiport_conf virtconconf;
Jesse Keating 3494df0
-	struct ports_device *portdev;
Jesse Keating 3494df0
-	struct virtio_device *vdev;
Jesse Keating 3494df0
-	int err;
Jesse Keating 3494df0
+	if (!use_multiport(portdev)) {
Jesse Keating 3494df0
+		struct port *port;
Jesse Keating 3494df0
+		u16 rows, cols;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	portdev = container_of(work, struct ports_device, config_work);
Jesse Keating 3494df0
+		vdev->config->get(vdev,
Jesse Keating 3494df0
+				  offsetof(struct virtio_console_config, cols),
Jesse Keating 3494df0
+				  &cols, sizeof(u16));
Jesse Keating 3494df0
+		vdev->config->get(vdev,
Jesse Keating 3494df0
+				  offsetof(struct virtio_console_config, rows),
Jesse Keating 3494df0
+				  &rows, sizeof(u16));
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	vdev = portdev->vdev;
Jesse Keating 3494df0
-	vdev->config->get(vdev,
Jesse Keating 3494df0
-			  offsetof(struct virtio_console_multiport_conf,
Jesse Keating 3494df0
-				   nr_ports),
Jesse Keating 3494df0
-			  &virtconconf.nr_ports,
Jesse Keating 3494df0
-			  sizeof(virtconconf.nr_ports));
Jesse Keating 3494df0
+		port = find_port_by_id(portdev, 0);
Jesse Keating 3494df0
+		set_console_size(port, rows, cols);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	if (portdev->config.nr_ports == virtconconf.nr_ports) {
Jesse Keating 3494df0
 		/*
Jesse Keating 3494df0
-		 * Port 0 got hot-added.  Since we already did all the
Jesse Keating 3494df0
-		 * other initialisation for it, just tell the Host
Jesse Keating 3494df0
-		 * that the port is ready if we find the port.  In
Jesse Keating 3494df0
-		 * case the port was hot-removed earlier, we call
Jesse Keating 3494df0
-		 * add_port to add the port.
Jesse Keating 3494df0
+		 * We'll use this way of resizing only for legacy
Jesse Keating 3494df0
+		 * support.  For newer userspace
Jesse Keating 3494df0
+		 * (VIRTIO_CONSOLE_F_MULTPORT+), use control messages
Jesse Keating 3494df0
+		 * to indicate console size changes so that it can be
Jesse Keating 3494df0
+		 * done per-port.
Jesse Keating 3494df0
 		 */
Jesse Keating 3494df0
-		struct port *port;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-		port = find_port_by_id(portdev, 0);
Jesse Keating 3494df0
-		if (!port)
Jesse Keating 3494df0
-			add_port(portdev, 0);
Jesse Keating 3494df0
-		else
Jesse Keating 3494df0
-			send_control_msg(port, VIRTIO_CONSOLE_PORT_READY, 1);
Jesse Keating 3494df0
-		return;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-	if (virtconconf.nr_ports > portdev->config.max_nr_ports) {
Jesse Keating 3494df0
-		dev_warn(&vdev->dev,
Jesse Keating 3494df0
-			 "More ports specified (%u) than allowed (%u)",
Jesse Keating 3494df0
-			 portdev->config.nr_ports + 1,
Jesse Keating 3494df0
-			 portdev->config.max_nr_ports);
Jesse Keating 3494df0
-		return;
Jesse Keating 3494df0
-	}
Jesse Keating 3494df0
-	if (virtconconf.nr_ports < portdev->config.nr_ports)
Jesse Keating 3494df0
-		return;
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	/* Hot-add ports */
Jesse Keating 3494df0
-	while (virtconconf.nr_ports - portdev->config.nr_ports) {
Jesse Keating 3494df0
-		err = add_port(portdev, portdev->config.nr_ports);
Jesse Keating 3494df0
-		if (err)
Jesse Keating 3494df0
-			break;
Jesse Keating 3494df0
-		portdev->config.nr_ports++;
Jesse Keating 3494df0
+		resize_console(port);
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 }
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -1414,7 +1482,6 @@ static const struct file_operations portdev_fops = {
Jesse Keating 3494df0
 static int __devinit virtcons_probe(struct virtio_device *vdev)
Jesse Keating 3494df0
 {
Jesse Keating 3494df0
 	struct ports_device *portdev;
Jesse Keating 3494df0
-	u32 i;
Jesse Keating 3494df0
 	int err;
Jesse Keating 3494df0
 	bool multiport;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
@@ -1443,37 +1510,19 @@ static int __devinit virtcons_probe(struct virtio_device *vdev)
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	multiport = false;
Jesse Keating 3494df0
-	portdev->config.nr_ports = 1;
Jesse Keating 3494df0
 	portdev->config.max_nr_ports = 1;
Jesse Keating 3494df0
-#if 0 /* Multiport is not quite ready yet --RR */
Jesse Keating 3494df0
 	if (virtio_has_feature(vdev, VIRTIO_CONSOLE_F_MULTIPORT)) {
Jesse Keating 3494df0
 		multiport = true;
Jesse Keating 3494df0
 		vdev->features[0] |= 1 << VIRTIO_CONSOLE_F_MULTIPORT;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-		vdev->config->get(vdev,
Jesse Keating 3494df0
-				  offsetof(struct virtio_console_multiport_conf,
Jesse Keating 3494df0
-					   nr_ports),
Jesse Keating 3494df0
-				  &portdev->config.nr_ports,
Jesse Keating 3494df0
-				  sizeof(portdev->config.nr_ports));
Jesse Keating 3494df0
-		vdev->config->get(vdev,
Jesse Keating 3494df0
-				  offsetof(struct virtio_console_multiport_conf,
Jesse Keating 3494df0
-					   max_nr_ports),
Jesse Keating 3494df0
+		vdev->config->get(vdev, offsetof(struct virtio_console_config,
Jesse Keating 3494df0
+						 max_nr_ports),
Jesse Keating 3494df0
 				  &portdev->config.max_nr_ports,
Jesse Keating 3494df0
 				  sizeof(portdev->config.max_nr_ports));
Jesse Keating 3494df0
-		if (portdev->config.nr_ports > portdev->config.max_nr_ports) {
Jesse Keating 3494df0
-			dev_warn(&vdev->dev,
Jesse Keating 3494df0
-				 "More ports (%u) specified than allowed (%u). Will init %u ports.",
Jesse Keating 3494df0
-				 portdev->config.nr_ports,
Jesse Keating 3494df0
-				 portdev->config.max_nr_ports,
Jesse Keating 3494df0
-				 portdev->config.max_nr_ports);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-			portdev->config.nr_ports = portdev->config.max_nr_ports;
Jesse Keating 3494df0
-		}
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	/* Let the Host know we support multiple ports.*/
Jesse Keating 3494df0
 	vdev->config->finalize_features(vdev);
Jesse Keating 3494df0
-#endif
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	err = init_vqs(portdev);
Jesse Keating 3494df0
 	if (err < 0) {
Jesse Keating 3494df0
@@ -1489,7 +1538,6 @@ static int __devinit virtcons_probe(struct virtio_device *vdev)
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 		spin_lock_init(&portdev->cvq_lock);
Jesse Keating 3494df0
 		INIT_WORK(&portdev->control_work, &control_work_handler);
Jesse Keating 3494df0
-		INIT_WORK(&portdev->config_work, &config_work_handler);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 		nr_added_bufs = fill_queue(portdev->c_ivq, &portdev->cvq_lock);
Jesse Keating 3494df0
 		if (!nr_added_bufs) {
Jesse Keating 3494df0
@@ -1498,16 +1546,22 @@ static int __devinit virtcons_probe(struct virtio_device *vdev)
Jesse Keating 3494df0
 			err = -ENOMEM;
Jesse Keating 3494df0
 			goto free_vqs;
Jesse Keating 3494df0
 		}
Jesse Keating 3494df0
+	} else {
Jesse Keating 3494df0
+		/*
Jesse Keating 3494df0
+		 * For backward compatibility: Create a console port
Jesse Keating 3494df0
+		 * if we're running on older host.
Jesse Keating 3494df0
+		 */
Jesse Keating 3494df0
+		add_port(portdev, 0);
Jesse Keating 3494df0
 	}
Jesse Keating 3494df0
 
Jesse Keating 3494df0
-	for (i = 0; i < portdev->config.nr_ports; i++)
Jesse Keating 3494df0
-		add_port(portdev, i);
Jesse Keating 3494df0
-
Jesse Keating 3494df0
-	/* Start using the new console output. */
Jesse Keating 3494df0
-	early_put_chars = NULL;
Jesse Keating 3494df0
+	__send_control_msg(portdev, VIRTIO_CONSOLE_BAD_ID,
Jesse Keating 3494df0
+			   VIRTIO_CONSOLE_DEVICE_READY, 1);
Jesse Keating 3494df0
 	return 0;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 free_vqs:
Jesse Keating 3494df0
+	/* The host might want to notify mgmt sw about device add failure */
Jesse Keating 3494df0
+	__send_control_msg(portdev, VIRTIO_CONSOLE_BAD_ID,
Jesse Keating 3494df0
+			   VIRTIO_CONSOLE_DEVICE_READY, 0);
Jesse Keating 3494df0
 	vdev->config->del_vqs(vdev);
Jesse Keating 3494df0
 	kfree(portdev->in_vqs);
Jesse Keating 3494df0
 	kfree(portdev->out_vqs);
Jesse Keating 3494df0
@@ -1529,7 +1583,6 @@ static void virtcons_remove(struct virtio_device *vdev)
Jesse Keating 3494df0
 	portdev = vdev->priv;
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	cancel_work_sync(&portdev->control_work);
Jesse Keating 3494df0
-	cancel_work_sync(&portdev->config_work);
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 	list_for_each_entry_safe(port, port2, &portdev->ports, list)
Jesse Keating 3494df0
 		remove_port(port);
Jesse Keating 3494df0
@@ -1556,6 +1609,7 @@ static struct virtio_device_id id_table[] = {
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 static unsigned int features[] = {
Jesse Keating 3494df0
 	VIRTIO_CONSOLE_F_SIZE,
Jesse Keating 3494df0
+	VIRTIO_CONSOLE_F_MULTIPORT,
Jesse Keating 3494df0
 };
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 static struct virtio_driver virtio_console = {
Jesse Keating 3494df0
diff --git a/include/linux/virtio_console.h b/include/linux/virtio_console.h
Jesse Keating 3494df0
index 92228a8..a85064d 100644
Jesse Keating 3494df0
--- a/include/linux/virtio_console.h
Jesse Keating 3494df0
+++ b/include/linux/virtio_console.h
Jesse Keating 3494df0
@@ -12,14 +12,39 @@
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 /* Feature bits */
Jesse Keating 3494df0
 #define VIRTIO_CONSOLE_F_SIZE	0	/* Does host provide console size? */
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_F_MULTIPORT 1	/* Does host provide multiple ports? */
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_BAD_ID		(~(u32)0)
Jesse Keating 3494df0
 
Jesse Keating 3494df0
 struct virtio_console_config {
Jesse Keating 3494df0
 	/* colums of the screens */
Jesse Keating 3494df0
 	__u16 cols;
Jesse Keating 3494df0
 	/* rows of the screens */
Jesse Keating 3494df0
 	__u16 rows;
Jesse Keating 3494df0
+	/* max. number of ports this device can hold */
Jesse Keating 3494df0
+	__u32 max_nr_ports;
Jesse Keating 3494df0
 } __attribute__((packed));
Jesse Keating 3494df0
 
Jesse Keating 3494df0
+/*
Jesse Keating 3494df0
+ * A message that's passed between the Host and the Guest for a
Jesse Keating 3494df0
+ * particular port.
Jesse Keating 3494df0
+ */
Jesse Keating 3494df0
+struct virtio_console_control {
Jesse Keating 3494df0
+	__u32 id;		/* Port number */
Jesse Keating 3494df0
+	__u16 event;		/* The kind of control event (see below) */
Jesse Keating 3494df0
+	__u16 value;		/* Extra information for the key */
Jesse Keating 3494df0
+};
Jesse Keating 3494df0
+
Jesse Keating 3494df0
+/* Some events for control messages */
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_DEVICE_READY	0
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_PORT_ADD		1
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_PORT_REMOVE	2
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_PORT_READY	3
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_CONSOLE_PORT	4
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_RESIZE		5
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_PORT_OPEN	6
Jesse Keating 3494df0
+#define VIRTIO_CONSOLE_PORT_NAME	7
Jesse Keating 3494df0
+
Jesse Keating 3494df0
 #ifdef __KERNEL__
Jesse Keating 3494df0
 int __init virtio_cons_early_init(int (*put_chars)(u32, const char *, int));
Jesse Keating 3494df0
 #endif /* __KERNEL__ */