Re: [Qemu-devel] [PATCH 11/35] scsi-disk: support READ DVD STRUCTURE

[Paolo Bonzini <pbonzini@redhat.com>]

On 10/21/2011 01:42 PM, Kevin Wolf wrote:
>> >  +    if (s->qdev.type != TYPE_ROM || !bdrv_is_inserted(s->bs)) {
>> >  +        return -1;
>> >  +    }
>> >  +    if (s->tray_open || !bdrv_is_inserted(s->bs)) {
>> >  +        scsi_check_condition(r, SENSE_CODE(NO_MEDIUM));
>> >  +        return -1;
>> >  +    }
> You are checking twice for bdrv_is_inserted, which one do you really mean?

The first is bogus.

> Also, format = 0xff should work even without a medium.

Will move the tray_open/bdrv_is_inserted/media_is_cd tests within the 
"if (format != 0xff)".

>> >  +    if (media_is_cd(s)) {
>> >  +        scsi_check_condition(r, SENSE_CODE(INCOMPATIBLE_FORMAT));
>> >  +        return -1;
>> >  +    }
>> >  +    if (media != 0) {
>> >  +        scsi_check_condition(r, SENSE_CODE(INCOMPATIBLE_FORMAT));
>> >  +        return -1;
>> >  +    }

media != 0 should return INVALID_FIELD too.

Paolo