[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: |
Alex Williamson |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] vfio/ccw: fix initialization of the Object DeviceState pointer in the common base-device |
Date: |
Mon, 17 Jul 2017 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.
Reviewed-by: Alex Williamson <address@hidden>
> 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",