Re: [PATCH] qemu: hotplug: ensure address generation for vfio-ccw

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ján Tomko <jtomko@xxxxxxxxxx> [2019-11-12, 10:47AM +0100]:
> > diff --git a/src/qemu/qemu_hotplug.c b/src/qemu/qemu_hotplug.c
> > index fd4bafef..7b775159 100644
> > --- a/src/qemu/qemu_hotplug.c
> > +++ b/src/qemu/qemu_hotplug.c
> > @@ -2790,6 +2790,8 @@ qemuDomainAttachMediatedDevice(virQEMUDriverPtr driver,
> > {
> >     int ret = -1;
> >     g_autofree char *devstr = NULL;
> > +    char *devName = NULL;
> > +    bool releaseaddr = false;
> >     bool added = false;
> >     bool teardowncgroup = false;
> >     bool teardownlabel = false;
> > @@ -2805,6 +2807,10 @@ qemuDomainAttachMediatedDevice(virQEMUDriverPtr driver,
> >             return -1;
> >         break;
> >     case VIR_MDEV_MODEL_TYPE_VFIO_CCW:
> > +        devName = hostdev->source.subsys.u.mdev.uuidstr;
> > +        if (qemuDomainEnsureVirtioAddress(&releaseaddr, vm, &dev, devName) < 0)
> 
> vfio-ccw is not really a virtio device, is it?

The naming of that function is rather odd and misleading. It sets the
address type of the device if not set and generates the correct address,
exactly what we need here.

Somewhere on my TODO-list I have a point to go over the address
generation for CCW and clean it up. For now, this must suffice.

-- 
IBM Systems
Linux on Z & Virtualization Development
--------------------------------------------------
IBM Deutschland Research & Development GmbH
Schönaicher Str. 220, 71032 Böblingen
Phone: +49 7031 16 1819
--------------------------------------------------
Vorsitzende des Aufsichtsrats: Matthias Hartmann
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux