On Fri, 2018-09-28 at 16:46 +0800, Yi Min Zhao wrote: [...] > struct _virPCIDeviceAddress { > unsigned int domain; > unsigned int bus; > unsigned int slot; > unsigned int function; > int multi; /* virTristateSwitch */ > + virZPCIDeviceAddress zpci; > }; It's kinda weird that we add the zPCI part right away but the flags which should be used to figure out whether the zPCI part should be taken into account only later, but I can live with that :) Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx> -- Andrea Bolognani / Red Hat / Virtualization -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list