Re: [PATCH v6 09/13] conf: Allocate/release 'uid' and 'fid' in PCI address

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

 



On Fri, 2018-09-28 at 16:46 +0800, Yi Min Zhao wrote:
>  # conf/device_conf.h
> +virDeviceInfoPCIAddressExtensionIsPresent;
> +virDeviceInfoPCIAddressExtensionIsWanted;
>  virDeviceInfoPCIAddressIsPresent;
>  virDeviceInfoPCIAddressIsWanted;
>  virDomainDeviceAddressIsValid;
> @@ -119,6 +121,9 @@ virDomainCCWAddressSetFree;
>  virDomainPCIAddressBusIsFullyReserved;
>  virDomainPCIAddressBusSetModel;
>  virDomainPCIAddressEnsureAddr;
> +virDomainPCIAddressExtensionReleaseAddr;
> +virDomainPCIAddressExtensionReserveAddr;
> +virDomainPCIAddressExtensionReserveNextAddr;

I'm not quite quire we need to export these functions.

With the recent changes, we've gotten to the point that we're not
even passing a virZPCIDeviceAddress to them, but rather they have
the very same signature as the regular virPCIDeviceAddress...

So it should be possible to just make them static and only call
them from the virDomainPCIAddressReserveAddr() and friends, right?
Which is where I was hoping we could eventually get. Or did I
miss something?


Everything else looks good.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
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