|
 |
9bac43 |
From 23c1feb875805a4c324c51c3916b249da0edbe02 Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: "Daniel P. Berrange" <berrange@redhat.com>
|
|
|
9bac43 |
Date: Wed, 29 Nov 2017 14:26:04 +0100
|
|
|
9bac43 |
Subject: [PATCH 18/21] scsi-disk: support reporting of rotation rate
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: Daniel P. Berrange <berrange@redhat.com>
|
|
|
9bac43 |
Message-id: <20171129142606.15965-2-berrange@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 77961
|
|
|
9bac43 |
O-Subject: [PATCH RHV-7.5 qemu-kvm-rhev 1/3] scsi-disk: support reporting of rotation rate
|
|
|
9bac43 |
Bugzilla: 1498042
|
|
|
9bac43 |
RH-Acked-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: John Snow <jsnow@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
The Linux kernel will query the SCSI "Block device characteristics"
|
|
|
9bac43 |
VPD to determine the rotations per minute of the disk. If this has
|
|
|
9bac43 |
the value 1, it is taken to be an SSD and so Linux sets the
|
|
|
9bac43 |
'rotational' flag to 0 for the I/O queue and will stop using that
|
|
|
9bac43 |
disk as a source of random entropy. Other operating systems may
|
|
|
9bac43 |
also take into account rotation rate when setting up default
|
|
|
9bac43 |
behaviour.
|
|
|
9bac43 |
|
|
|
9bac43 |
Mgmt apps should be able to set the rotation rate for virtualized
|
|
|
9bac43 |
block devices, based on characteristics of the host storage in use,
|
|
|
9bac43 |
so that the guest OS gets sensible behaviour out of the box. This
|
|
|
9bac43 |
patch thus adds a 'rotation-rate' parameter for 'scsi-hd' and
|
|
|
9bac43 |
'scsi-block' device types. For the latter, this parameter will be
|
|
|
9bac43 |
ignored unless the host device has TYPE_DISK.
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
|
|
|
9bac43 |
Message-Id: <20171004114008.14849-2-berrange@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
|
|
|
9bac43 |
(cherry picked from commit 070f80095ad5b1143b50d2faffd2b1a84292e00d)
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
---
|
|
|
9bac43 |
hw/scsi/scsi-disk.c | 20 ++++++++++++++++++++
|
|
|
9bac43 |
1 file changed, 20 insertions(+)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
|
|
|
9bac43 |
index 5f1e5e8..21d6b4e 100644
|
|
|
9bac43 |
--- a/hw/scsi/scsi-disk.c
|
|
|
9bac43 |
+++ b/hw/scsi/scsi-disk.c
|
|
|
9bac43 |
@@ -104,6 +104,14 @@ typedef struct SCSIDiskState
|
|
|
9bac43 |
char *product;
|
|
|
9bac43 |
bool tray_open;
|
|
|
9bac43 |
bool tray_locked;
|
|
|
9bac43 |
+ /*
|
|
|
9bac43 |
+ * 0x0000 - rotation rate not reported
|
|
|
9bac43 |
+ * 0x0001 - non-rotating medium (SSD)
|
|
|
9bac43 |
+ * 0x0002-0x0400 - reserved
|
|
|
9bac43 |
+ * 0x0401-0xffe - rotations per minute
|
|
|
9bac43 |
+ * 0xffff - reserved
|
|
|
9bac43 |
+ */
|
|
|
9bac43 |
+ uint16_t rotation_rate;
|
|
|
9bac43 |
} SCSIDiskState;
|
|
|
9bac43 |
|
|
|
9bac43 |
static int scsi_handle_rw_error(SCSIDiskReq *r, int error, bool acct_failed);
|
|
|
9bac43 |
@@ -597,6 +605,7 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req, uint8_t *outbuf)
|
|
|
9bac43 |
outbuf[buflen++] = 0x83; // device identification
|
|
|
9bac43 |
if (s->qdev.type == TYPE_DISK) {
|
|
|
9bac43 |
outbuf[buflen++] = 0xb0; // block limits
|
|
|
9bac43 |
+ outbuf[buflen++] = 0xb1; /* block device characteristics */
|
|
|
9bac43 |
outbuf[buflen++] = 0xb2; // thin provisioning
|
|
|
9bac43 |
}
|
|
|
9bac43 |
break;
|
|
|
9bac43 |
@@ -739,6 +748,15 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req, uint8_t *outbuf)
|
|
|
9bac43 |
outbuf[43] = max_io_sectors & 0xff;
|
|
|
9bac43 |
break;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
+ case 0xb1: /* block device characteristics */
|
|
|
9bac43 |
+ {
|
|
|
9bac43 |
+ buflen = 8;
|
|
|
9bac43 |
+ outbuf[4] = (s->rotation_rate >> 8) & 0xff;
|
|
|
9bac43 |
+ outbuf[5] = s->rotation_rate & 0xff;
|
|
|
9bac43 |
+ outbuf[6] = 0;
|
|
|
9bac43 |
+ outbuf[7] = 0;
|
|
|
9bac43 |
+ break;
|
|
|
9bac43 |
+ }
|
|
|
9bac43 |
case 0xb2: /* thin provisioning */
|
|
|
9bac43 |
{
|
|
|
9bac43 |
buflen = 8;
|
|
|
9bac43 |
@@ -2903,6 +2921,7 @@ static Property scsi_hd_properties[] = {
|
|
|
9bac43 |
DEFAULT_MAX_UNMAP_SIZE),
|
|
|
9bac43 |
DEFINE_PROP_UINT64("max_io_size", SCSIDiskState, max_io_size,
|
|
|
9bac43 |
DEFAULT_MAX_IO_SIZE),
|
|
|
9bac43 |
+ DEFINE_PROP_UINT16("rotation_rate", SCSIDiskState, rotation_rate, 0),
|
|
|
9bac43 |
DEFINE_BLOCK_CHS_PROPERTIES(SCSIDiskState, qdev.conf),
|
|
|
9bac43 |
DEFINE_PROP_END_OF_LIST(),
|
|
|
9bac43 |
};
|
|
|
9bac43 |
@@ -2973,6 +2992,7 @@ static const TypeInfo scsi_cd_info = {
|
|
|
9bac43 |
#ifdef __linux__
|
|
|
9bac43 |
static Property scsi_block_properties[] = {
|
|
|
9bac43 |
DEFINE_PROP_DRIVE("drive", SCSIDiskState, qdev.conf.blk),
|
|
|
9bac43 |
+ DEFINE_PROP_UINT16("rotation_rate", SCSIDiskState, rotation_rate, 0),
|
|
|
9bac43 |
DEFINE_PROP_END_OF_LIST(),
|
|
|
9bac43 |
};
|
|
|
9bac43 |
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|