[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH 01/10] scsi: simplify handling of the VPD page lengt
From: |
Paolo Bonzini |
Subject: |
[Qemu-devel] [PATCH 01/10] scsi: simplify handling of the VPD page length field |
Date: |
Tue, 12 Jun 2012 18:42:33 +0200 |
The last four bytes of the thin provisioning page were cut out.
Signed-off-by: Paolo Bonzini <address@hidden>
---
hw/scsi-disk.c | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c
index 045c764..2bd8907 100644
--- a/hw/scsi-disk.c
+++ b/hw/scsi-disk.c
@@ -511,6 +511,7 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
{
SCSIDiskState *s = DO_UPCAST(SCSIDiskState, qdev, req->dev);
int buflen = 0;
+ int start;
if (req->cmd.buf[1] & 0x1) {
/* Vital product data */
@@ -519,14 +520,14 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
outbuf[buflen++] = s->qdev.type & 0x1f;
outbuf[buflen++] = page_code ; // this page
outbuf[buflen++] = 0x00;
+ outbuf[buflen++] = 0x00;
+ start = buflen;
switch (page_code) {
case 0x00: /* Supported page codes, mandatory */
{
- int pages;
DPRINTF("Inquiry EVPD[Supported pages] "
"buffer size %zd\n", req->cmd.xfer);
- pages = buflen++;
outbuf[buflen++] = 0x00; // list of supported pages (this page)
if (s->serial) {
outbuf[buflen++] = 0x80; // unit serial number
@@ -536,7 +537,6 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
outbuf[buflen++] = 0xb0; // block limits
outbuf[buflen++] = 0xb2; // thin provisioning
}
- outbuf[pages] = buflen - pages - 1; // number of pages
break;
}
case 0x80: /* Device serial number, optional */
@@ -555,7 +555,6 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
DPRINTF("Inquiry EVPD[Serial number] "
"buffer size %zd\n", req->cmd.xfer);
- outbuf[buflen++] = l;
memcpy(outbuf+buflen, s->serial, l);
buflen += l;
break;
@@ -573,7 +572,6 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
DPRINTF("Inquiry EVPD[Device identification] "
"buffer size %zd\n", req->cmd.xfer);
- outbuf[buflen++] = 4 + id_len;
outbuf[buflen++] = 0x2; // ASCII
outbuf[buflen++] = 0; // not officially assigned
outbuf[buflen++] = 0; // reserved
@@ -598,8 +596,7 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
return -1;
}
/* required VPD size with unmap support */
- outbuf[3] = buflen = 0x3c;
-
+ buflen = 0x40;
memset(outbuf + 4, 0, buflen - 4);
/* optimal transfer length granularity */
@@ -621,7 +618,7 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
}
case 0xb2: /* thin provisioning */
{
- outbuf[3] = buflen = 8;
+ buflen = 8;
outbuf[4] = 0;
outbuf[5] = 0x60; /* write_same 10/16 supported */
outbuf[6] = s->qdev.conf.discard_granularity ? 2 : 1;
@@ -632,6 +629,8 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
uint8_t *outbuf)
return -1;
}
/* done with EVPD */
+ assert(buflen - start <= 255);
+ outbuf[start - 1] = buflen - start;
return buflen;
}
--
1.7.10.2
- [Qemu-devel] [PULL 00/10] SCSI updates for 2012-06-12, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 01/10] scsi: simplify handling of the VPD page length field,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 03/10] atapi: implement READ DISC INFORMATION, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 02/10] scsi: add a qdev property for the disk's WWN, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 04/10] scsi-disk: implement READ DISC INFORMATION, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 06/10] ISCSI: force use of sg for SMC and SSC devices, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 05/10] ISCSI: Add SCSI passthrough via scsi-generic to libiscsi, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 07/10] megasas: Add header file, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 08/10] megasas: LSI Megaraid SAS HBA emulation, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 10/10] virtio-scsi: do not crash on adding buffers to the event queue, Paolo Bonzini, 2012/06/12
- [Qemu-devel] [PATCH 09/10] megasas: Add trace events, Paolo Bonzini, 2012/06/12
- Re: [Qemu-devel] [PULL 00/10] SCSI updates for 2012-06-12, Anthony Liguori, 2012/06/20