Page MenuHomeFreeBSD

Add arp/ndp tests in addition to rtsock ll tests.
AbandonedPublic

Authored by melifaro on Feb 19 2021, 10:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Mar 29, 2:00 AM
Unknown Object (File)
Feb 10 2024, 9:37 PM
Unknown Object (File)
Feb 3 2024, 8:16 AM
Unknown Object (File)
Dec 23 2023, 12:18 AM
Unknown Object (File)
Nov 10 2023, 12:08 PM
Unknown Object (File)
Nov 10 2023, 11:54 AM
Unknown Object (File)
Oct 9 2023, 11:04 AM
Unknown Object (File)
Oct 9 2023, 10:50 AM

Details

Reviewers
kp
gnn
Group Reviewers
network

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37210
Build 34099: arc lint + arc unit

Event Timeline

We could potentially use pft_ping.py to actually verify that the stack sets the correct MAC address on packets (and thus also check that the insertion really succeeded), but this'll catch a number of potential issues already.

In D28812#645023, @kp wrote:

We could potentially use pft_ping.py to actually verify that the stack sets the correct MAC address on packets (and thus also check that the insertion really succeeded), but this'll catch a number of potential issues already.

IMHO there are multiple things worth testing:
(1) rtsock API - that's what we have in tests/sys/net/routing/test_rtsock_lladdr
(2) that actual arp ndp binaries work (as they have logic other than just sending a proper message - that's what these tests are targeting
(3) datapath actually uses these entries - partially covered in IPv4/IPv6 multipath tests, but we certainly need an explicit tests for these as well

(3) datapath actually uses these entries - partially covered in IPv4/IPv6 multipath tests, but we certainly need an explicit tests for these as well

I'm in the process of adding some basic Ethernet filtering capabilities to pf. I discovered the problem because one of the tests I have for that started failing. We could potentially just consider that as a test for this. Although I've only got v4 tests, because there's not much of a difference from an Ethernet perspective.

Either way, this is a step forwards in testing our code.

(If I'm nitpicking, our shell style appears to be to put the opening { on a new line, and to not have empty lines after that.)

This revision is now accepted and ready to land.Feb 19 2021, 10:50 PM

Landed in a different fashion.