Skip to content

Commit

Permalink
network: Rename primary to master
Browse files Browse the repository at this point in the history
The previous change made things confusing by impliying there may be a
secondary when VLAN/IPVLAN/bridge members can only have a single parent
device.

Signed-off-by: Stéphane Graber <stgraber@ubuntu.com>
  • Loading branch information
stgraber committed Jun 18, 2020
1 parent 3d88831 commit 0dc9a14
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 12 deletions.
2 changes: 1 addition & 1 deletion src/lxc/macro.h
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@ extern int __build_bug_on_failed;
#endif

#ifndef BRIDGE_FLAGS_MASTER
#define BRIDGE_FLAGS_MASTER 1 /* Bridge command to/from primary */
#define BRIDGE_FLAGS_MASTER 1 /* Bridge command to/from parent */
#endif

#ifndef BRIDGE_VLAN_INFO_PVID
Expand Down
18 changes: 9 additions & 9 deletions src/lxc/network.c
Original file line number Diff line number Diff line change
Expand Up @@ -847,7 +847,7 @@ static int instantiate_macvlan(struct lxc_handler *handler, struct lxc_netdev *n
return -1;
}

static int lxc_ipvlan_create(const char *primary, const char *name, int mode, int isolation)
static int lxc_ipvlan_create(const char *parent, const char *name, int mode, int isolation)
{
call_cleaner(nlmsg_free) struct nlmsg *answer = NULL, *nlmsg = NULL;
struct nl_handler nlh;
Expand All @@ -856,15 +856,15 @@ static int lxc_ipvlan_create(const char *primary, const char *name, int mode, in
struct ifinfomsg *ifi;
struct rtattr *nest, *nest2;

len = strlen(primary);
len = strlen(parent);
if (len == 1 || len >= IFNAMSIZ)
return ret_errno(EINVAL);

len = strlen(name);
if (len == 1 || len >= IFNAMSIZ)
return ret_errno(EINVAL);

index = if_nametoindex(primary);
index = if_nametoindex(parent);
if (!index)
return ret_errno(EINVAL);

Expand Down Expand Up @@ -2052,7 +2052,7 @@ int lxc_veth_create(const char *name1, const char *name2, pid_t pid, unsigned in
}

/* TODO: merge with lxc_macvlan_create */
int lxc_vlan_create(const char *primary, const char *name, unsigned short vlanid)
int lxc_vlan_create(const char *parent, const char *name, unsigned short vlanid)
{
call_cleaner(nlmsg_free) struct nlmsg *answer = NULL, *nlmsg = NULL;
struct nl_handler nlh;
Expand All @@ -2065,7 +2065,7 @@ int lxc_vlan_create(const char *primary, const char *name, unsigned short vlanid
if (err)
return err;

len = strlen(primary);
len = strlen(parent);
if (len == 1 || len >= IFNAMSIZ)
return ret_errno(EINVAL);

Expand All @@ -2081,7 +2081,7 @@ int lxc_vlan_create(const char *primary, const char *name, unsigned short vlanid
if (!answer)
return ret_errno(ENOMEM);

lindex = if_nametoindex(primary);
lindex = if_nametoindex(parent);
if (!lindex)
return ret_errno(EINVAL);

Expand Down Expand Up @@ -2120,7 +2120,7 @@ int lxc_vlan_create(const char *primary, const char *name, unsigned short vlanid
return netlink_transaction(nlh_ptr, nlmsg, answer);
}

int lxc_macvlan_create(const char *primary, const char *name, int mode)
int lxc_macvlan_create(const char *parent, const char *name, int mode)
{
call_cleaner(nlmsg_free) struct nlmsg *answer = NULL, *nlmsg = NULL;
struct nl_handler nlh;
Expand All @@ -2133,7 +2133,7 @@ int lxc_macvlan_create(const char *primary, const char *name, int mode)
if (err)
return err;

len = strlen(primary);
len = strlen(parent);
if (len == 1 || len >= IFNAMSIZ)
return ret_errno(EINVAL);

Expand All @@ -2149,7 +2149,7 @@ int lxc_macvlan_create(const char *primary, const char *name, int mode)
if (!answer)
return ret_errno(ENOMEM);

index = if_nametoindex(primary);
index = if_nametoindex(parent);
if (!index)
return ret_errno(EINVAL);

Expand Down
4 changes: 2 additions & 2 deletions src/lxc/network.h
Original file line number Diff line number Diff line change
Expand Up @@ -208,8 +208,8 @@ extern int lxc_netdev_set_mtu(const char *name, int mtu);
/* Create a virtual network devices. */
extern int lxc_veth_create(const char *name1, const char *name2, pid_t pid,
unsigned int mtu);
extern int lxc_macvlan_create(const char *primary, const char *name, int mode);
extern int lxc_vlan_create(const char *primary, const char *name,
extern int lxc_macvlan_create(const char *parent, const char *name, int mode);
extern int lxc_vlan_create(const char *parent, const char *name,
unsigned short vid);

/* Set ip address. */
Expand Down

0 comments on commit 0dc9a14

Please sign in to comment.