0aaee59
From 43cf1b6a4ee31e69581042a0c85d1398f83dcedc Mon Sep 17 00:00:00 2001
0aaee59
From: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
Date: Fri, 20 Jan 2012 17:27:20 +0100
0aaee59
Cc: <linux-scsi@vger.kernel.org>
0aaee59
Cc: Rusty Russell <rusty@rustcorp.com.au>
0aaee59
Cc: kvm@vger.kernel.org
0aaee59
Cc: Pekka Enberg <penberg@kernel.org>
0aaee59
Cc: Michael S. Tsirkin <mst@redhat.com>
0aaee59
Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>, Mike Christie <michaelc@cs.wisc.edu>
0aaee59
Subject: [PATCH v5 0/3] virtio-scsi driver
0aaee59
0aaee59
This is the first implementation of the virtio-scsi driver, a virtual
0aaee59
HBA that will be supported by KVM.  It implements a subset of the spec,
0aaee59
in particular it does not implement asynchronous notifications for either
0aaee59
LUN reset/removal/addition or CD-ROM media events, but it is already
0aaee59
functional and usable.
0aaee59
0aaee59
Other matching bits:
0aaee59
0aaee59
- spec at http://people.redhat.com/pbonzini/virtio-spec.pdf
0aaee59
0aaee59
- QEMU implementation at git://github.com/bonzini/qemu.git,
0aaee59
  branch virtio-scsi
0aaee59
0aaee59
Please review.  Getting this in 3.3 is starting to look like wishful thinking,
0aaee59
but the possibility of regressions is obviously zero so I'm still dreaming.
0aaee59
Otherwise, that would be 3.4.
0aaee59
0aaee59
Paolo Bonzini (3):
0aaee59
  virtio-scsi: first version
0aaee59
  virtio-scsi: add error handling
0aaee59
  virtio-scsi: add power management support
0aaee59
0aaee59
v4->v5: change virtio id from 7 to 8
0aaee59
0aaee59
v3->v4: renamed VIRTIO_SCSI_S_UNDERRUN to VIRTIO_SCSI_S_OVERRUN;
0aaee59
    fixed 32-bit compilation; added power management support;
0aaee59
    adjusted calls to virtqueue_add_buf
0aaee59
0aaee59
 drivers/scsi/Kconfig        |    8 +
0aaee59
 drivers/scsi/Makefile       |    1 +
0aaee59
 drivers/scsi/virtio_scsi.c  |  594 +++++++++++++++++++++++++++++++++++++++++++
0aaee59
 include/linux/virtio_ids.h  |    1 +
0aaee59
 include/linux/virtio_scsi.h |  114 +++++++++
0aaee59
 5 files changed, 718 insertions(+), 0 deletions(-)
0aaee59
 create mode 100644 drivers/scsi/virtio_scsi.c
0aaee59
 create mode 100644 include/linux/virtio_scsi.h
0aaee59
0aaee59
From 84ad93b7215e18ab1755a625ede0fb00175e79bb Mon Sep 17 00:00:00 2001
0aaee59
From: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
Date: Tue, 29 Nov 2011 16:31:09 +0100
0aaee59
Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>, Mike Christie <michaelc@cs.wisc.edu>, Pekka Enberg <penberg@kernel.org>
0aaee59
Subject: [PATCH v5 1/3] virtio-scsi: first version
0aaee59
0aaee59
The virtio-scsi HBA is the basis of an alternative storage stack
0aaee59
for QEMU-based virtual machines (including KVM).  Compared to
0aaee59
virtio-blk it is more scalable, because it supports many LUNs
0aaee59
on a single PCI slot), more powerful (it more easily supports
0aaee59
passthrough of host devices to the guest) and more easily
0aaee59
extensible (new SCSI features implemented by QEMU should not
0aaee59
require updating the driver in the guest).
0aaee59
0aaee59
Cc: linux-scsi <linux-scsi@vger.kernel.org>
0aaee59
Cc: Rusty Russell <rusty@rustcorp.com.au>
0aaee59
Cc: Michael S. Tsirkin <mst@redhat.com>
0aaee59
Cc: kvm@vger.kernel.org
0aaee59
Acked-by: Pekka Enberg <penberg@kernel.org> 
0aaee59
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
---
0aaee59
	v4->v5: change virtio id from 7 to 8
0aaee59
0aaee59
	v3->v4: renamed VIRTIO_SCSI_S_UNDERRUN to VIRTIO_SCSI_S_OVERRUN;
0aaee59
	fixed 32-bit compilation; adjust call to virtqueue_add_buf
0aaee59
0aaee59
	v2->v3: added mempool, formatting fixes
0aaee59
0aaee59
	v1->v2: use dbg_dev, sdev_printk, scmd_printk
0aaee59
	   - renamed lock to vq_lock
0aaee59
	   - renamed cmd_vq to req_vq (and other similar changes)
0aaee59
	   - fixed missing break in VIRTIO_SCSI_S_OVERRUN
0aaee59
	   - added VIRTIO_SCSI_S_BUSY
0aaee59
	   - removed unused argument from virtscsi_map_cmd
0aaee59
	   - fixed two tabs that had slipped in
0aaee59
	   - moved max_sectors and cmd_per_lun from template to config space
0aaee59
	   - __attribute__((packed)) -> __packed
0aaee59
0aaee59
 drivers/scsi/Kconfig        |    8 +
0aaee59
 drivers/scsi/Makefile       |    1 +
0aaee59
 drivers/scsi/virtio_scsi.c  |  503 +++++++++++++++++++++++++++++++++++++++++++
0aaee59
 include/linux/virtio_ids.h  |    1 +
0aaee59
 include/linux/virtio_scsi.h |  114 ++++++++++
0aaee59
 5 files changed, 627 insertions(+), 0 deletions(-)
0aaee59
 create mode 100644 drivers/scsi/virtio_scsi.c
0aaee59
 create mode 100644 include/linux/virtio_scsi.h
0aaee59
0aaee59
diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig
0aaee59
index 16570aa..827ebaf 100644
0aaee59
--- a/drivers/scsi/Kconfig
0aaee59
+++ b/drivers/scsi/Kconfig
0aaee59
@@ -1897,6 +1897,14 @@ config SCSI_BFA_FC
0aaee59
 	  To compile this driver as a module, choose M here. The module will
0aaee59
 	  be called bfa.
0aaee59
 
0aaee59
+config SCSI_VIRTIO
0aaee59
+	tristate "virtio-scsi support (EXPERIMENTAL)"
0aaee59
+	depends on EXPERIMENTAL && VIRTIO
0aaee59
+	help
0aaee59
+          This is the virtual HBA driver for virtio.  If the kernel will
0aaee59
+          be used in a virtual machine, say Y or M.
0aaee59
+
0aaee59
+
0aaee59
 endif # SCSI_LOWLEVEL
0aaee59
 
0aaee59
 source "drivers/scsi/pcmcia/Kconfig"
0aaee59
diff --git a/drivers/scsi/Makefile b/drivers/scsi/Makefile
0aaee59
index 2b88749..351b28b 100644
0aaee59
--- a/drivers/scsi/Makefile
0aaee59
+++ b/drivers/scsi/Makefile
0aaee59
@@ -141,6 +141,7 @@ obj-$(CONFIG_SCSI_CXGB4_ISCSI)	+= libiscsi.o libiscsi_tcp.o cxgbi/
0aaee59
 obj-$(CONFIG_SCSI_BNX2_ISCSI)	+= libiscsi.o bnx2i/
0aaee59
 obj-$(CONFIG_BE2ISCSI)		+= libiscsi.o be2iscsi/
0aaee59
 obj-$(CONFIG_SCSI_PMCRAID)	+= pmcraid.o
0aaee59
+obj-$(CONFIG_SCSI_VIRTIO)	+= virtio_scsi.o
0aaee59
 obj-$(CONFIG_VMWARE_PVSCSI)	+= vmw_pvscsi.o
0aaee59
 
0aaee59
 obj-$(CONFIG_ARM)		+= arm/
0aaee59
diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c
0aaee59
new file mode 100644
0aaee59
index 0000000..3f87ae0
0aaee59
--- /dev/null
0aaee59
+++ b/drivers/scsi/virtio_scsi.c
0aaee59
@@ -0,0 +1,503 @@
0aaee59
+/*
0aaee59
+ * Virtio SCSI HBA driver
0aaee59
+ *
0aaee59
+ * Copyright IBM Corp. 2010
0aaee59
+ * Copyright Red Hat, Inc. 2011
0aaee59
+ *
0aaee59
+ * Authors:
0aaee59
+ *  Stefan Hajnoczi   <stefanha@linux.vnet.ibm.com>
0aaee59
+ *  Paolo Bonzini   <pbonzini@redhat.com>
0aaee59
+ *
0aaee59
+ * This work is licensed under the terms of the GNU GPL, version 2 or later.
0aaee59
+ * See the COPYING file in the top-level directory.
0aaee59
+ *
0aaee59
+ */
0aaee59
+
0aaee59
+#include <linux/module.h>
0aaee59
+#include <linux/slab.h>
0aaee59
+#include <linux/mempool.h>
0aaee59
+#include <linux/virtio.h>
0aaee59
+#include <linux/virtio_ids.h>
0aaee59
+#include <linux/virtio_config.h>
0aaee59
+#include <linux/virtio_scsi.h>
0aaee59
+#include <scsi/scsi_host.h>
0aaee59
+#include <scsi/scsi_device.h>
0aaee59
+#include <scsi/scsi_cmnd.h>
0aaee59
+
0aaee59
+#define VIRTIO_SCSI_MEMPOOL_SZ 64
0aaee59
+
0aaee59
+/* Command queue element */
0aaee59
+struct virtio_scsi_cmd {
0aaee59
+	struct scsi_cmnd *sc;
0aaee59
+	union {
0aaee59
+		struct virtio_scsi_cmd_req       cmd;
0aaee59
+		struct virtio_scsi_ctrl_tmf_req  tmf;
0aaee59
+		struct virtio_scsi_ctrl_an_req   an;
0aaee59
+	} req;
0aaee59
+	union {
0aaee59
+		struct virtio_scsi_cmd_resp      cmd;
0aaee59
+		struct virtio_scsi_ctrl_tmf_resp tmf;
0aaee59
+		struct virtio_scsi_ctrl_an_resp  an;
0aaee59
+		struct virtio_scsi_event         evt;
0aaee59
+	} resp;
0aaee59
+} ____cacheline_aligned_in_smp;
0aaee59
+
0aaee59
+/* Driver instance state */
0aaee59
+struct virtio_scsi {
0aaee59
+	/* Protects ctrl_vq, req_vq and sg[] */
0aaee59
+	spinlock_t vq_lock;
0aaee59
+
0aaee59
+	struct virtio_device *vdev;
0aaee59
+	struct virtqueue *ctrl_vq;
0aaee59
+	struct virtqueue *event_vq;
0aaee59
+	struct virtqueue *req_vq;
0aaee59
+
0aaee59
+	/* For sglist construction when adding commands to the virtqueue.  */
0aaee59
+	struct scatterlist sg[];
0aaee59
+};
0aaee59
+
0aaee59
+static struct kmem_cache *virtscsi_cmd_cache;
0aaee59
+static mempool_t *virtscsi_cmd_pool;
0aaee59
+
0aaee59
+static inline struct Scsi_Host *virtio_scsi_host(struct virtio_device *vdev)
0aaee59
+{
0aaee59
+	return vdev->priv;
0aaee59
+}
0aaee59
+
0aaee59
+static void virtscsi_compute_resid(struct scsi_cmnd *sc, u32 resid)
0aaee59
+{
0aaee59
+	if (!resid)
0aaee59
+		return;
0aaee59
+
0aaee59
+	if (!scsi_bidi_cmnd(sc)) {
0aaee59
+		scsi_set_resid(sc, resid);
0aaee59
+		return;
0aaee59
+	}
0aaee59
+
0aaee59
+	scsi_in(sc)->resid = min(resid, scsi_in(sc)->length);
0aaee59
+	scsi_out(sc)->resid = resid - scsi_in(sc)->resid;
0aaee59
+}
0aaee59
+
0aaee59
+/**
0aaee59
+ * virtscsi_complete_cmd - finish a scsi_cmd and invoke scsi_done
0aaee59
+ *
0aaee59
+ * Called with vq_lock held.
0aaee59
+ */
0aaee59
+static void virtscsi_complete_cmd(void *buf)
0aaee59
+{
0aaee59
+	struct virtio_scsi_cmd *cmd = buf;
0aaee59
+	struct scsi_cmnd *sc = cmd->sc;
0aaee59
+	struct virtio_scsi_cmd_resp *resp = &cmd->resp.cmd;
0aaee59
+
0aaee59
+	dev_dbg(&sc->device->sdev_gendev,
0aaee59
+		"cmd %p response %u status %#02x sense_len %u\n",
0aaee59
+		sc, resp->response, resp->status, resp->sense_len);
0aaee59
+
0aaee59
+	sc->result = resp->status;
0aaee59
+	virtscsi_compute_resid(sc, resp->resid);
0aaee59
+	switch (resp->response) {
0aaee59
+	case VIRTIO_SCSI_S_OK:
0aaee59
+		set_host_byte(sc, DID_OK);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_OVERRUN:
0aaee59
+		set_host_byte(sc, DID_ERROR);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_ABORTED:
0aaee59
+		set_host_byte(sc, DID_ABORT);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_BAD_TARGET:
0aaee59
+		set_host_byte(sc, DID_BAD_TARGET);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_RESET:
0aaee59
+		set_host_byte(sc, DID_RESET);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_BUSY:
0aaee59
+		set_host_byte(sc, DID_BUS_BUSY);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_TRANSPORT_FAILURE:
0aaee59
+		set_host_byte(sc, DID_TRANSPORT_DISRUPTED);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_TARGET_FAILURE:
0aaee59
+		set_host_byte(sc, DID_TARGET_FAILURE);
0aaee59
+		break;
0aaee59
+	case VIRTIO_SCSI_S_NEXUS_FAILURE:
0aaee59
+		set_host_byte(sc, DID_NEXUS_FAILURE);
0aaee59
+		break;
0aaee59
+	default:
0aaee59
+		scmd_printk(KERN_WARNING, sc, "Unknown response %d",
0aaee59
+			    resp->response);
0aaee59
+		/* fall through */
0aaee59
+	case VIRTIO_SCSI_S_FAILURE:
0aaee59
+		set_host_byte(sc, DID_ERROR);
0aaee59
+		break;
0aaee59
+	}
0aaee59
+
0aaee59
+	WARN_ON(resp->sense_len > VIRTIO_SCSI_SENSE_SIZE);
0aaee59
+	if (sc->sense_buffer) {
0aaee59
+		memcpy(sc->sense_buffer, resp->sense,
0aaee59
+		       min_t(u32, resp->sense_len, VIRTIO_SCSI_SENSE_SIZE));
0aaee59
+		if (resp->sense_len)
0aaee59
+			set_driver_byte(sc, DRIVER_SENSE);
0aaee59
+	}
0aaee59
+
0aaee59
+	mempool_free(cmd, virtscsi_cmd_pool);
0aaee59
+	sc->scsi_done(sc);
0aaee59
+}
0aaee59
+
0aaee59
+static void virtscsi_vq_done(struct virtqueue *vq, void (*fn)(void *buf))
0aaee59
+{
0aaee59
+	struct Scsi_Host *sh = virtio_scsi_host(vq->vdev);
0aaee59
+	struct virtio_scsi *vscsi = shost_priv(sh);
0aaee59
+	void *buf;
0aaee59
+	unsigned long flags;
0aaee59
+	unsigned int len;
0aaee59
+
0aaee59
+	spin_lock_irqsave(&vscsi->vq_lock, flags);
0aaee59
+
0aaee59
+	do {
0aaee59
+		virtqueue_disable_cb(vq);
0aaee59
+		while ((buf = virtqueue_get_buf(vq, &len)) != NULL)
0aaee59
+			fn(buf);
0aaee59
+	} while (!virtqueue_enable_cb(vq));
0aaee59
+
0aaee59
+	spin_unlock_irqrestore(&vscsi->vq_lock, flags);
0aaee59
+}
0aaee59
+
0aaee59
+static void virtscsi_req_done(struct virtqueue *vq)
0aaee59
+{
0aaee59
+	virtscsi_vq_done(vq, virtscsi_complete_cmd);
0aaee59
+};
0aaee59
+
0aaee59
+/* These are still stubs.  */
0aaee59
+static void virtscsi_complete_free(void *buf)
0aaee59
+{
0aaee59
+	struct virtio_scsi_cmd *cmd = buf;
0aaee59
+
0aaee59
+	mempool_free(cmd, virtscsi_cmd_pool);
0aaee59
+}
0aaee59
+
0aaee59
+static void virtscsi_ctrl_done(struct virtqueue *vq)
0aaee59
+{
0aaee59
+	virtscsi_vq_done(vq, virtscsi_complete_free);
0aaee59
+};
0aaee59
+
0aaee59
+static void virtscsi_event_done(struct virtqueue *vq)
0aaee59
+{
0aaee59
+	virtscsi_vq_done(vq, virtscsi_complete_free);
0aaee59
+};
0aaee59
+
0aaee59
+static void virtscsi_map_sgl(struct scatterlist *sg, unsigned int *p_idx,
0aaee59
+			     struct scsi_data_buffer *sdb)
0aaee59
+{
0aaee59
+	struct sg_table *table = &sdb->table;
0aaee59
+	struct scatterlist *sg_elem;
0aaee59
+	unsigned int idx = *p_idx;
0aaee59
+	int i;
0aaee59
+
0aaee59
+	for_each_sg(table->sgl, sg_elem, table->nents, i)
0aaee59
+		sg_set_buf(&sg[idx++], sg_virt(sg_elem), sg_elem->length);
0aaee59
+
0aaee59
+	*p_idx = idx;
0aaee59
+}
0aaee59
+
0aaee59
+/**
0aaee59
+ * virtscsi_map_cmd - map a scsi_cmd to a virtqueue scatterlist
0aaee59
+ * @vscsi	: virtio_scsi state
0aaee59
+ * @cmd		: command structure
0aaee59
+ * @out_num	: number of read-only elements
0aaee59
+ * @in_num	: number of write-only elements
0aaee59
+ * @req_size	: size of the request buffer
0aaee59
+ * @resp_size	: size of the response buffer
0aaee59
+ *
0aaee59
+ * Called with vq_lock held.
0aaee59
+ */
0aaee59
+static void virtscsi_map_cmd(struct virtio_scsi *vscsi,
0aaee59
+			     struct virtio_scsi_cmd *cmd,
0aaee59
+			     unsigned *out_num, unsigned *in_num,
0aaee59
+			     size_t req_size, size_t resp_size)
0aaee59
+{
0aaee59
+	struct scsi_cmnd *sc = cmd->sc;
0aaee59
+	struct scatterlist *sg = vscsi->sg;
0aaee59
+	unsigned int idx = 0;
0aaee59
+
0aaee59
+	if (sc) {
0aaee59
+		struct Scsi_Host *shost = virtio_scsi_host(vscsi->vdev);
0aaee59
+		BUG_ON(scsi_sg_count(sc) > shost->sg_tablesize);
0aaee59
+
0aaee59
+		/* TODO: check feature bit and fail if unsupported?  */
0aaee59
+		BUG_ON(sc->sc_data_direction == DMA_BIDIRECTIONAL);
0aaee59
+	}
0aaee59
+
0aaee59
+	/* Request header.  */
0aaee59
+	sg_set_buf(&sg[idx++], &cmd->req, req_size);
0aaee59
+
0aaee59
+	/* Data-out buffer.  */
0aaee59
+	if (sc && sc->sc_data_direction != DMA_FROM_DEVICE)
0aaee59
+		virtscsi_map_sgl(sg, &idx, scsi_out(sc));
0aaee59
+
0aaee59
+	*out_num = idx;
0aaee59
+
0aaee59
+	/* Response header.  */
0aaee59
+	sg_set_buf(&sg[idx++], &cmd->resp, resp_size);
0aaee59
+
0aaee59
+	/* Data-in buffer */
0aaee59
+	if (sc && sc->sc_data_direction != DMA_TO_DEVICE)
0aaee59
+		virtscsi_map_sgl(sg, &idx, scsi_in(sc));
0aaee59
+
0aaee59
+	*in_num = idx - *out_num;
0aaee59
+}
0aaee59
+
0aaee59
+static int virtscsi_kick_cmd(struct virtio_scsi *vscsi, struct virtqueue *vq,
0aaee59
+			     struct virtio_scsi_cmd *cmd,
0aaee59
+			     size_t req_size, size_t resp_size, gfp_t gfp)
0aaee59
+{
0aaee59
+	unsigned int out_num, in_num;
0aaee59
+	unsigned long flags;
0aaee59
+	int ret;
0aaee59
+
0aaee59
+	spin_lock_irqsave(&vscsi->vq_lock, flags);
0aaee59
+
0aaee59
+	virtscsi_map_cmd(vscsi, cmd, &out_num, &in_num, req_size, resp_size);
0aaee59
+
0aaee59
+	ret = virtqueue_add_buf(vq, vscsi->sg, out_num, in_num, cmd, gfp);
0aaee59
+	if (ret >= 0)
0aaee59
+		virtqueue_kick(vq);
0aaee59
+
0aaee59
+	spin_unlock_irqrestore(&vscsi->vq_lock, flags);
0aaee59
+	return ret;
0aaee59
+}
0aaee59
+
0aaee59
+static int virtscsi_queuecommand(struct Scsi_Host *sh, struct scsi_cmnd *sc)
0aaee59
+{
0aaee59
+	struct virtio_scsi *vscsi = shost_priv(sh);
0aaee59
+	struct virtio_scsi_cmd *cmd;
0aaee59
+	int ret;
0aaee59
+
0aaee59
+	dev_dbg(&sc->device->sdev_gendev,
0aaee59
+		"cmd %p CDB: %#02x\n", sc, sc->cmnd[0]);
0aaee59
+
0aaee59
+	ret = SCSI_MLQUEUE_HOST_BUSY;
0aaee59
+	cmd = mempool_alloc(virtscsi_cmd_pool, GFP_ATOMIC);
0aaee59
+	if (!cmd)
0aaee59
+		goto out;
0aaee59
+
0aaee59
+	memset(cmd, 0, sizeof(*cmd));
0aaee59
+	cmd->sc = sc;
0aaee59
+	cmd->req.cmd = (struct virtio_scsi_cmd_req){
0aaee59
+		.lun[0] = 1,
0aaee59
+		.lun[1] = sc->device->id,
0aaee59
+		.lun[2] = (sc->device->lun >> 8) | 0x40,
0aaee59
+		.lun[3] = sc->device->lun & 0xff,
0aaee59
+		.tag = (unsigned long)sc,
0aaee59
+		.task_attr = VIRTIO_SCSI_S_SIMPLE,
0aaee59
+		.prio = 0,
0aaee59
+		.crn = 0,
0aaee59
+	};
0aaee59
+
0aaee59
+	BUG_ON(sc->cmd_len > VIRTIO_SCSI_CDB_SIZE);
0aaee59
+	memcpy(cmd->req.cmd.cdb, sc->cmnd, sc->cmd_len);
0aaee59
+
0aaee59
+	if (virtscsi_kick_cmd(vscsi, vscsi->req_vq, cmd,
0aaee59
+			      sizeof cmd->req.cmd, sizeof cmd->resp.cmd,
0aaee59
+			      GFP_ATOMIC) >= 0)
0aaee59
+		ret = 0;
0aaee59
+
0aaee59
+out:
0aaee59
+	return ret;
0aaee59
+}
0aaee59
+
0aaee59
+static struct scsi_host_template virtscsi_host_template = {
0aaee59
+	.module = THIS_MODULE,
0aaee59
+	.name = "Virtio SCSI HBA",
0aaee59
+	.proc_name = "virtio_scsi",
0aaee59
+	.queuecommand = virtscsi_queuecommand,
0aaee59
+	.this_id = -1,
0aaee59
+
0aaee59
+	.can_queue = 1024,
0aaee59
+	.dma_boundary = UINT_MAX,
0aaee59
+	.use_clustering = ENABLE_CLUSTERING,
0aaee59
+};
0aaee59
+
0aaee59
+#define virtscsi_config_get(vdev, fld) \
0aaee59
+	({ \
0aaee59
+		typeof(((struct virtio_scsi_config *)0)->fld) __val; \
0aaee59
+		vdev->config->get(vdev, \
0aaee59
+				  offsetof(struct virtio_scsi_config, fld), \
0aaee59
+				  &__val, sizeof(__val)); \
0aaee59
+		__val; \
0aaee59
+	})
0aaee59
+
0aaee59
+#define virtscsi_config_set(vdev, fld, val) \
0aaee59
+	(void)({ \
0aaee59
+		typeof(((struct virtio_scsi_config *)0)->fld) __val = (val); \
0aaee59
+		vdev->config->set(vdev, \
0aaee59
+				  offsetof(struct virtio_scsi_config, fld), \
0aaee59
+				  &__val, sizeof(__val)); \
0aaee59
+	})
0aaee59
+
0aaee59
+static int __devinit virtscsi_init(struct virtio_device *vdev,
0aaee59
+				   struct virtio_scsi *vscsi)
0aaee59
+{
0aaee59
+	int err;
0aaee59
+	struct virtqueue *vqs[3];
0aaee59
+	vq_callback_t *callbacks[] = {
0aaee59
+		virtscsi_ctrl_done,
0aaee59
+		virtscsi_event_done,
0aaee59
+		virtscsi_req_done
0aaee59
+	};
0aaee59
+	const char *names[] = {
0aaee59
+		"control",
0aaee59
+		"event",
0aaee59
+		"request"
0aaee59
+	};
0aaee59
+
0aaee59
+	/* Discover virtqueues and write information to configuration.  */
0aaee59
+	err = vdev->config->find_vqs(vdev, 3, vqs, callbacks, names);
0aaee59
+	if (err)
0aaee59
+		return err;
0aaee59
+
0aaee59
+	vscsi->ctrl_vq = vqs[0];
0aaee59
+	vscsi->event_vq = vqs[1];
0aaee59
+	vscsi->req_vq = vqs[2];
0aaee59
+
0aaee59
+	virtscsi_config_set(vdev, cdb_size, VIRTIO_SCSI_CDB_SIZE);
0aaee59
+	virtscsi_config_set(vdev, sense_size, VIRTIO_SCSI_SENSE_SIZE);
0aaee59
+	return 0;
0aaee59
+}
0aaee59
+
0aaee59
+static int __devinit virtscsi_probe(struct virtio_device *vdev)
0aaee59
+{
0aaee59
+	struct Scsi_Host *shost;
0aaee59
+	struct virtio_scsi *vscsi;
0aaee59
+	int err;
0aaee59
+	u32 sg_elems;
0aaee59
+	u32 cmd_per_lun;
0aaee59
+
0aaee59
+	/* We need to know how many segments before we allocate.
0aaee59
+	 * We need an extra sg elements at head and tail.
0aaee59
+	 */
0aaee59
+	sg_elems = virtscsi_config_get(vdev, seg_max) ?: 1;
0aaee59
+
0aaee59
+	/* Allocate memory and link the structs together.  */
0aaee59
+	shost = scsi_host_alloc(&virtscsi_host_template,
0aaee59
+		sizeof(*vscsi) + sizeof(vscsi->sg[0]) * (sg_elems + 2));
0aaee59
+
0aaee59
+	if (!shost)
0aaee59
+		return -ENOMEM;
0aaee59
+
0aaee59
+	shost->sg_tablesize = sg_elems;
0aaee59
+	vscsi = shost_priv(shost);
0aaee59
+	vscsi->vdev = vdev;
0aaee59
+	vdev->priv = shost;
0aaee59
+
0aaee59
+	/* Random initializations.  */
0aaee59
+	spin_lock_init(&vscsi->vq_lock);
0aaee59
+	sg_init_table(vscsi->sg, sg_elems + 2);
0aaee59
+
0aaee59
+	err = virtscsi_init(vdev, vscsi);
0aaee59
+	if (err)
0aaee59
+		goto virtscsi_init_failed;
0aaee59
+
0aaee59
+	cmd_per_lun = virtscsi_config_get(vdev, cmd_per_lun) ?: 1;
0aaee59
+	shost->cmd_per_lun = min_t(u32, cmd_per_lun, shost->can_queue);
0aaee59
+	shost->max_sectors = virtscsi_config_get(vdev, max_sectors) ?: 0xFFFF;
0aaee59
+	shost->max_lun = virtscsi_config_get(vdev, max_lun) + 1;
0aaee59
+	shost->max_id = virtscsi_config_get(vdev, max_target) + 1;
0aaee59
+	shost->max_channel = 0;
0aaee59
+	shost->max_cmd_len = VIRTIO_SCSI_CDB_SIZE;
0aaee59
+	err = scsi_add_host(shost, &vdev->dev);
0aaee59
+	if (err)
0aaee59
+		goto scsi_add_host_failed;
0aaee59
+
0aaee59
+	scsi_scan_host(shost);
0aaee59
+
0aaee59
+	return 0;
0aaee59
+
0aaee59
+scsi_add_host_failed:
0aaee59
+	vdev->config->del_vqs(vdev);
0aaee59
+virtscsi_init_failed:
0aaee59
+	scsi_host_put(shost);
0aaee59
+	return err;
0aaee59
+}
0aaee59
+
0aaee59
+static void __devexit virtscsi_remove_vqs(struct virtio_device *vdev)
0aaee59
+{
0aaee59
+	/* Stop all the virtqueues. */
0aaee59
+	vdev->config->reset(vdev);
0aaee59
+
0aaee59
+	vdev->config->del_vqs(vdev);
0aaee59
+}
0aaee59
+
0aaee59
+static void __devexit virtscsi_remove(struct virtio_device *vdev)
0aaee59
+{
0aaee59
+	struct Scsi_Host *shost = virtio_scsi_host(vdev);
0aaee59
+
0aaee59
+	scsi_remove_host(shost);
0aaee59
+
0aaee59
+	virtscsi_remove_vqs(vdev);
0aaee59
+	scsi_host_put(shost);
0aaee59
+}
0aaee59
+
0aaee59
+static struct virtio_device_id id_table[] = {
0aaee59
+	{ VIRTIO_ID_SCSI, VIRTIO_DEV_ANY_ID },
0aaee59
+	{ 0 },
0aaee59
+};
0aaee59
+
0aaee59
+static struct virtio_driver virtio_scsi_driver = {
0aaee59
+	.driver.name = KBUILD_MODNAME,
0aaee59
+	.driver.owner = THIS_MODULE,
0aaee59
+	.id_table = id_table,
0aaee59
+	.probe = virtscsi_probe,
0aaee59
+	.remove = __devexit_p(virtscsi_remove),
0aaee59
+};
0aaee59
+
0aaee59
+static int __init init(void)
0aaee59
+{
0aaee59
+	int ret = -ENOMEM;
0aaee59
+
0aaee59
+	virtscsi_cmd_cache = KMEM_CACHE(virtio_scsi_cmd, 0);
0aaee59
+	if (!virtscsi_cmd_cache) {
0aaee59
+		printk(KERN_ERR "kmem_cache_create() for "
0aaee59
+				"virtscsi_cmd_cache failed\n");
0aaee59
+		goto error;
0aaee59
+	}
0aaee59
+
0aaee59
+
0aaee59
+	virtscsi_cmd_pool =
0aaee59
+		mempool_create_slab_pool(VIRTIO_SCSI_MEMPOOL_SZ,
0aaee59
+					 virtscsi_cmd_cache);
0aaee59
+	if (!virtscsi_cmd_pool) {
0aaee59
+		printk(KERN_ERR "mempool_create() for"
0aaee59
+				"virtscsi_cmd_pool failed\n");
0aaee59
+		goto error;
0aaee59
+	}
0aaee59
+	ret = register_virtio_driver(&virtio_scsi_driver);
0aaee59
+	if (ret < 0)
0aaee59
+		goto error;
0aaee59
+
0aaee59
+	return 0;
0aaee59
+
0aaee59
+error:
0aaee59
+	if (virtscsi_cmd_pool) {
0aaee59
+		mempool_destroy(virtscsi_cmd_pool);
0aaee59
+		virtscsi_cmd_pool = NULL;
0aaee59
+	}
0aaee59
+	if (virtscsi_cmd_cache) {
0aaee59
+		kmem_cache_destroy(virtscsi_cmd_cache);
0aaee59
+		virtscsi_cmd_cache = NULL;
0aaee59
+	}
0aaee59
+	return ret;
0aaee59
+}
0aaee59
+
0aaee59
+static void __exit fini(void)
0aaee59
+{
0aaee59
+	unregister_virtio_driver(&virtio_scsi_driver);
0aaee59
+	mempool_destroy(virtscsi_cmd_pool);
0aaee59
+	kmem_cache_destroy(virtscsi_cmd_cache);
0aaee59
+}
0aaee59
+module_init(init);
0aaee59
+module_exit(fini);
0aaee59
+
0aaee59
+MODULE_DEVICE_TABLE(virtio, id_table);
0aaee59
+MODULE_DESCRIPTION("Virtio SCSI HBA driver");
0aaee59
+MODULE_LICENSE("GPL");
0aaee59
diff --git a/include/linux/virtio_ids.h b/include/linux/virtio_ids.h
0aaee59
index 85bb0bb..d83ae52 100644
0aaee59
--- a/include/linux/virtio_ids.h
0aaee59
+++ b/include/linux/virtio_ids.h
0aaee59
@@ -34,6 +34,7 @@
0aaee59
 #define VIRTIO_ID_CONSOLE	3 /* virtio console */
0aaee59
 #define VIRTIO_ID_RNG		4 /* virtio ring */
0aaee59
 #define VIRTIO_ID_BALLOON	5 /* virtio balloon */
0aaee59
+#define VIRTIO_ID_SCSI		8 /* virtio scsi */
0aaee59
 #define VIRTIO_ID_9P		9 /* 9p virtio console */
0aaee59
 
0aaee59
 #endif /* _LINUX_VIRTIO_IDS_H */
0aaee59
diff --git a/include/linux/virtio_scsi.h b/include/linux/virtio_scsi.h
0aaee59
new file mode 100644
0aaee59
index 0000000..8ddeafd
0aaee59
--- /dev/null
0aaee59
+++ b/include/linux/virtio_scsi.h
0aaee59
@@ -0,0 +1,114 @@
0aaee59
+#ifndef _LINUX_VIRTIO_SCSI_H
0aaee59
+#define _LINUX_VIRTIO_SCSI_H
0aaee59
+/* This header is BSD licensed so anyone can use the definitions to implement
0aaee59
+ * compatible drivers/servers. */
0aaee59
+
0aaee59
+#define VIRTIO_SCSI_CDB_SIZE   32
0aaee59
+#define VIRTIO_SCSI_SENSE_SIZE 96
0aaee59
+
0aaee59
+/* SCSI command request, followed by data-out */
0aaee59
+struct virtio_scsi_cmd_req {
0aaee59
+	u8 lun[8];		/* Logical Unit Number */
0aaee59
+	u64 tag;		/* Command identifier */
0aaee59
+	u8 task_attr;		/* Task attribute */
0aaee59
+	u8 prio;
0aaee59
+	u8 crn;
0aaee59
+	u8 cdb[VIRTIO_SCSI_CDB_SIZE];
0aaee59
+} __packed;
0aaee59
+
0aaee59
+/* Response, followed by sense data and data-in */
0aaee59
+struct virtio_scsi_cmd_resp {
0aaee59
+	u32 sense_len;		/* Sense data length */
0aaee59
+	u32 resid;		/* Residual bytes in data buffer */
0aaee59
+	u16 status_qualifier;	/* Status qualifier */
0aaee59
+	u8 status;		/* Command completion status */
0aaee59
+	u8 response;		/* Response values */
0aaee59
+	u8 sense[VIRTIO_SCSI_SENSE_SIZE];
0aaee59
+} __packed;
0aaee59
+
0aaee59
+/* Task Management Request */
0aaee59
+struct virtio_scsi_ctrl_tmf_req {
0aaee59
+	u32 type;
0aaee59
+	u32 subtype;
0aaee59
+	u8 lun[8];
0aaee59
+	u64 tag;
0aaee59
+} __packed;
0aaee59
+
0aaee59
+struct virtio_scsi_ctrl_tmf_resp {
0aaee59
+	u8 response;
0aaee59
+} __packed;
0aaee59
+
0aaee59
+/* Asynchronous notification query/subscription */
0aaee59
+struct virtio_scsi_ctrl_an_req {
0aaee59
+	u32 type;
0aaee59
+	u8 lun[8];
0aaee59
+	u32 event_requested;
0aaee59
+} __packed;
0aaee59
+
0aaee59
+struct virtio_scsi_ctrl_an_resp {
0aaee59
+	u32 event_actual;
0aaee59
+	u8 response;
0aaee59
+} __packed;
0aaee59
+
0aaee59
+struct virtio_scsi_event {
0aaee59
+	u32 event;
0aaee59
+	u8 lun[8];
0aaee59
+	u32 reason;
0aaee59
+} __packed;
0aaee59
+
0aaee59
+struct virtio_scsi_config {
0aaee59
+	u32 num_queues;
0aaee59
+	u32 seg_max;
0aaee59
+	u32 max_sectors;
0aaee59
+	u32 cmd_per_lun;
0aaee59
+	u32 event_info_size;
0aaee59
+	u32 sense_size;
0aaee59
+	u32 cdb_size;
0aaee59
+	u16 max_channel;
0aaee59
+	u16 max_target;
0aaee59
+	u32 max_lun;
0aaee59
+} __packed;
0aaee59
+
0aaee59
+/* Response codes */
0aaee59
+#define VIRTIO_SCSI_S_OK                       0
0aaee59
+#define VIRTIO_SCSI_S_OVERRUN                  1
0aaee59
+#define VIRTIO_SCSI_S_ABORTED                  2
0aaee59
+#define VIRTIO_SCSI_S_BAD_TARGET               3
0aaee59
+#define VIRTIO_SCSI_S_RESET                    4
0aaee59
+#define VIRTIO_SCSI_S_BUSY                     5
0aaee59
+#define VIRTIO_SCSI_S_TRANSPORT_FAILURE        6
0aaee59
+#define VIRTIO_SCSI_S_TARGET_FAILURE           7
0aaee59
+#define VIRTIO_SCSI_S_NEXUS_FAILURE            8
0aaee59
+#define VIRTIO_SCSI_S_FAILURE                  9
0aaee59
+#define VIRTIO_SCSI_S_FUNCTION_SUCCEEDED       10
0aaee59
+#define VIRTIO_SCSI_S_FUNCTION_REJECTED        11
0aaee59
+#define VIRTIO_SCSI_S_INCORRECT_LUN            12
0aaee59
+
0aaee59
+/* Controlq type codes.  */
0aaee59
+#define VIRTIO_SCSI_T_TMF                      0
0aaee59
+#define VIRTIO_SCSI_T_AN_QUERY                 1
0aaee59
+#define VIRTIO_SCSI_T_AN_SUBSCRIBE             2
0aaee59
+
0aaee59
+/* Valid TMF subtypes.  */
0aaee59
+#define VIRTIO_SCSI_T_TMF_ABORT_TASK           0
0aaee59
+#define VIRTIO_SCSI_T_TMF_ABORT_TASK_SET       1
0aaee59
+#define VIRTIO_SCSI_T_TMF_CLEAR_ACA            2
0aaee59
+#define VIRTIO_SCSI_T_TMF_CLEAR_TASK_SET       3
0aaee59
+#define VIRTIO_SCSI_T_TMF_I_T_NEXUS_RESET      4
0aaee59
+#define VIRTIO_SCSI_T_TMF_LOGICAL_UNIT_RESET   5
0aaee59
+#define VIRTIO_SCSI_T_TMF_QUERY_TASK           6
0aaee59
+#define VIRTIO_SCSI_T_TMF_QUERY_TASK_SET       7
0aaee59
+
0aaee59
+/* Events.  */
0aaee59
+#define VIRTIO_SCSI_T_EVENTS_MISSED            0x80000000
0aaee59
+#define VIRTIO_SCSI_T_NO_EVENT                 0
0aaee59
+#define VIRTIO_SCSI_T_TRANSPORT_RESET          1
0aaee59
+#define VIRTIO_SCSI_T_ASYNC_NOTIFY             2
0aaee59
+
0aaee59
+#define VIRTIO_SCSI_S_SIMPLE                   0
0aaee59
+#define VIRTIO_SCSI_S_ORDERED                  1
0aaee59
+#define VIRTIO_SCSI_S_HEAD                     2
0aaee59
+#define VIRTIO_SCSI_S_ACA                      3
0aaee59
+
0aaee59
+
0aaee59
+#endif /* _LINUX_VIRTIO_SCSI_H */
0aaee59
-- 
0aaee59
1.7.1
0aaee59
0aaee59
0aaee59
From 3c0e8846ac0fc2175dd0e06f495b16a30b549762 Mon Sep 17 00:00:00 2001
0aaee59
From: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
Date: Tue, 29 Nov 2011 16:33:28 +0100
0aaee59
Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>, Mike Christie <michaelc@cs.wisc.edu>, Pekka Enberg <penberg@kernel.org>
0aaee59
Subject: [PATCH v5 2/3] virtio-scsi: add error handling
0aaee59
0aaee59
This commit adds basic error handling to the virtio-scsi
0aaee59
HBA device.  Task management functions are sent synchronously
0aaee59
via the control virtqueue.
0aaee59
0aaee59
Cc: linux-scsi <linux-scsi@vger.kernel.org>
0aaee59
Cc: Rusty Russell <rusty@rustcorp.com.au>
0aaee59
Cc: Michael S. Tsirkin <mst@redhat.com>
0aaee59
Cc: kvm@vger.kernel.org
0aaee59
Acked-by: Pekka Enberg <penberg@kernel.org> 
0aaee59
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
---
0aaee59
	v3->v4: fixed 32-bit compilation; adjusted call to virtscsi_kick_cmd
0aaee59
0aaee59
	v2->v3: added mempool, used GFP_NOIO instead of GFP_ATOMIC,
0aaee59
	formatting fixes
0aaee59
0aaee59
	v1->v2: use scmd_printk
0aaee59
0aaee59
 drivers/scsi/virtio_scsi.c |   73 +++++++++++++++++++++++++++++++++++++++++++-
0aaee59
 1 files changed, 72 insertions(+), 1 deletions(-)
0aaee59
0aaee59
diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c
0aaee59
index 3f87ae0..68104cd 100644
0aaee59
--- a/drivers/scsi/virtio_scsi.c
0aaee59
+++ b/drivers/scsi/virtio_scsi.c
0aaee59
@@ -29,6 +29,7 @@
0aaee59
 /* Command queue element */
0aaee59
 struct virtio_scsi_cmd {
0aaee59
 	struct scsi_cmnd *sc;
0aaee59
+	struct completion *comp;
0aaee59
 	union {
0aaee59
 		struct virtio_scsi_cmd_req       cmd;
0aaee59
 		struct virtio_scsi_ctrl_tmf_req  tmf;
0aaee59
@@ -168,11 +169,12 @@ static void virtscsi_req_done(struct virtqueue *vq)
0aaee59
 	virtscsi_vq_done(vq, virtscsi_complete_cmd);
0aaee59
 };
0aaee59
 
0aaee59
-/* These are still stubs.  */
0aaee59
 static void virtscsi_complete_free(void *buf)
0aaee59
 {
0aaee59
 	struct virtio_scsi_cmd *cmd = buf;
0aaee59
 
0aaee59
+	if (cmd->comp)
0aaee59
+		complete_all(cmd->comp);
0aaee59
 	mempool_free(cmd, virtscsi_cmd_pool);
0aaee59
 }
0aaee59
 
0aaee59
@@ -306,12 +308,81 @@ out:
0aaee59
 	return ret;
0aaee59
 }
0aaee59
 
0aaee59
+static int virtscsi_tmf(struct virtio_scsi *vscsi, struct virtio_scsi_cmd *cmd)
0aaee59
+{
0aaee59
+	DECLARE_COMPLETION_ONSTACK(comp);
0aaee59
+	int ret;
0aaee59
+
0aaee59
+	cmd->comp = ∁
0aaee59
+	ret = virtscsi_kick_cmd(vscsi, vscsi->ctrl_vq, cmd,
0aaee59
+			       sizeof cmd->req.tmf, sizeof cmd->resp.tmf,
0aaee59
+			       GFP_NOIO);
0aaee59
+	if (ret < 0)
0aaee59
+		return FAILED;
0aaee59
+
0aaee59
+	wait_for_completion(&comp);
0aaee59
+	if (cmd->resp.tmf.response != VIRTIO_SCSI_S_OK &&
0aaee59
+	    cmd->resp.tmf.response != VIRTIO_SCSI_S_FUNCTION_SUCCEEDED)
0aaee59
+		return FAILED;
0aaee59
+
0aaee59
+	return SUCCESS;
0aaee59
+}
0aaee59
+
0aaee59
+static int virtscsi_device_reset(struct scsi_cmnd *sc)
0aaee59
+{
0aaee59
+	struct virtio_scsi *vscsi = shost_priv(sc->device->host);
0aaee59
+	struct virtio_scsi_cmd *cmd;
0aaee59
+
0aaee59
+	sdev_printk(KERN_INFO, sc->device, "device reset\n");
0aaee59
+	cmd = mempool_alloc(virtscsi_cmd_pool, GFP_NOIO);
0aaee59
+	if (!cmd)
0aaee59
+		return FAILED;
0aaee59
+
0aaee59
+	memset(cmd, 0, sizeof(*cmd));
0aaee59
+	cmd->sc = sc;
0aaee59
+	cmd->req.tmf = (struct virtio_scsi_ctrl_tmf_req){
0aaee59
+		.type = VIRTIO_SCSI_T_TMF,
0aaee59
+		.subtype = VIRTIO_SCSI_T_TMF_LOGICAL_UNIT_RESET,
0aaee59
+		.lun[0] = 1,
0aaee59
+		.lun[1] = sc->device->id,
0aaee59
+		.lun[2] = (sc->device->lun >> 8) | 0x40,
0aaee59
+		.lun[3] = sc->device->lun & 0xff,
0aaee59
+	};
0aaee59
+	return virtscsi_tmf(vscsi, cmd);
0aaee59
+}
0aaee59
+
0aaee59
+static int virtscsi_abort(struct scsi_cmnd *sc)
0aaee59
+{
0aaee59
+	struct virtio_scsi *vscsi = shost_priv(sc->device->host);
0aaee59
+	struct virtio_scsi_cmd *cmd;
0aaee59
+
0aaee59
+	scmd_printk(KERN_INFO, sc, "abort\n");
0aaee59
+	cmd = mempool_alloc(virtscsi_cmd_pool, GFP_NOIO);
0aaee59
+	if (!cmd)
0aaee59
+		return FAILED;
0aaee59
+
0aaee59
+	memset(cmd, 0, sizeof(*cmd));
0aaee59
+	cmd->sc = sc;
0aaee59
+	cmd->req.tmf = (struct virtio_scsi_ctrl_tmf_req){
0aaee59
+		.type = VIRTIO_SCSI_T_TMF,
0aaee59
+		.subtype = VIRTIO_SCSI_T_TMF_ABORT_TASK,
0aaee59
+		.lun[0] = 1,
0aaee59
+		.lun[1] = sc->device->id,
0aaee59
+		.lun[2] = (sc->device->lun >> 8) | 0x40,
0aaee59
+		.lun[3] = sc->device->lun & 0xff,
0aaee59
+		.tag = (unsigned long)sc,
0aaee59
+	};
0aaee59
+	return virtscsi_tmf(vscsi, cmd);
0aaee59
+}
0aaee59
+
0aaee59
 static struct scsi_host_template virtscsi_host_template = {
0aaee59
 	.module = THIS_MODULE,
0aaee59
 	.name = "Virtio SCSI HBA",
0aaee59
 	.proc_name = "virtio_scsi",
0aaee59
 	.queuecommand = virtscsi_queuecommand,
0aaee59
 	.this_id = -1,
0aaee59
+	.eh_abort_handler = virtscsi_abort,
0aaee59
+	.eh_device_reset_handler = virtscsi_device_reset,
0aaee59
 
0aaee59
 	.can_queue = 1024,
0aaee59
 	.dma_boundary = UINT_MAX,
0aaee59
-- 
0aaee59
1.7.1
0aaee59
0aaee59
0aaee59
From 43cf1b6a4ee31e69581042a0c85d1398f83dcedc Mon Sep 17 00:00:00 2001
0aaee59
From: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
Date: Fri, 13 Jan 2012 15:30:08 +0100
0aaee59
Cc: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>, Mike Christie <michaelc@cs.wisc.edu>, Pekka Enberg <penberg@kernel.org>
0aaee59
Subject: [PATCH v5 3/3] virtio-scsi: add power management support
0aaee59
0aaee59
This patch adds freeze/restore handlers for the HBA.  Block queues
0aaee59
are managed independently by the disk devices.
0aaee59
0aaee59
Cc: linux-scsi <linux-scsi@vger.kernel.org>
0aaee59
Cc: Rusty Russell <rusty@rustcorp.com.au>
0aaee59
Cc: Michael S. Tsirkin <mst@redhat.com>
0aaee59
Cc: kvm@vger.kernel.org
0aaee59
Acked-by: Pekka Enberg <penberg@kernel.org> 
0aaee59
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
0aaee59
---
0aaee59
	The feature has been merged in the virtio core for 3.3, so the patch
0aaee59
	is new in v4.
0aaee59
0aaee59
 drivers/scsi/virtio_scsi.c |   26 +++++++++++++++++++++++---
0aaee59
 1 files changed, 23 insertions(+), 3 deletions(-)
0aaee59
0aaee59
diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c
0aaee59
index 68104cd..efccd72 100644
0aaee59
--- a/drivers/scsi/virtio_scsi.c
0aaee59
+++ b/drivers/scsi/virtio_scsi.c
0aaee59
@@ -406,8 +406,8 @@ static struct scsi_host_template virtscsi_host_template = {
0aaee59
 				  &__val, sizeof(__val)); \
0aaee59
 	})
0aaee59
 
0aaee59
-static int __devinit virtscsi_init(struct virtio_device *vdev,
0aaee59
-				   struct virtio_scsi *vscsi)
0aaee59
+static int virtscsi_init(struct virtio_device *vdev,
0aaee59
+			 struct virtio_scsi *vscsi)
0aaee59
 {
0aaee59
 	int err;
0aaee59
 	struct virtqueue *vqs[3];
0aaee59
@@ -491,7 +491,7 @@ virtscsi_init_failed:
0aaee59
 	return err;
0aaee59
 }
0aaee59
 
0aaee59
-static void __devexit virtscsi_remove_vqs(struct virtio_device *vdev)
0aaee59
+static void virtscsi_remove_vqs(struct virtio_device *vdev)
0aaee59
 {
0aaee59
 	/* Stop all the virtqueues. */
0aaee59
 	vdev->config->reset(vdev);
0aaee59
@@ -509,6 +509,22 @@ static void __devexit virtscsi_remove(struct virtio_device *vdev)
0aaee59
 	scsi_host_put(shost);
0aaee59
 }
0aaee59
 
0aaee59
+#ifdef CONFIG_PM
0aaee59
+static int virtscsi_freeze(struct virtio_device *vdev)
0aaee59
+{
0aaee59
+	virtscsi_remove_vqs(vdev);
0aaee59
+	return 0;
0aaee59
+}
0aaee59
+
0aaee59
+static int virtscsi_restore(struct virtio_device *vdev)
0aaee59
+{
0aaee59
+	struct Scsi_Host *sh = virtio_scsi_host(vdev);
0aaee59
+	struct virtio_scsi *vscsi = shost_priv(sh);
0aaee59
+
0aaee59
+	return virtscsi_init(vdev, vscsi);
0aaee59
+}
0aaee59
+#endif
0aaee59
+
0aaee59
 static struct virtio_device_id id_table[] = {
0aaee59
 	{ VIRTIO_ID_SCSI, VIRTIO_DEV_ANY_ID },
0aaee59
 	{ 0 },
0aaee59
@@ -519,6 +535,10 @@ static struct virtio_driver virtio_scsi_driver = {
0aaee59
 	.driver.owner = THIS_MODULE,
0aaee59
 	.id_table = id_table,
0aaee59
 	.probe = virtscsi_probe,
0aaee59
+#ifdef CONFIG_PM
0aaee59
+	.freeze = virtscsi_freeze,
0aaee59
+	.restore = virtscsi_restore,
0aaee59
+#endif
0aaee59
 	.remove = __devexit_p(virtscsi_remove),
0aaee59
 };
0aaee59
 
0aaee59
-- 
0aaee59
1.7.1
0aaee59