Igalia / snabb

Snabb Switch: Fast open source packet processing
Apache License 2.0
47 stars 5 forks source link

Fix hardware ping test #1163

Closed dpino closed 5 years ago

dpino commented 5 years ago

Fixes #1162.

There's a test script that currently is not run being run by CI* to check if ping resolution to lwAFTR's internal and external interfaces work. I realized the script stopped working for internal interfaces. After lots of research I realized the issue was that the configuration file required that both interfaces had a different MAC address. This condition was guarded by an assert, but the assert didn't work well :)

Applying this fix, ping resolution using a physical NIC works:

$ sudo SNABB_PCI0=0000:82:00.0 SNABB_PCI1=0000:02:00.0 ./program/lwaftr/tests/hw/test_ping_on_a_stick.sh                                    
Success: Ping to internal interface         
Success: Ping to external interface