Il 17/07/2012 13:14, Kevin Wolf ha scritto:
>> > - if (s->features & (1 << SCSI_DISK_F_REMOVABLE)) {
>> > - bdrv_set_dev_ops(s->qdev.conf.bs, &scsi_cd_block_ops, s);
>> > - }
>> > + bdrv_set_dev_ops(s->qdev.conf.bs, &scsi_disk_block_ops, s);
> Are you aware of this code?
>
> bool bdrv_dev_has_removable_media(BlockDriverState *bs)
> {
> return !bs->dev || (bs->dev_ops && bs->dev_ops->change_media_cb);
> }
>
> This means that now all SCSI disks have removable media from the
> monitor's point of view.
I remembered there was something similar but I couldn't find it. I'll
rework the patch to have two separate sets of dev_ops.
Thanks very much!
Paolo
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
