At startup lldpad walks through all network interfaces in the system and creates internal data structures for them. Some interfaces as e.g. vlan and wlan are skipped in the walkthrough, some have to be treated special (e.g. bond devices). This patch adds macvtap and macvlan interfaces to the list of devices that are skipped as we do not want to send out EVB/ECP frames on them. Signed-off-by: Jens Osterkamp <jens@xxxxxxxxxxxxxxxxxx> --- config.c | 2 + include/lldp_util.h | 1 + lldp_util.c | 90 +++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 93 insertions(+), 0 deletions(-) diff --git a/config.c b/config.c index 26afe3b..fdb63f7 100644 --- a/config.c +++ b/config.c @@ -365,6 +365,8 @@ void init_ports(void) p->if_name); } else if (is_vlan(p->if_name)) { ; + } else if (is_macvtap(p->if_name)) { + ; } else if (is_bridge(p->if_name)) { ; /* ignore bridge device */ } else if (check_link_status(p->if_name)) { diff --git a/include/lldp_util.h b/include/lldp_util.h index 3353067..ef2c562 100644 --- a/include/lldp_util.h +++ b/include/lldp_util.h @@ -40,6 +40,7 @@ int is_bridge(const char *ifname); int is_vlan(const char *ifname); int is_vlan_capable(const char *ifname); int is_wlan(const char *ifname); +int is_macvtap(const char *ifname); int is_valid_mac(const u8 *mac); int is_san_mac(u8 *addr); int is_ether(const char *ifname); diff --git a/lldp_util.c b/lldp_util.c index f39fe6b..40db741 100644 --- a/lldp_util.c +++ b/lldp_util.c @@ -34,6 +34,7 @@ #include <sys/types.h> #include <sys/socket.h> #include <net/if_arp.h> +#include <netlink/msg.h> #include <arpa/inet.h> #include <linux/if.h> #include <linux/if_bonding.h> @@ -42,6 +43,7 @@ #include <linux/wireless.h> #include <linux/sockios.h> #include <linux/ethtool.h> +#include <linux/rtnetlink.h> #include <dirent.h> #include "lldp.h" #include "lldp_util.h" @@ -57,6 +59,8 @@ int is_valid_lldp_device(const char *device_name) return 0; if (is_bridge(device_name)) return 0; + if (is_macvtap(device_name)) + return 0; return 1; } @@ -534,6 +538,92 @@ int is_wlan(const char *ifname) return rc; } +#define NLMSG_SIZE 1024 + +static struct nla_policy ifla_info_policy[IFLA_INFO_MAX + 1] = +{ + [IFLA_INFO_KIND] = { .type = NLA_STRING}, + [IFLA_INFO_DATA] = { .type = NLA_NESTED }, +}; + +int is_macvtap(const char *ifname) +{ + int ret, s; + struct nlmsghdr *nlh; + struct ifinfomsg *ifinfo; + struct nlattr *tb[IFLA_MAX+1], + *tb2[IFLA_INFO_MAX+1]; + + s = socket(PF_NETLINK, SOCK_DGRAM, NETLINK_ROUTE); + + if (s < 0) { + goto out; + } + + nlh = malloc(NLMSG_SIZE); + memset(nlh, 0, NLMSG_SIZE); + + if (!nlh) { + goto out; + } + + nlh->nlmsg_len = NLMSG_LENGTH(sizeof(struct ifinfomsg)); + nlh->nlmsg_type = RTM_GETLINK; + nlh->nlmsg_flags = NLM_F_REQUEST; + + ifinfo = NLMSG_DATA(nlh); + ifinfo->ifi_family = AF_UNSPEC; + ifinfo->ifi_index = get_ifidx(ifname); + + ret = send(s, nlh, nlh->nlmsg_len, 0); + + if (ret < 0) { + goto out_free; + } + + memset(nlh, 0, NLMSG_SIZE); + + do { + ret = recv(s, (void *) nlh, NLMSG_SIZE, MSG_DONTWAIT); + } while ((ret < 0) && errno == EINTR); + + if (nlmsg_parse(nlh, sizeof(struct ifinfomsg), + (struct nlattr **)&tb, IFLA_MAX, NULL)) { + goto out_free; + } + + if (tb[IFLA_IFNAME]) { + ifname = (char *)RTA_DATA(tb[IFLA_IFNAME]); + } else { + ifinfo = (struct ifinfomsg *)NLMSG_DATA(nlh); + } + + if (tb[IFLA_LINKINFO]) { + if (nla_parse_nested(tb2, IFLA_INFO_MAX, tb[IFLA_LINKINFO], + ifla_info_policy)) { + goto out_free; + } + + if (tb2[IFLA_INFO_KIND]) { + char *kind = (char*)(RTA_DATA(tb2[IFLA_INFO_KIND])); + if (!(strcmp("macvtap", kind) && strcmp("macvlan", kind))) { + free(nlh); + close(s); + return true; + } + } + + } else { + goto out_free; + } + +out_free: + free(nlh); +out: + close(s); + return false; +} + int is_router(const char *ifname) { int rc = 0; -- 1.7.2.3 _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/virtualization