Skip to content

Conversation

msune
Copy link
Member

@msune msune commented Aug 11, 2025

This patchset:

  • Hugely simplifies tc_l2_announce6.c using scapy buffers.
  • Adds test coverage for targeted NSs in tc_l2_announce6.c. Also makes untargeted NSs use IPv6 solicited node address.
  • Cleans up a bit bpf/tests/scapy/pkt_defs.py

Please note this PR builds on top #41017, so this PR can't be merged before #41017.

@msune msune requested a review from a team as a code owner August 11, 2025 14:14
@msune msune requested a review from YutaroHayakawa August 11, 2025 14:14
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 11, 2025
@msune
Copy link
Member Author

msune commented Aug 11, 2025

/test

Copy link
Member

@YutaroHayakawa YutaroHayakawa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test changes looks good to me. Thanks!

@msune msune force-pushed the bpf_tc_l2_announce6_2_scapy branch from a61647e to 8a0e20d Compare August 14, 2025 09:00
@msune
Copy link
Member Author

msune commented Aug 14, 2025

Rebased on top of main and #41017

@msune
Copy link
Member Author

msune commented Aug 14, 2025

/test

@msune msune force-pushed the bpf_tc_l2_announce6_2_scapy branch from 8a0e20d to 3ff9aeb Compare August 14, 2025 13:07
msune added 4 commits August 15, 2025 10:05
This commit adapts tc_l2_announce6.c to scapy.

Signed-off-by: Marc Suñé <marc.sune@isovalent.com>
This commit:

* Makes pkt_defs.py more readable by adjusting to max 80cols.

Signed-off-by: Marc Suñé <marc.sune@isovalent.com>
This commit:

* Refactors pkt_defs.h to use scapy utilities to calculate the
  neighbour Solicited-Node Address for the target v6 address and
  use it as IPv6 target address (as well-formed NS packets should).
* It also makes sure the LLSRC matches the SRC address of the pkt,
  which technically is not necessarily true (e.g. with proxies) but
  it's the common use-case.

Signed-off-by: Marc Suñé <marc.sune@isovalent.com>
This commit adds BPF unit test coverage for targeted IPv6 NSs on
the L2 announce IPv6 support.

Signed-off-by: Marc Suñé <marc.sune@isovalent.com>
@msune msune force-pushed the bpf_tc_l2_announce6_2_scapy branch from 3ff9aeb to 1099745 Compare August 15, 2025 08:05
@msune
Copy link
Member Author

msune commented Aug 15, 2025

/test

@msune
Copy link
Member Author

msune commented Aug 15, 2025

🟢 Ready to merge. No release note.

@gentoo-root gentoo-root added the release-note/misc This PR makes changes that have no direct user impact. label Aug 15, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Aug 15, 2025
@msune
Copy link
Member Author

msune commented Aug 19, 2025

4 days in ready-to-merge with all green lights, yet not merged. This is a source of potentially unnecessary rebases. Can someone please merge?

cc @aanm @joestringer @brb

@aanm aanm added this pull request to the merge queue Aug 19, 2025
Merged via the queue into cilium:main with commit 040a7de Aug 19, 2025
68 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants