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

[copp] Add CoPP trap for DHCP L2 broadcasts #2987

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lukasstockner
Copy link

What I did
This PR adds support for the DHCP_L2 trap types.

Why I did it
It appears that these traps are needed on Innovium SAI in order for DHCP broadcast packets to be trapped.
Out of the box, the DHCP Relay doesn't do anything (since DHCP broadcasts don't appear on the CPU), but with these extra traps applied it works.

How I verified it

  • Set up test switch using Innovium SAI (e.g. Supermicro T7132S)
  • Run tcpdump on switch to watch for DHCP packets
  • Run DHCP client on a connected server
  • Observe that no packets are received
  • Apply patch, enable L2 traps
  • Re-run tcpdump and DHCP client
  • Observe that packets are received and relayed, and that the client receives an answer

Details if related

Signed-off-by: Lukas Stockner <lstockner@genesiscloud.com>
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