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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html