On 2020-12-08 at 09:29, Laine Stump wrote:
On 12/4/20 2:01 AM, Shi Lei wrote:
> Signed-off-by: Shi Lei <shi_lei(a)massclouds.com>
> ---
> src/util/virnetdev.c | 2 ++
> src/util/virnetdev.h | 3 +++
> 2 files changed, 5 insertions(+)
>
> diff --git a/src/util/virnetdev.c b/src/util/virnetdev.c
> index 5ff8e35f..ff1b1fa0 100644
> --- a/src/util/virnetdev.c
> +++ b/src/util/virnetdev.c
> @@ -102,6 +102,7 @@ VIR_ENUM_IMPL(virNetDevGenNameType,
> "tap",
> "macvtap",
> "macvlan",
> + "veth",
As discussed in patch 1, this ENUM_IMPL is unnecessary.
Okay.
> );
>
> static virNetDevGenName
> @@ -110,6 +111,7 @@ virNetDevGenNames[VIR_NET_DEV_GEN_NAME_LAST] = {
> {-1, VIR_NET_GENERATED_TAP_PREFIX, VIR_MUTEX_INITIALIZER},
> {-1, VIR_NET_GENERATED_MACVTAP_PREFIX, VIR_MUTEX_INITIALIZER},
> {-1, VIR_NET_GENERATED_MACVLAN_PREFIX, VIR_MUTEX_INITIALIZER},
> + {-1, VIR_NET_GENERATED_VETH_PREFIX, VIR_MUTEX_INITIALIZER},
> };
>
> typedef enum {
> diff --git a/src/util/virnetdev.h b/src/util/virnetdev.h
> index 19f37b61..097d0f8e 100644
> --- a/src/util/virnetdev.h
> +++ b/src/util/virnetdev.h
> @@ -40,6 +40,8 @@ typedef void virIfreq;
> */
> #define VIR_NET_GENERATED_TAP_PREFIX "vnet"
>
> +#define VIR_NET_GENERATED_VETH_PREFIX "veth"
Up until now, libvirt has named the veth devices as "vnetN", not
"vethN". I don't know that it would cause any problem to change to using
"vethN" (since already-running domains would have their "vnetN"
device
name available in the domain status - only newly started domains would
use "vethN"). However I don't see any concrete reason for making that
change.
That being the case, I think you can just drop this patch, and use the
TAP prefix in the next patch (actually that's a good indication that it
shouldn't be called VIR_NET_DEV_GEN_NAME_TAP, but should instead be
called VIR_NET_DEV_GEN_NAME_VNET).
Okay.
> +
> /* libvirt will start macvtap/macvlan interface names with one of
> * these prefixes when it auto-generates the name
> */
> @@ -156,6 +158,7 @@ typedef enum {
> VIR_NET_DEV_GEN_NAME_TAP,
> VIR_NET_DEV_GEN_NAME_MACVTAP,
> VIR_NET_DEV_GEN_NAME_MACVLAN,
> + VIR_NET_DEV_GEN_NAME_VETH,
> VIR_NET_DEV_GEN_NAME_LAST
> } virNetDevGenNameType;
>