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

Support src_mac, dst_mac and vlan id in flower filters #809

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

shuowang-123
Copy link

@shuowang-123 shuowang-123 commented Sep 3, 2022

See title, so users are able to filter flower rules based on src/dst mac address. I ran the test on an Oracle Linux 8 virtual machine and it passed:

[] go test -run "TestFilterFlowerAddDel"
PASS
ok github.com/vishvananda/netlink 0.017s

I also ran the test (without cleanup) on the same machine, was able to see the flower rule was created correctly, as reported by "tc":

[] ip netns exec netns1 tc filter show dev foo ingress
filter parent ffff: protocol all pref 1 flower chain 0
filter parent ffff: protocol all pref 1 flower chain 0 handle 0x1
dst_mac 2c:54:91:88:c9:e5
src_mac 2c:54:91:88:c9:e3
eth_type ipv4
dst_ip 1.0.0.1/24
src_ip 2.0.0.1/24
enc_dst_ip 3.0.0.1/24
enc_src_ip 4.0.0.1/24
enc_key_id 1234
enc_dst_port 8472
not_in_hw
action order 1: mirred (Egress Redirect to device bar) stolen
index 1 ref 1 bind 1

@shuowang-123
Copy link
Author

This should address second request from #589

@shuowang-123 shuowang-123 changed the title Support src_mac and dst_mac in flower filters Support src_mac, dst_mac and vlan id in flower filters Sep 8, 2022
@aboch
Copy link
Collaborator

aboch commented Aug 23, 2024

@shuowang-123

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