Re: [Qemu-devel] [PATCH] scsi: Report error when lun number is in use

[Fam Zheng <famz@redhat.com>]

On Tue, 06/10 10:18, Paolo Bonzini wrote:
> Il 10/06/2014 08:55, Fam Zheng ha scritto:
> >In the case that the lun number is taken by another scsi device, don't
> >release the existing device siliently, but report an error to user.
> >
> >Signed-off-by: Fam Zheng <famz@redhat.com>
> >---
> > hw/scsi/scsi-bus.c | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> >diff --git a/hw/scsi/scsi-bus.c b/hw/scsi/scsi-bus.c
> >index 06399fa..d219da3 100644
> >--- a/hw/scsi/scsi-bus.c
> >+++ b/hw/scsi/scsi-bus.c
> >@@ -177,7 +177,8 @@ static int scsi_qdev_init(DeviceState *qdev)
> >         d = scsi_device_find(bus, dev->channel, dev->id, dev->lun);
> >         assert(d);
> >         if (d->lun == dev->lun && dev != d) {
> >-            object_unparent(OBJECT(d));
> >+            error_report("lun already used by '%s'", d->qdev.id);
> >+            goto err;
> >         }
> >     }
> >
> >
> 
> Thanks, applying.  Would you look into converting the SCSI bus to realize?

OK, will look into it!

Thanks,
Fam