Page MenuHomeFreeBSD

bridge tests: Test for #216510
ClosedPublic

Authored by kp on Apr 1 2020, 2:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 29, 1:52 PM
Unknown Object (File)
Feb 11 2024, 9:12 PM
Unknown Object (File)
Dec 22 2023, 10:26 PM
Unknown Object (File)
Dec 10 2023, 6:46 PM
Unknown Object (File)
Nov 7 2023, 1:32 PM
Unknown Object (File)
Nov 5 2023, 3:22 AM
Unknown Object (File)
Oct 10 2023, 7:22 PM
Unknown Object (File)
Oct 6 2023, 12:11 PM
Subscribers

Details

Summary

We used to have an issue with recursive locking with
net.link.bridge.inherit_mac. 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

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

LGTM

tests/sys/net/if_bridge_test.sh
251 ↗(On Diff #70092)

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.

tests/sys/net/if_bridge_test.sh
251 ↗(On Diff #70092)

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.