Page MenuHomeFreeBSD

Skip tarfs regression test if module doesn't exist
ClosedPublic

Authored by olivier on Feb 28 2023, 8:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 2, 2:26 AM
Unknown Object (File)
Sun, Nov 30, 6:52 AM
Unknown Object (File)
Tue, Nov 25, 7:55 AM
Unknown Object (File)
Oct 27 2025, 1:01 AM
Unknown Object (File)
Oct 21 2025, 7:21 AM
Unknown Object (File)
Oct 10 2025, 8:03 PM
Unknown Object (File)
Oct 10 2025, 8:02 PM
Unknown Object (File)
Oct 10 2025, 8:02 PM
Subscribers

Details

Summary

When building customized FreeBSD without tarfs module, this regression test fail because
it can't load the module.

Test Plan
 mv /boot/kernel/tarfs.ko /boot/kernel/tarfs.ko.bak
# kyua test sys/fs/tarfs/
sys/fs/tarfs/tarfs_test:tarfs_basic  ->  skipped: This test requires tarfs and could not load it  [0.020s]
sys/fs/tarfs/tarfs_test:tarfs_notdir_device  ->  skipped: This test requires tarfs and could not load it  [0.020s]
sys/fs/tarfs/tarfs_test:tarfs_notdir_dot  ->  skipped: This test requires tarfs and could not load it  [0.019s]
sys/fs/tarfs/tarfs_test:tarfs_notdir_dotdot  ->  skipped: This test requires tarfs and could not load it  [0.019s]
sys/fs/tarfs/tarfs_test:tarfs_notdir_file  ->  skipped: This test requires tarfs and could not load it  [0.019s]

Results file id is usr_tests.20230228-200204-811621
Results saved to /root/.kyua/store/results.usr_tests.20230228-200204-811621.db

5/5 passed (0 failed)

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

olivier created this revision.
This revision is now accepted and ready to land.Feb 28 2023, 8:10 PM

The kldload check function seems like it should be in a common function. Also: I might reword the message to state: “This test requires the tarfs kernel module (not available).”

tests/sys/fs/tarfs/tarfs_test.sh
51

Should this use umount -f ?

tests/sys/fs/tarfs/tarfs_test.sh
51

Or maybe a different sanity check to run beforehand, like ‘test -d “${mnt}”’?

In D38828#884135, @ngie wrote:

The kldload check function seems like it should be in a common function. Also: I might reword the message to state: “This test requires the tarfs kernel module (not available).”

About the "not available", I didn't add this precision because in some case it could be a loading failure too (like a child module not available).