[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] Chage disk serial length from 20 to 36
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] Chage disk serial length from 20 to 36 |
Date: |
Tue, 30 Aug 2016 14:08:55 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> On 29/08/2016 09:52, address@hidden wrote:
>> From: ronyweng <address@hidden>
>>
>> Openstack Cinder assigns volume a 36 characters uuid as serial.
>> Qemu will shrinks the uuid to 20 characters,
>> missmatch with original uuid.
>>
>> Signed-off-by: Rony Weng <address@hidden>
>> ---
>> hw/scsi/scsi-disk.c | 4 ++--
>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
>> index 836a155..0e1bb68 100644
>> --- a/hw/scsi/scsi-disk.c
>> +++ b/hw/scsi/scsi-disk.c
>> @@ -599,8 +599,8 @@ static int scsi_disk_emulate_inquiry(SCSIRequest *req,
>> uint8_t *outbuf)
>> }
>>
>> l = strlen(s->serial);
>> - if (l > 20) {
>> - l = 20;
>> + if (l > 36) {
>> + l = 36;
>> }
>>
>> DPRINTF("Inquiry EVPD[Serial number] "
>>
>
> Cc: address@hidden
>
> Thanks, queued for 2.8 and 2.7.1
>
> Paolo
Pointing to the applicable SCSI spec chapter & verse in the commit
message would be nice. As is, 36 looks as arbitrary as 20.