Page MenuHomeFreeBSD

gmultipath: add ATF tests
ClosedPublic

Authored by asomers on Nov 4 2019, 6:01 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 3:30 PM
Unknown Object (File)
Oct 21 2024, 5:47 PM
Unknown Object (File)
Oct 19 2024, 8:18 AM
Unknown Object (File)
Oct 19 2024, 2:14 AM
Unknown Object (File)
Oct 19 2024, 2:13 AM
Unknown Object (File)
Oct 19 2024, 2:13 AM
Unknown Object (File)
Oct 19 2024, 2:13 AM
Unknown Object (File)
Oct 19 2024, 1:54 AM

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 Not Applicable
Unit
Tests Not Applicable

Event Timeline

asomers added inline comments.
tests/sys/geom/class/multipath/misc.sh
201 ↗(On Diff #63932)

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.