Page MenuHomeFreeBSD

bridge tests: Test for #216510

Authored by kp on Apr 1 2020, 2:26 PM.



We used to have an issue with recursive locking with This causes us to send an ARP request while
we hold the BRIDGE_LOCK, which used to cause us to acquire the
BRIDGE_LOCK again. We can't re-acquire it, so this caused a panic.

Now that we no longer need to acquire the BRIDGE_LOCK for
bridge_transmit() this should no longer panic. Test this.

PR: 216510
Sponsored by: The FreeBSD Foundation

Diff Detail

Lint WarningsExcuse: y
Warningtests/sys/net/ Executable
No Unit Test Coverage
Build Status
Buildable 30230
Build 28015: arc lint + arc unit

Event Timeline



Is this a standard way we reference PRs in tests? I'd probably expect PR216510

This revision is now accepted and ready to land.Apr 1 2020, 2:42 PM

Thanks for the review. This can't be committed until the lock rework of the bridge is pushed, because without it this will trigger a recursive lock of BRIDGE_LOCK and panic.


I don't know if we have a standard way of doing this. There are a couple more that use #12345, but those are mostly mine.

There's also a few 'PR 123456' and at least one 'PR kern/123456'.

This looks good. Thank you for adding more tests!

This revision was landed with ongoing or failed builds.Apr 26 2020, 4:27 PM
Closed by commit rS360346: bridge tests: Test for #216510 (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.