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

vrrp: work around missing promiscuous netlink notifications #2340

Merged
merged 2 commits into from
Sep 17, 2023

Conversation

pqarmitage
Copy link
Collaborator

If the base interface does not implement IFF_UNICAST_FLT, for example it is a bridge interface, no netlink notification is sent by the kernel when promiscuity is set on the base interface.

The promiscuous state of the base interface is correct in the kernel but it is in incorrect in daemons that listen to the interface netlink messages (eg. DPDK).

The issue is still there in kernel 6.4.6.

Force a notification by re-setting IFLA_GROUP for the base interface.

Initial patch by and signed-off-by: Louis Scalbert [email protected]

If the base interface does not implement IFF_UNICAST_FLT, for example
it is a bridge interface, no netlink notification is sent by the kernel
when promiscuity is set on the base interface.

The promiscuous state of the base interface is correct in the kernel but
it is in incorrect in daemons that listen to the interface netlink
messages (eg. DPDK).

The issue is still there in kernel 6.4.6.

Force a notification by re-setting IFLA_GROUP for the base interface.

Initial patch by and signed-off-by: Louis Scalbert <[email protected]>

Signed-off-by: Quentin Armitage <[email protected]>
Commit edf6cd5 - "vrrp: work around missing promiscuous netlink
notifications" caused a build failure with configuration option
--disable-vmac.

Signed-off-by: Quentin Armitage <[email protected]>
@pqarmitage pqarmitage merged commit 6f8cae3 into acassen:master Sep 17, 2023
10 checks passed
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.

1 participant