Page MenuHomeFreeBSD

gmultipath: add ATF tests
ClosedPublic

Authored by asomers on Nov 4 2019, 6:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 11 2024, 10:44 PM
Unknown Object (File)
Dec 22 2023, 10:58 PM
Unknown Object (File)
Sep 11 2023, 4:58 AM
Unknown Object (File)
Aug 31 2023, 12:30 AM
Unknown Object (File)
Aug 31 2023, 12:27 AM
Unknown Object (File)
Aug 31 2023, 12:26 AM
Unknown Object (File)
Aug 30 2023, 3:22 PM
Unknown Object (File)
Aug 27 2023, 2:32 PM

Details

Summary

gmultipath: add ATF tests

Add ATF tests for most gmultipath operations. Add some dtrace probes too,
primarily for configuration changes that happen in response to provider
errors.

PR: 178473

Diff Detail

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

Event Timeline

asomers added inline comments.
tests/sys/geom/class/multipath/misc.sh
202

Turns out this test case is invalid. In fact, AFAICT the entire bug was misdiagnosed. gmultipath *does* limit the number of retries at the end of g_multipath_done_error. But my test case *appears* to show a long loop because when dd releases the multipath device, every other geom class gets a chance to taste. You can easily see the correct behavior by setting sysctl kern.geom.notaste=1 (after creating the multipath device). I need to do some more work to determine whether there really is any bug at all.

Fix the failloop test.

There never was a bug to begin with; it only appeared that way due to GEOM
tasting. Now tasting is disabled during the failloop test. That requires
failloop to run exclusively, so it doesn't screw up other tests by disabling
tasting.

Also add a new "restart" dtrace probe. It helps to confirm the nonexistence
of the reported bug.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 6 2019, 12:12 AM
Closed by commit rS355431: gmultipath: add ATF tests (authored by asomers). · Explain Why
This revision was automatically updated to reflect the committed changes.