On 6/28/21 11:18 AM, zhangjl02 wrote:
> From: zhangjl02 <zhangjl02@inspur.com>
>
> Tell whether a port definition is an ovs managed virtual port
> ---
> src/qemu/qemu_domain.c | 13 +++++++++++++
> src/qemu/qemu_domain.h | 3 +++
> 2 files changed, 16 insertions(+)
>
> diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
> index fc60e15eea..da5a226fc2 100644
> --- a/src/qemu/qemu_domain.c
> +++ b/src/qemu/qemu_domain.c
> @@ -11575,3 +11575,16 @@ qemuDomainNamePathsCleanup(virQEMUDriverConfig *cfg,
>
> return 0;
> }
> +
> +/*
> + * Check whether the port is an ovs managed port
> + */
> +bool qemuDomainDefIsOvsport(virDomainNetDef *net,
> + int actualType) {
> + const virNetDevVPortProfile *vport = virDomainNetGetActualVirtPortProfile(net);
> + if ((actualType == VIR_DOMAIN_NET_TYPE_BRIDGE) && vport &&
> + vport->virtPortType == VIR_NETDEV_VPORT_PROFILE_OPENVSWITCH) {
> + return true;
> + }
> + return false;
> +}
> \ No newline at end of file
> diff --git a/src/qemu/qemu_domain.h b/src/qemu/qemu_domain.h
> index acf6ca5ab6..81a9bf0cfb 100644
> --- a/src/qemu/qemu_domain.h
> +++ b/src/qemu/qemu_domain.h
> @@ -1063,3 +1063,6 @@ int
> qemuDomainNamePathsCleanup(virQEMUDriverConfig *cfg,
> const char *name,
> bool bestEffort);
> +
> +bool qemuDomainDefIsOvsport(virDomainNetDef *net,
> + int actualType);
> \ No newline at end of file
>
There's nothing QEMU specific in this function. I think it can go into
src/conf/domain_conf.c (somewhere near
virDomainNetGetActualVirtPortProfile() perhaps) and that check from the
function - we have a few places like it, those can be replaced with the
function call.
Michal