[libvirt PATCH] esx: implement connectListAllNetworks
Pino Toscano
ptoscano at redhat.com
Mon Sep 14 16:27:46 UTC 2020
On Monday, 14 September 2020 17:16:50 CEST Michal Privoznik wrote:
> On 9/14/20 11:10 AM, Pino Toscano wrote:
> > Implement the .connectListAllNetworks networks API in the esx network
> > driver.
> >
> > Signed-off-by: Pino Toscano <ptoscano at redhat.com>
> > ---
> > src/esx/esx_network_driver.c | 64 ++++++++++++++++++++++++++++++++++++
> > 1 file changed, 64 insertions(+)
> >
> > diff --git a/src/esx/esx_network_driver.c b/src/esx/esx_network_driver.c
> > index 88843aae2f..5d9c1e3c2c 100644
> > --- a/src/esx/esx_network_driver.c
> > +++ b/src/esx/esx_network_driver.c
> > @@ -863,6 +863,69 @@ esxNetworkIsPersistent(virNetworkPtr network G_GNUC_UNUSED)
> >
> >
> >
> > +#define MATCH(FLAG) (flags & (FLAG))
> > +static int
> > +esxConnectListAllNetworks(virConnectPtr conn,
> > + virNetworkPtr **nets,
> > + unsigned int flags)
> > +{
> > + int ret = -1;
> > + esxPrivate *priv = conn->privateData;
> > + esxVI_HostVirtualSwitch *hostVirtualSwitchList = NULL;
> > + esxVI_HostVirtualSwitch *hostVirtualSwitch = NULL;
> > + size_t count = 0;
> > + size_t i;
> > +
> > + virCheckFlags(VIR_CONNECT_LIST_NETWORKS_FILTERS_ALL, -1);
> > +
> > + /*
> > + * ESX networks are always active, persistent, and
> > + * autostarted, so return zero elements in case we are asked
> > + * for networks different than that.
> > + */
> > + if (MATCH(VIR_CONNECT_LIST_NETWORKS_FILTERS_ACTIVE) &&
> > + !(MATCH(VIR_CONNECT_LIST_NETWORKS_ACTIVE)))
> > + return 0;
> > + if (MATCH(VIR_CONNECT_LIST_NETWORKS_FILTERS_PERSISTENT) &&
> > + !(MATCH(VIR_CONNECT_LIST_NETWORKS_PERSISTENT)))
> > + return 0;
> > + if (MATCH(VIR_CONNECT_LIST_NETWORKS_FILTERS_AUTOSTART) &&
> > + !(MATCH(VIR_CONNECT_LIST_NETWORKS_AUTOSTART)))
> > + return 0;
> > +
> > + if (esxVI_EnsureSession(priv->primary) < 0 ||
> > + esxVI_LookupHostVirtualSwitchList(priv->primary,
> > + &hostVirtualSwitchList) < 0) {
> > + return -1;
> > + }
> > +
> > + for (hostVirtualSwitch = hostVirtualSwitchList; hostVirtualSwitch;
> > + hostVirtualSwitch = hostVirtualSwitch->_next) {
> > + virNetworkPtr net = virtualswitchToNetwork(conn, hostVirtualSwitch);
> > +
> > + if (VIR_APPEND_ELEMENT(*nets, count, net) < 0)
>
> The virConnectListAllNetworks() documents that @nets can be NULL if the
> caller is interested only in the count of networks.
Ah, I missed that bit, thanks. I will send v2 with a tweaked version.
Thanks,
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20200914/f22088bd/attachment-0001.sig>
More information about the libvir-list
mailing list