[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] vfio/ccw: fix initialization of the Object
From: |
Dong Jia Shi |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] vfio/ccw: fix initialization of the Object DeviceState pointer in the common base-device |
Date: |
Tue, 18 Jul 2017 10:42:41 +0800 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
* Alex Williamson <address@hidden> [2017-07-17 20:26:53 -0600]:
> On Tue, 18 Jul 2017 03:49:26 +0200
> Dong Jia Shi <address@hidden> wrote:
>
> > Commit 7da624e2 ("vfio: Test realized when using VFIOGroup.device_list
> > iterator") introduced a pointer to the Object DeviceState in the VFIO
> > common base-device and skiped non-realized devices as we iterate
> > VFIOGroup.device_list. While it missed to initialize the pointer for
> > the vfio-ccw case. Let's fix it.
> >
> > Fixes: 7da624e2 ("vfio: Test realized when using VFIOGroup.device_list
> > iterator")
>
> Sorry for that.
No problem.
>
> Reviewed-by: Alex Williamson <address@hidden>
Thanks! :D
>
> > Cc: Alex Williamson <address@hidden>
> > Reviewed-by: Halil Pasic <address@hidden>
> > Signed-off-by: Dong Jia Shi <address@hidden>
> > ---
> > hw/vfio/ccw.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > diff --git a/hw/vfio/ccw.c b/hw/vfio/ccw.c
> > index 8d97b53e77..a8baadf57a 100644
> > --- a/hw/vfio/ccw.c
> > +++ b/hw/vfio/ccw.c
> > @@ -338,6 +338,7 @@ static void vfio_ccw_realize(DeviceState *dev, Error
> > **errp)
> > vcdev->vdev.type = VFIO_DEVICE_TYPE_CCW;
> > vcdev->vdev.name = g_strdup_printf("%x.%x.%04x", cdev->hostid.cssid,
> > cdev->hostid.ssid,
> > cdev->hostid.devid);
> > + vcdev->vdev.dev = dev;
> > QLIST_FOREACH(vbasedev, &group->device_list, next) {
> > if (strcmp(vbasedev->name, vcdev->vdev.name) == 0) {
> > error_setg(&err, "vfio: subchannel %s has already been
> > attached",
>
--
Dong Jia Shi