Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Extend bridge info support #406

Merged
merged 4 commits into from
Sep 19, 2024

Conversation

ronand-atl
Copy link
Contributor

This PR adds a few more supported attributes in route/link/bridge_info.

Support for the following bridge attributes is added:

  • IFLA_BR_STP_STATE
  • IFLA_BR_MCAST_ROUTER
  • IFLA_BR_MCAST_SNOOPING

Support for IFLA_BR_MULTI_BOOLOPT is added, so the following boolopts can be set:

  • BR_BOOLOPT_NO_LL_LEARN
  • BR_BOOLOPT_MCAST_VLAN_SNOOPING
  • BR_BOOLOPT_MST_ENABLE

Additionally, I've added a few missing getter functions for already supported attributes.

libnl-route-3.sym Show resolved Hide resolved
include/netlink/route/link/bridge_info.h Outdated Show resolved Hide resolved
include/netlink/route/link/bridge_info.h Outdated Show resolved Hide resolved
lib/route/link/bridge_info.c Outdated Show resolved Hide resolved
lib/route/link/bridge_info.c Outdated Show resolved Hide resolved
A corresponding entry for rtnl_link_bridge_set_vlan_default_pvid is
missing from the libnl-route-3.sym file, so add it.
There are bridge info setter functions for nf_call_iptables,
nf_call_ip6tables and nf_call_arptables, but no corresponding getter
functions, so add them.
Add support for the following attributes for bridge links:

IFLA_BR_STP_STATE
IFLA_BR_MCAST_ROUTER
IFLA_BR_MCAST_SNOOPING
@ronand-atl ronand-atl force-pushed the extend-bridge-info-support branch from 20b2d79 to 220c725 Compare September 18, 2024 06:01
Add support for the IFLA_BR_MULTI_BOOLOPT bridge link attribute.
@ronand-atl ronand-atl force-pushed the extend-bridge-info-support branch from 220c725 to 8a73b24 Compare September 18, 2024 22:12
@thom311 thom311 merged commit 83f29cf into thom311:main Sep 19, 2024
4 checks passed
@thom311
Copy link
Owner

thom311 commented Sep 19, 2024

merged.

Very nice, thank you!

@ronand-atl ronand-atl deleted the extend-bridge-info-support branch September 19, 2024 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants