aboutsummaryrefslogtreecommitdiff
path: root/tests/sys/geom/class
Commit message (Collapse)AuthorAgeFilesLines
* tests: Increase timeoutJose Luis Duran2026-01-072-1/+2
| | | | | | | | | | Bump the timeout value, to avoid cutoff on emulated architectures on ci.freebsd.org. Reported by: Jenkins Reviewed by: ngie MFC after: 1 week Differential Revision: https://reviews.freebsd.org/D54550
* glabel: add a basic testBrad Davis2025-10-173-0/+67
| | | | | | | Reviewed by: asomers MFC after: 3 days Sponsored by: Rubicon Communications, LLC ("Netgate") Differential Revision: https://reviews.freebsd.org/D53178
* ggate tests: Use unique ports among testsMark Johnston2024-09-041-6/+6
| | | | | | This helps avoid failures when running tests in parallel. MFC after: 1 week
* Remove residual blank line at start of MakefileWarner Losh2024-07-1512-12/+0
| | | | | | | This is a residual of the $FreeBSD$ removal. MFC After: 3 days (though I'll just run the command on the branches) Sponsored by: Netflix
* gnop tests: SerializeMark Johnston2024-07-141-0/+4
| | | | | | | Some of these tests make use of test disk devices, but then they can't run in parallel. MFC after: 2 weeks
* virstor: basic functional testRyan Libby2024-06-104-0/+114
| | | | | | Reviewed by: asomers, markj Sponsored by: Dell EMC Isilon Differential Revision: https://reviews.freebsd.org/D45535
* geli: allocate a UMA pool earlierMariusz Zaborski2024-05-191-0/+29
| | | | | | | | | | | | | | | | | | | The functions g_eli_init_uma and g_eli_fini_uma are used to trace the number of devices in GELI. There is an issue where the g_eli_create function may fail before g_eli_init_uma is called, however g_eli_fini_uma is still executed in the fail path. This can incorrectly decrease the device count to zero, potentially leading to the UMA pool being freed. Accessing the device after the pool has been freed causes a system panic. This commit resolves the issue by ensuring devices count is increassed eariler. PR: 278828 Reported by: Andre Albsmeier <mail@fbsd2.e4m.org> Reviewed by: asomers MFC after: 3 days Differential Revision: https://reviews.freebsd.org/D45225
* tests: don't run atf_* in a subshellGleb Smirnoff2023-11-2750-168/+172
| | | | | | | | | | | | | Shell limitation is that a classic function call via $() is a subshell and atf-sh(3) commands won't work as epxected there. Subsequently, atf_skip inside a function won't skip a test. The test will fail later. A working approach is to pass desired variable name as argument to a function and don't run subshell. Reviewed by: ngie Differential Revision: https://reviews.freebsd.org/D42646 Fixes: ea82362219ee715cfbb195b2114e73fdc8599fa5
* Add tests for gunion(8)Yan-Hao Wang2023-11-164-0/+388
| | | | | | | Reviewed by: mckusick (earlier version) MFC after: 3 days Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D41645
* geli tests: Use shorter passphrases in setkey_passphraseMark Johnston2023-10-021-3/+3
| | | | | | | | | There is an undocumented limit on the length of a GELI passphrase, and the test exceeded that. Most of the time it worked because a nul terminator would appear early enough in the string. Reported by: Jenkins Fixes: 2b7b09ac9675 ("geli tests: Add a regression test for geli setkey -J")
* tests: Skip all tests that require mdconfig when /dev/mdctl missingWarner Losh2023-09-023-0/+7
| | | | | | | | | | When run in a jail, /dev/mdctl is missing. So skip any tests that use mdconfig or mdmfs with md in this case: they can't possibly work. This is in line with other tests that test for presence of required features and skip if they aren't present. I did this instead of checking for jails so they can still run in jails that allow creation of md devices. Sponsored by: Netflix
* geli tests: Add a regression test for geli setkey -JMark Johnston2023-08-281-0/+59
| | | | | | | | | Make sure that it can be used to change the passphrase of both attached and detached providers. PR: 254966 MFC after: 1 week Sponsored by: The FreeBSD Foundation
* Remove $FreeBSD$: one-line sh patternWarner Losh2023-08-1690-90/+0
| | | | Remove /^\s*#[#!]?\s*\$FreeBSD\$.*$\n/
* Remove $FreeBSD$: one-line .c comment patternWarner Losh2023-08-161-1/+0
| | | | Remove /^/[*/]\s*\$FreeBSD\$.*\n/
* Remove $FreeBSD$: one-line .h patternWarner Losh2023-08-161-1/+0
| | | | Remove /^\s*\*+\s*\$FreeBSD\$.*$\n/
* ggate tests: SerializeMark Johnston2023-07-311-0/+3
|
* gmirror tests: SerializeMark Johnston2023-07-311-0/+3
|
* geli tests: Add a regression test for PR 271766Mark Johnston2023-06-123-2/+170
| | | | | | | | | | This test case catches both of the bugs reported there. PR: 271766 Reviewed by: imp MFC after: 1 week Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D40469
* spdx: The BSD-2-Clause-FreeBSD identifier is obsolete, drop -FreeBSDWarner Losh2023-05-123-3/+3
| | | | | | | | | The SPDX folks have obsoleted the BSD-2-Clause-FreeBSD identifier. Catch up to that fact and revert to their recommended match of BSD-2-Clause. Discussed with: pfg MFC After: 3 days Sponsored by: Netflix
* Skip sys.geom.class.multipath.failloop.failloop if dtrace fails to runLi-Wen Hsu2022-03-081-1/+1
| | | | | | | | This test case depends on dtrace and sometimes gets affected if dtrace has issues. Make it report skipped instead of failure when dtrace fails to run. Sponsored by: The FreeBSD Foundation
* gmultipath tests: Re-enable the failloop test in CIMark Johnston2022-03-071-4/+0
| | | | Sponsored by: The FreeBSD Foundation
* Temporarily skip sys.geom.class.multipath.failloop.failloop in CILi-Wen Hsu2022-02-231-0/+4
| | | | | | | | | | | | | This test case uses `dtrace -c` but it has some issues at the moment so disable it until dtrace fixed. From markj: This is the CTF type ID limit which has come up quite a few times lately. It'll be fixed with the introduction of CTFv3. PR: 258763 Sponsored by: The FreeBSD Foundation
* gmultipath tests: Re-enable the failloop test in CIMark Johnston2021-10-041-4/+0
| | | | | | PR: 258763 MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
* gmultipath failloop test: Put the dtrace sanity checker in right placeLi-Wen Hsu2021-09-281-3/+3
| | | | | | Check if dtrace excution is successful or not right after execution. Sponsored by: The FreeBSD Foundation
* Fix typoLi-Wen Hsu2021-09-281-1/+1
| | | | | Reported by: swills Sponsored by: The FreeBSD Foundation
* Temporarily skip sys.geom.class.multipath.failloop.failloop in CILi-Wen Hsu2021-09-281-0/+7
| | | | | | | | | | This test case uses `dtrace -c` but it has some issues at the moment While here, add a checker for dtrace executes successfully or not to provide a more informative error message. PR: 258763 Sponsored by: The FreeBSD Foundation
* Add a regression test for ggatec remote code executionAlan Somers2021-08-271-0/+34
| | | | | | | | | | Tests that ggatec appropriately handles unsupported BIO operations, rather than overflowing a buffer. Submitted by: Johannes Bruelltuete <johannes@jo-t.de> PR: 213479 Reviewed by: asomers Differential Revision: https://reviews.freebsd.org/D31318
* gconcat: Add some simple regression tests for the new append verbMark Johnston2021-06-143-0/+127
| | | | Sponsored by: The FreeBSD Foundation
* gmultipath: make physpath distinct from the underlying providers'Alan Somers2021-05-061-2/+2
| | | | | | | | | | | | | | | | | zfsd uses a device's physical path attribute to automatically replace a missing ZFS disk when a blank disk is inserted into the same physical slot. Currently gmultipath passes through its underlying providers' physical path attribute. That may cause zfsd to replace a missing gmultipath provider with a newly arrived, single-path disk. That would be bad. This commit fixes that problem by simply appending "/mp" to the underlying providers' physical path, in a manner similar to what geli already does. Sponsored by: Axcient MFC after: 3 weeks Differential Revision: https://reviews.freebsd.org/D29941
* build: provide a default WARNS for all in-tree buildsKyle Evans2020-09-181-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | The current default is provided in various Makefile.inc in some top-level directories and covers a good portion of the tree, but doesn't cover parts of the build a little deeper (e.g. libcasper). Provide a default in src.sys.mk and set WARNS to it in bsd.sys.mk if that variable is defined. This lets us relatively cleanly provide a default WARNS no matter where you're building in the src tree without breaking things outside of the tree. Crunchgen has been updated as a bootstrap tool to work on this change because it needs r365605 at a minimum to succeed. The cleanup necessary to successfully walk over this change on WITHOUT_CLEAN builds has been added. There is a supplemental project to this to list all of the warnings that are encountered when the environment has WARNS=6 NO_WERROR=yes: https://warns.kevans.dev -- this project will hopefully eventually go away in favor of CI doing a much better job than it. Reviewed by: emaste, brooks, ngie (all earlier version) Reviewed by: emaste, arichardson (depend-cleanup.sh change) Differential Revision: https://reviews.freebsd.org/D26455 Notes: svn path=/head/; revision=365887
* Limit gmirror failpoint tests to the test workerBryan Drewery2020-07-237-7/+12
| | | | | | | | | | | | | | This avoids injecting errors into the test system's mirrors. gnop seems like a good solution here but it injects errors at the wrong place vs where these tests expect and does not support a 'max global count' like the failpoints do with 'n*' syntax. Reviewed by: cem, vangyzen Sponsored by: Dell EMC Isilon Notes: svn path=/head/; revision=363463
* Fix sys.geom.class.eli.onetime_test.onetime after r363402Li-Wen Hsu2020-07-221-1/+1
| | | | | | | | | PR: 247954 X-MFC with: r363402 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=363423
* Fix geli's null cipher, and add a test caseAlan Somers2020-07-211-0/+45
| | | | | | | | | | | PR: 247954 Submitted by: jhb (sys), asomers (tests) Reviewed by: jhb (tests), asomers (sys) MFC after: 2 weeks Sponsored by: Axcient Notes: svn path=/head/; revision=363402
* geli: enable direct dispatchAlan Somers2020-07-082-0/+70
| | | | | | | | | | | | | | | geli does all of its crypto operations in a separate thread pool, so g_eli_start, g_eli_read_done, and g_eli_write_done don't actually do very much work. Enabling direct dispatch eliminates the g_up/g_down bottlenecks, doubling IOPs on my system. This change does not affect the thread pool. Reviewed by: markj MFC after: 2 weeks Sponsored by: Axcient Differential Revision: https://reviews.freebsd.org/D25587 Notes: svn path=/head/; revision=363014
* Temporarily skip unstable sys.geom.class.gate.ggate_test.ggated on i386 in CILi-Wen Hsu2020-06-241-0/+5
| | | | | | | | PR: 244737 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=362573
* Revert 357780, `dtrace -c` has been fixed in head`Li-Wen Hsu2020-05-181-4/+0
| | | | | | | | PR: 244053 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361229
* Reenable sys.geom.class.gate.ggate_test.ggated in CIAlan Somers2020-05-171-4/+0
| | | | | | | | | | Should be fixed by r360613 PR: 244737 Reported by: lwhsu Notes: svn path=/head/; revision=361129
* Fix the sys.geom.class.multipath.misc.fail_on_error test on stable/12Alan Somers2020-05-081-1/+1
| | | | | | | | | | | | | This test uses a gnop feature (delay probability) that isn't available on stable/12. But it's unnecessary; the test works fine without it. Removing it simplifies the test and, once MFCed, will allow it to pass on stable/12. PR: 244158 Reported by: lwhsu MFC after: 2 weeks Notes: svn path=/head/; revision=360807
* Fix intermittent cleanup failures in the ggated testAlan Somers2020-05-031-1/+5
| | | | | | | MFC after: 2 weeks Notes: svn path=/head/; revision=360613
* geli tests require OpenSSL - exclude them under WITHOUT_OPENSSLEd Maste2020-04-301-0/+4
| | | | | | | | PR: 245931 Submitted by: The FreeBSD Foundation Notes: svn path=/head/; revision=360509
* Temporarily skip sys.geom.class.gate.ggate_test.ggated in CILi-Wen Hsu2020-03-111-0/+4
| | | | | | | | PR: 244737 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=358886
* Temporarily skip sys.geom.class.multipath.failloop.failloop in CILi-Wen Hsu2020-02-111-0/+4
| | | | | | | | | | This case uses `dtrace -c` but it has some issues at the moment PR: 244053 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=357780
* geli: add a test case for attaching multiple providers with 1 commandAlan Somers2020-01-281-0/+31
| | | | | | | | | | Reviewed by: cem MFC after: 2 weeks Sponsored by: Axcient Differential Revision: https://reviews.freebsd.org/D23400 Notes: svn path=/head/; revision=357211
* Revert r355861 because the issue has been fixed in r356443Li-Wen Hsu2020-01-071-5/+0
| | | | | | | | | PR: 242689 Notified by: markj Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=356458
* Temporarily skip failing sys.geom.class.multipath.failloop.failloop on i386 CILi-Wen Hsu2019-12-171-0/+5
| | | | | | | | PR: 242689 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=355861
* gmultipath: add ATF testsAlan Somers2019-12-065-0/+561
| | | | | | | | | | | | | | 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 MFC after: 2 weeks Sponsored by: Axcient Differential Revision: https://reviews.freebsd.org/D22235 Notes: svn path=/head/; revision=355431
* Remove tests for the deprecated algorithms in r348206Li-Wen Hsu2019-05-312-18/+1
| | | | | | | | | | | | | The tests are failing because the return value and output have changed, but before test code structure adjusted, removing these test cases help people be able to focus on more important cases. Discussed with: emaste MFC with: r348206 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=348454
* Skip test component_selection:run_latest_genid if gmirror/gnop GEOM classesOlivier Cochard2019-04-231-1/+8
| | | | | | | | | | | | | | aren't available PR: 237051 Reviewed by: asomers, imp, ngie, emaste (IRC) Approved by: ngie MFC after: 1 month Sponsored by: Netflix Differential Revision: https://reviews.freebsd.org/D19958 Notes: svn path=/head/; revision=346605
* Fix geli device cleanupEnji Cooper2019-04-091-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | Final cleanup routines shouldn't be called from testcases; it should be called from the testcase cleanup routine. Furthermore, `geli_test_cleanup` should take care of cleaning up geli providers and the memory disks used for the geli providers. `geli_test_cleanup` will always be executed whereas the equivalent logic in `geli_test_body`, may not have been executed if the test failed prior to the logic being run. Prior to this change, the test case was trying to clean up `$md` twice: once in at the end of the test case body function, and the other in the cleanup function. The cleanup function logic was failing because there wasn't anything to clean up in the cleanup function and the errors weren't being ignored. This fixes FreeBSD test suite runs after r345864. PR: 237128 Reviewed by: asomers, pjd Approved by: emaste (mentor) MFC with: r345864 Differential Revision: https://reviews.freebsd.org/D19854 Notes: svn path=/head/; revision=346057
* Implement tests for online expansion:Pawel Jakub Dawidek2019-04-042-0/+197
| | | | | | | | | | | | | | | | | | | - init, init -R - onetime, onetime -R - 512 and 4k sectors - encryption only - encryption and authentication - configure -r/-R for detached providers - configure -r/-R for attached providers - all keys allocated (10, 20 and 30MB provider sizes) - keys allocated on demand (10, 20 and 30PB provider sizes) - reading and writing to provider after expansion (10-30MB only) - checking if metadata in old location is cleared. Obtained from: Fudo Security Notes: svn path=/head/; revision=345864