src/tests/sys/cddl, branch stable/13FreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=stable%2F132025-11-19T20:47:50Zsrc.conf: Add a MK_ZFS_TESTS knob2025-11-19T20:47:50ZMark Johnstonmarkj@FreeBSD.org2024-09-07T14:36:28Zurn:sha1:90118a35a3ae469ca91a4c52b3623bf4d0c5212e
The in-tree ZFS test suite is somewhat outdated and I see a number of
failures there. I tend to think that we want to integrate the OpenZFS
test suite somehow, replacing the legacy one, though it's also possible
to run that as a separate test suite.
In any case, if one wants to run the OpenZFS test suite separately, it's
useful to be able to disable installation of the legacy ZFS test suite,
so let's provide a src.conf option to do that.
Reviewed by: asomers
MFC after: 2 weeks
Differential Revision: https://reviews.freebsd.org/D46476
(cherry picked from commit 24affded3d4ec5fafb6b22f773ec1e20d73b9b03)
zfsd: fault disks that generate too many I/O delay events2024-01-19T20:16:48ZAlan Somersasomers@FreeBSD.org2023-07-12T20:46:27Zurn:sha1:e2ce586899ff94d8e3baf10de614dfe61d06a5e0
If ZFS reports that a disk had at least 8 I/O operations over 60s that
were each delayed by at least 30s (implying a queue depth > 4 or I/O
aggregation, obviously), fault that disk. Disks that respond this
slowly can degrade the entire system's performance.
Sponsored by: Axcient
Reviewed by: delphij
Differential Revision: https://reviews.freebsd.org/D42825
(cherry picked from commit d565784a7ebaa59e26febdcfd4a60329786ea5f5)
mkfile: getopt() returns an int, not a char.2023-09-14T14:59:52ZDag-Erling Smørgravdes@FreeBSD.org2023-09-10T18:33:18Zurn:sha1:f7523720d2eca9287a0bddf9d2233366fa37ae99
MFC after: 3 days
Reviewed by: kevans
Differential Revision: https://reviews.freebsd.org/D41804
(cherry picked from commit ef8abddf103d9dfd5660d50d15e6dbfbb2f47f62)
Multiple fixes to the zfsd test suite2023-09-06T21:49:06ZAlan Somersasomers@FreeBSD.org2023-04-03T21:43:17Zurn:sha1:f85da5f88efce6177d0e5be2b74c48599e5c471a
* Wait for gnop devices to disappear after "gnop destroy".
Apparently that process is asynchronous now, or maybe it's just slower
than it used to be. Also, after removing a gnop wait for its pool to
be degraded. That isn't instant.
* The zfsd tests no longer require camcontrol.
This was a harmless oversight from
11ed0a95bfa76791dc6428eb2d47a986c0c6f8a3
* Fix the zfsd_degrade_001_pos test for recent zfs versions.
ZFS now rate limits checksum errors to about 20 per second. But
zfsd's threshold for degrading a disk is 50 per minute. So we must
alternately corrupt and scrub the pool to ensure that checksum errors
are generated in multiple 1-second windows, so that zfsd will see
enough of them.
* Fix the zfsd_fault_001_pos test in VMs
And, for that matter, when using NVME or SATA disks. As originally
written, the test used the da driver to inject errors. Rewrite it to
use gnop vdevs. gnop can also inject errors. It works on top of any
disk device, and it's also faster than using da.
Sponsored by: Axcient
Differential Revision: https://reviews.freebsd.org/D39437
(cherry picked from commit dba2e89ea7a13469ee2e47a2a1d627ca28bb94c2)
Fix the zfsd_autoreplace_001_neg test2023-09-06T20:44:53ZAlan Somersasomers@FreeBSD.org2023-07-05T22:37:39Zurn:sha1:b2c207ddfc459b82c0ec5d0c52ab66a9cc169257
It only ever worked by accident.
* Actually set autoreplace=off as stated in the description
* Wait for the removed device to disappear from the pool before
proceeding.
* In the assertion, verify that the new disk does not get added, as
described in the description.
Sponsored by: Axcient
(cherry picked from commit be092bcde96bdcfde9013d60e442cca023bfbd1b)
Remove $FreeBSD$: one-line sh pattern2023-08-23T17:43:30ZWarner Loshimp@FreeBSD.org2023-08-22T01:32:01Zurn:sha1:023fc80ee38a117fa65b2ccb2abf8bdc7dbd6fd9
Remove /^\s*#[#!]?\s*\$FreeBSD\$.*$\n/
Similar commit in main:
(cherry picked from commit d0b2dbfa0ecf)
Remove $FreeBSD$: one-line .h pattern2023-08-23T17:43:22ZWarner Loshimp@FreeBSD.org2023-08-22T01:31:13Zurn:sha1:34041aac835a0bce462bccb7e0239c0ba092f872
Remove /^\s*\*+\s*\$FreeBSD\$.*$\n/
Similar commit in main:
(cherry picked from commit 42b388439bd3)
Remove $FreeBSD$: two-line .h pattern2023-08-23T17:43:21ZWarner Loshimp@FreeBSD.org2023-08-22T01:31:07Zurn:sha1:17da660ad5b3b9cd90e164dd4dbb9beaa7203054
Remove /^\s*\*\n \*\s+\$FreeBSD\$$\n/
Similar commit in main:
(cherry picked from commit b3e7694832e8)
Merge fix for zfs readmmap test from CheriBSD2022-07-19T17:59:44ZDimitry Andricdim@FreeBSD.org2022-07-15T19:10:09Zurn:sha1:3ef2e7e27925747f5b9d0943a496f50875bc6610
Merge commit 1737d8397a0 by Brooks Davis:
time() is declared in time.h
This fixes a -Werror warning from clang 15:
tests/sys/cddl/zfs/bin/readmmap.c:97:9: error: call to undeclared function 'time'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
Obtained from: https://github.com/CTSRD-CHERI/cheribsd/commit/1737d8397a0
MFC after: 3 days
(cherry picked from commit 47be48457393ba6747cc3144272b1f4d8b9077fb)
zfs: Update test format strings to match variable typtes2022-03-08T00:33:21ZEd Masteemaste@FreeBSD.org2022-02-28T01:11:20Zurn:sha1:71f452d58c7aa4625816015eec153718021b5db4
And drop stray 'd' from the end of some printed numbers. I assume this
was the result of someone thinking u is a printf length modifier for d,
not a format specifier itself.
Reviewed by: kevans, rew
MFC after: 1 week
Sponsored by: The FreeBSD Foundation
Differential Revision: https://reviews.freebsd.org/D34387
(cherry picked from commit f27fb06cadc6a8e75ebebd3b60c8a9dc9ae1b833)