|
From: | Hannes Reinecke |
Subject: | Re: [Qemu-devel] [PATCH 5/6] scsi-disk: Remove 'drive_kind' |
Date: | Tue, 26 Jul 2011 08:21:06 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.14) Gecko/20110221 SUSE/3.1.8 Thunderbird/3.1.8 |
On 07/25/2011 05:59 PM, Markus Armbruster wrote:
Hannes Reinecke<address@hidden> writes:Instead of using its own definitions scsi-disk should be using the device type of the parent device. Signed-off-by: Hannes Reinecke<address@hidden> --- hw/scsi-defs.h | 6 +++++- hw/scsi-disk.c | 48 ++++++++++++++++++++++++------------------------ 2 files changed, 29 insertions(+), 25 deletions(-)
[ .. ]
@@ -1217,44 +1214,47 @@ static int scsi_initfn(SCSIDevice *dev, SCSIDriveKind kind) return -1; } - if (kind == SCSI_CD) { + if (scsi_type == TYPE_ROM) { s->qdev.blocksize = 2048; - } else { + } else if (scsi_type == TYPE_DISK) { s->qdev.blocksize = s->qdev.conf.logical_block_size; + } else { + error_report("scsi-disk: Unhandled SCSI type %02x", scsi_type); + return -1; } s->cluster_size = s->qdev.blocksize / 512; s->bs->buffer_alignment = s->qdev.blocksize; - s->qdev.type = TYPE_DISK; + s->qdev.type = scsi_type;Is this a bug fix?
No, proper initialisation.s->qdev.type is the SCSI type we're told to emulate. So we have to set it to the correct value otherwise the emulation will return wrong values.
qemu_add_vm_change_state_handler(scsi_dma_restart_cb, s); - bdrv_set_removable(s->bs, kind == SCSI_CD); + bdrv_set_removable(s->bs, scsi_type == TYPE_ROM); add_boot_device_path(s->qdev.conf.bootindex,&dev->qdev, ",0"); return 0; } static int scsi_hd_initfn(SCSIDevice *dev) { - return scsi_initfn(dev, SCSI_HD); + return scsi_initfn(dev, TYPE_DISK); } static int scsi_cd_initfn(SCSIDevice *dev) { - return scsi_initfn(dev, SCSI_CD); + return scsi_initfn(dev, TYPE_ROM); } static int scsi_disk_initfn(SCSIDevice *dev) { - SCSIDriveKind kind; DriveInfo *dinfo; + uint8_t scsi_type = TYPE_DISK; - if (!dev->conf.bs) { - kind = SCSI_HD; /* will die in scsi_initfn() */The comment explains why we don't explicitly fail when !dev->conf.bs, like all the other block device models. I'd rather keep it.
Ah. The magic of block devices. By all means, keep it. Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage address@hidden +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
[Prev in Thread] | Current Thread | [Next in Thread] |