[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] ide scsi: Mess with geometry only for hard disk
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] ide scsi: Mess with geometry only for hard disk devices |
Date: |
Mon, 23 Jul 2012 12:03:55 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120615 Thunderbird/13.0.1 |
Il 23/07/2012 09:25, Markus Armbruster ha scritto:
> Ping?
>
> Markus Armbruster <address@hidden> writes:
>
>> Legacy -drive cyls=... are now ignored completely when the drive
>> doesn't back a hard disk device. Before, they were first checked
>> against a hard disk's limits, then ignored.
>>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> hw/ide/qdev.c | 3 ++-
>> hw/scsi-disk.c | 3 ++-
>> 2 files changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/hw/ide/qdev.c b/hw/ide/qdev.c
>> index 22e58df..5ea9b8f 100644
>> --- a/hw/ide/qdev.c
>> +++ b/hw/ide/qdev.c
>> @@ -149,7 +149,8 @@ static int ide_dev_initfn(IDEDevice *dev, IDEDriveKind
>> kind)
>> }
>>
>> blkconf_serial(&dev->conf, &dev->serial);
>> - if (blkconf_geometry(&dev->conf, &dev->chs_trans, 65536, 16, 255) < 0) {
>> + if (kind != IDE_CD
>> + && blkconf_geometry(&dev->conf, &dev->chs_trans, 65536, 16, 255) <
>> 0) {
>> return -1;
>> }
>>
>> diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c
>> index 525816c..318318c 100644
>> --- a/hw/scsi-disk.c
>> +++ b/hw/scsi-disk.c
>> @@ -1750,7 +1750,8 @@ static int scsi_initfn(SCSIDevice *dev)
>> }
>>
>> blkconf_serial(&s->qdev.conf, &s->serial);
>> - if (blkconf_geometry(&dev->conf, NULL, 65535, 255, 255) < 0) {
>> + if (dev->type == TYPE_DISK
>> + && blkconf_geometry(&dev->conf, NULL, 65535, 255, 255) < 0) {
>> return -1;
>> }
>
>
Acked-by: Paolo Bonzini <address@hidden>
Kevin, are you taking this patch?
Paolo