aboutsummaryrefslogtreecommitdiff
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
...
* fusefs: fix the FUSE_FORGET unit test after r364064Alan Somers2020-08-111-1/+2
| | | | | | | | | | | | Thanks to r364064, the name cache now returns a hit where previously it would miss. Adjust the expectations accordingly. PR: 248583 Reported by: lwhsu MFC with: r364064 Notes: svn path=/head/; revision=364094
* Use static inline for iface_{setup,delete}_addr in tests/sys/net/routing.Dimitry Andric2020-08-081-2/+2
| | | | | | | | | | | | | | | | | | | | | This fixes possible link errors, similar to: ld: error: undefined symbol: iface_setup_addr >>> referenced by test_rtsock_l3.c:111 (tests/sys/net/routing/test_rtsock_l3.c:111) >>> test_rtsock_l3.o:(presetup_ipv4) >>> referenced by test_rtsock_l3.c:79 (tests/sys/net/routing/test_rtsock_l3.c:79) >>> test_rtsock_l3.o:(presetup_ipv6) >>> referenced by test_rtsock_l3.c:512 (tests/sys/net/routing/test_rtsock_l3.c:512) >>> test_rtsock_l3.o:(atfu_rtm_change_v4_gw_success_body) >>> referenced 10 more times In C (not C++), 'naked' inline is almost always a mistake. Either use static inline (this is appropriate for most cases), or extern inline. MFC after: 3 days Notes: svn path=/head/; revision=364050
* 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
* add a few tests for sendfile.Chuck Silvers2020-07-203-0/+305
| | | | | | | | | Reviewed by: markj Sponsored by: Netflix Differential Revision: https://reviews.freebsd.org/D25431 Notes: svn path=/head/; revision=363377
* tests/sys/opencrypto: use python3Alan Somers2020-07-204-6/+6
| | | | | | | | | | | | python2 will be EOL soon Reviewed by: lwhsu, jmg MFC after: 2 weeks Sponsored by: Axcient Differential Revision: https://reviews.freebsd.org/D25682 Notes: svn path=/head/; revision=363361
* Add safexcel(4) to cryptotest.Mark Johnston2020-07-141-2/+3
| | | | | | | | MFC after: 1 week Sponsored by: Rubicon Communications, LLC (Netgate) Notes: svn path=/head/; revision=363181
* Don't leave `path` behind when executing `:chflags_success`Enji Cooper2020-07-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | Prior to this change a `SF_IMMUTABLE` chflagsat(2)'ed file (`path`) was left behind, which sabotaged kyua(1) from being able to clean up the work directory, This resulted in unnecessary work for folks having to clean up the work directory on non-disposable systems, which defaults to `/tmp`. Use `UF_OFFLINE` instead of `SF_IMMUTABLE`, in part because setting `SF_IMMUTABLE` isn't relevant to the test and `SF_IMMUTABLE` cannot be cleared at all securelevels, as pointed out by @asomers. Additional work is required to catch cases like this upfront in the future to avoid tester headache. See PR # 247765 for more details/followup. Suggested by: asomers Reviewed By: asomers, #tests MFC after: 1 week PR: 247761 Sponsored by: DellEMC Differential Revision: https://reviews.freebsd.org/D25561 Notes: svn path=/head/; revision=363132
* memfd_create: turn on SHM_GROW_ON_WRITEKyle Evans2020-07-101-4/+13
| | | | | | | | | | | memfd_create fds will no longer require an ftruncate(2) to set the size; they'll grow (to the extent that it's possible) upon write(2)-like syscalls. Reviewed by: kib Differential Revision: https://reviews.freebsd.org/D25502 Notes: svn path=/head/; revision=363066
* 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
* Fix IPv6 regression introduced by r362900.Alexander V. Chernikov2020-07-031-1/+9
| | | | | | | PR: kern/247729 Notes: svn path=/head/; revision=362909
* Temporarily skip sys.netinet6.redirect.valid_redirect in CILi-Wen Hsu2020-07-031-0/+4
| | | | | | | | PR: 247729 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=362904
* Make sys.auditpipe depend on /dev/auditpipe. This fixes a few failuresEdward Tomasz Napierala2020-06-241-0/+1
| | | | | | | | | | on armv7. MFC after: 2 weeks Sponsored by: DARPA Notes: svn path=/head/; revision=362575
* 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
* Make audit tests depend on /dev/auditpipe. This should fixEdward Tomasz Napierala2020-06-181-1/+1
| | | | | | | | | | | | | | some 416 failing tests on armv7: https://ci.freebsd.org/job/FreeBSD-head-armv7-test/lastCompletedBuild/testReport/ Reviewed by: asomers MFC after: 2 weeks Sponsored by: DARPA Differential Revision: https://reviews.freebsd.org/D25144 Notes: svn path=/head/; revision=362330
* Skip sys.net.if_lagg_test.lacp_linkstate_destroy_stress in CI because of panicLi-Wen Hsu2020-06-111-2/+1
| | | | | | | | PR: 244168 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=362072
* bridge tests: Re-enable STP testKristof Provost2020-06-071-4/+0
| | | | | | | | | | | | | This test should no longer provoke large amounts of traffic, which can overwhelm single-core systems, preventing them from making progress in the tests. The test can now be re-enabled. PR: 246448 Notes: svn path=/head/; revision=361893
* bridge tests: Avoid building a switching loopKristof Provost2020-06-011-14/+3
| | | | | | | | | | | | | | | Enable STP before bringing the bridges up. This avoids a switching loop, which has a tendency to drown out progress in userspace processes, especially on single-core systems. Only check that we have indeed shut down one of the looped interfaces PR: 246448 Reviewed by: melifaro Differential Revision: https://reviews.freebsd.org/D25084 Notes: svn path=/head/; revision=361701
* Revert r361497, these cases are fixed by r361617.Li-Wen Hsu2020-05-294-16/+0
| | | | | | | | PR: 246737 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361623
* Proper check if divert(4) module is present by the relevant testsShteryana Shopova2020-05-272-2/+2
| | | | | | | | | | | Fix the netinet/netinet6 divert tests falsely reporting 'ipdivert module is not loaded' when the divert module is built into the kernel Sponsored by: Axiado Differential Revision: https://reviews.freebsd.org/D25026 Notes: svn path=/head/; revision=361553
* Disable failing test cases in CI:Li-Wen Hsu2020-05-264-0/+16
| | | | | | | | | | | | | sys.netipsec.tunnel.aes_cbc_128_hmac_sha1.v4 sys.netipsec.tunnel.aes_cbc_256_hmac_sha2_256.v4 sys.netipsec.tunnel.aesni_aes_cbc_128_hmac_sha1.v4 sys.netipsec.tunnel.aesni_aes_cbc_256_hmac_sha2_256.v4 PR: 246737 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361497
* Fix issues with FUSE_ACCESS when default_permissions is disabledAlan Somers2020-05-227-90/+230
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes two issues relating to FUSE_ACCESS when the default_permissions mount option is disabled: * VOP_ACCESS() calls with VADMIN set should never be sent to a fuse server in the form of FUSE_ACCESS operations. The FUSE protocol has no equivalent of VADMIN, so we must evaluate such things kernel-side, regardless of the default_permissions setting. * The FUSE protocol only requires FUSE_ACCESS to be sent for two purposes: for the access(2) syscall and to check directory permissions for searchability during lookup. FreeBSD sends it much more frequently, due to differences between our VFS and Linux's, for which FUSE was designed. But this patch does eliminate several cases not required by the FUSE protocol: * for any FUSE_*XATTR operation * when creating a new file * when deleting a file * when setting timestamps, such as by utimensat(2). * Additionally, when default_permissions is disabled, this patch removes one FUSE_GETATTR operation when deleting a file. PR: 245689 Reported by: MooseFS FreeBSD Team <freebsd@moosefs.pro> Reviewed by: cem MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D24777 Notes: svn path=/head/; revision=361401
* Avoid hard-coding pipe buffer sizes in the pipe and fifo kqueue tests.Mark Johnston2020-05-202-57/+61
| | | | | | | | | | | Fix some style issues in the modified tests while here. Reported by: Jenkins via lwhsu MFC after: 1 week Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361286
* Make sys.net.if_bridge_test depend on python; sys.net.if_bridge_test.spanEdward Tomasz Napierala2020-05-191-0/+1
| | | | | | | | | | requires it. MFC after: 2 weeks Sponsored by: DARPA Notes: svn path=/head/; revision=361268
* Make sys.netinet.output and sys.netinet6.output6 tests depend on python;Edward Tomasz Napierala2020-05-192-0/+2
| | | | | | | | | | they need to be able to run net_receiver.py. MFC after: 2 weeks Sponsored by: DARPA Notes: svn path=/head/; revision=361267
* Revert r361237, it's fixed by r361231Li-Wen Hsu2020-05-191-3/+0
| | | | | | | | PR: 246560 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361241
* Temporarily disable test case causes kernel panic in CI:Li-Wen Hsu2020-05-191-0/+3
| | | | | | | | | | - sys.netinet.so_reuseport_lb_test.basic_ipv6 PR: 246560 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361237
* 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
* fusefs: fix intermittency in some ENOENT testsAlan Somers2020-05-183-0/+35
| | | | | | | | | | | | | When a FUSE operation other than LOOKUP returns ENOENT, the kernel will reclaim that vnode, resuling in a FUSE_FORGET being sent a short while later. Many of the ENOENT tests weren't expecting those FUSE_FORGET operations. They usually passed by luck since FUSE_FORGET is often delayed. This commit adds appropriate expectations. MFC after: 2 weeks Notes: svn path=/head/; revision=361223
* 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
* pf tests: fix up a couple WARNS= 6 nitsKyle Evans2020-05-141-1/+2
| | | | | | | | | | | | | | | | | | | | | | common_init_tbl is only used within this single CU, so it should be marked static. WARNS=6 also complained about the var defined by `ATF_TC_WITH_CLEANUP(getastats);` being unused, which turns out to be because it's not been hooked up in ATF_TP_ADD_TCS. kp@ did not immediately recall any reason for this, and the case passes on my local system, so hook it up. Note that I've not yet set WARNS= 6 here. Investigation is underway to see if we can feasibly default WARNS to 6 for src builds to catch directories too deep to inherit a WARNS from the top-level subdirectories' Makefile.inc. Those particular WARNS settings will be subsequently removed as they become redundant with a more-global default. MFC after: 1 week Notes: svn path=/head/; revision=361065
* Only skip sys.net.if_clone_test.epair_stress in CI envLi-Wen Hsu2020-05-131-1/+3
| | | | | | | | PR: 246443 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361019
* Temporarily skip sys.net.if_bridge_test.stp in CI as it always times outLi-Wen Hsu2020-05-131-0/+4
| | | | | | | | PR: 244229 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361018
* Temporarily skip sys.net.if_clone_test.epair_stressLi-Wen Hsu2020-05-131-0/+1
| | | | | | | | | | This case timed out so often PR: 246443 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=361017
* Add a basic table entry counter regression test.Mark Johnston2020-05-112-0/+116
| | | | | | | | | Reviewed by: kp MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D24809 Notes: svn path=/head/; revision=360906
* Remove the svn:executable property from some pf test files.Mark Johnston2020-05-1114-0/+0
| | | | | | | | | | | The test makefiles will handle setting mode bits during install. Also, Phabricator gets upset when uploading an executable plain-text file without a shebang. MFC after: 1 week Notes: svn path=/head/; revision=360905
* fusefs: fix two small bugs in the tests' expectationsAlan Somers2020-05-082-2/+2
| | | | | | | | | | | These two errors have been present since the tests' introduction. Coincidentally every test (I think there's only one) that cares about that field also works when the field's value is 0. MFC after: 2 weeks Notes: svn path=/head/; revision=360829
* 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
* Add basic routing LPM tests.Alexander V. Chernikov2020-05-064-2/+381
| | | | | | | Differential Revision: https://reviews.freebsd.org/D24684 Notes: svn path=/head/; revision=360685
* Revert r360204, sys.netipsec.tunnel.empty.v{4,6} are fixed by r360560Li-Wen Hsu2020-05-041-8/+0
| | | | | | | | PR: 245832 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=360626
* Fix intermittent cleanup failures in the ggated testAlan Somers2020-05-031-1/+5
| | | | | | | MFC after: 2 weeks Notes: svn path=/head/; revision=360613
* Resolve conflict between the fusefs(5) and mac_bsdextended(4) testsAlan Somers2020-05-022-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | mac_bsdextended(4), when enabled, causes ordinary operations to send many more VOP_GETATTRs to file system. The fusefs tests expectations aren't written with those in mind. Optionally expecting them would greatly obfuscate the fusefs tests. Worse, certain fusefs functionality (like attribute caching) would be impossible to test if the tests couldn't expect an exact number of GETATTR operations. This commit resolves that conflict by making two changes: 1. The fusefs tests will now check for mac_bsdextended, and skip if it's enabled. 2. The mac_bsdextended tests will now check whether the module is enabled, not merely loaded. If it's loaded but disabled, the tests will automatically enable it for the duration of the tests. With these changes, a CI system can achieve best coverage by loading both fusefs and mac_bsdextended at boot, and setting security.mac.bsdextended.enabled=0 PR: 244229 Reported by: lwhsu Reviewed by: cem MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D24577 Notes: svn path=/head/; revision=360567
* Remove support for the algorithms deprecated in r348876.John Baldwin2020-05-021-41/+0
| | | | | | | | | | | | | | | | | | | | | | This removes support for the following algorithms: - ARC4 - Blowfish - CAST128 - DES - 3DES - MD5-HMAC - Skipjack Since /dev/crypto no longer supports 3DES, stop testing the 3DES KAT vectors in cryptotest.py. Reviewed by: cem (previous version) Relnotes: yes Sponsored by: Chelsio Communications Differential Revision: https://reviews.freebsd.org/D24346 Notes: svn path=/head/; revision=360562
* 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
* Add some regression tests for read and write kevents on pipes.Mark Johnston2020-04-274-0/+798
| | | | | | | | | Submitted by: Jan Kokemüller <jan.kokemueller@gmail.com> MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D24528 Notes: svn path=/head/; revision=360381
* bridge tests: Test for #216510Kristof Provost2020-04-261-0/+28
| | | | | | | | | | | | | | | | | | | We used to have an issue with recursive locking with net.link.bridge.inherit_mac. This causes us to send an ARP request while we hold the BRIDGE_LOCK, which used to cause us to acquire the BRIDGE_LOCK again. We can't re-acquire it, so this caused a panic. Now that we no longer need to acquire the BRIDGE_LOCK for bridge_transmit() this should no longer panic. Test this. PR: 216510 Reviewed by: emaste, philip MFC after: 2 months Sponsored by: The FreeBSD Foundation Differential Revision: https://reviews.freebsd.org/D24251 Notes: svn path=/head/; revision=360346
* mac_bsdextended: ATFify the testsAlan Somers2020-04-262-302/+329
| | | | | | | | | | | | The new tests have more complete setup and cleanup, are more granular, and correctly annotate expected failures and skipped tests. A follow-up commit will resolve a conflict with the fusefs tests (bug 244229). MFC after: 2 weeks Differential Revision: https://reviews.freebsd.org/D24257 Notes: svn path=/head/; revision=360339
* Temporarily skip sys.netipsec.tunnel.empty.v{4,6} in CILi-Wen Hsu2020-04-221-0/+8
| | | | | | | | PR: 245832 Sponsored by: The FreeBSD Foundation Notes: svn path=/head/; revision=360204
* Update blake2 accelerated software tests to work after OCF refactoring.John Baldwin2020-04-221-14/+23
| | | | | | | | | | | | | | | | - Lookup device drivers to test by name instead of assuming that the software / hardware flags will select specific drivers. - Set the sysctl to permit software /dev/crypto requests when testing the accelerated software blake2 driver. PR: 245825 Reported by: lwhsu Reviewed by: cem, lwhsu Sponsored by: Chelsio Communications Differential Revision: https://reviews.freebsd.org/D24540 Notes: svn path=/head/; revision=360203