| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
| |
PR: 247954
X-MFC with: r363402
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=363423
|
| |
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
| |
Reviewed by: markj
Sponsored by: Netflix
Differential Revision: https://reviews.freebsd.org/D25431
Notes:
svn path=/head/; revision=363377
|
| |
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
MFC after: 1 week
Sponsored by: Rubicon Communications, LLC (Netgate)
Notes:
svn path=/head/; revision=363181
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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 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 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
|
| |
|
|
|
|
|
| |
PR: kern/247729
Notes:
svn path=/head/; revision=362909
|
| |
|
|
|
|
|
|
| |
PR: 247729
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=362904
|
| |
|
|
|
|
|
|
|
|
| |
on armv7.
MFC after: 2 weeks
Sponsored by: DARPA
Notes:
svn path=/head/; revision=362575
|
| |
|
|
|
|
|
|
| |
PR: 244737
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=362573
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
PR: 244168
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=362072
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
PR: 246737
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361623
|
| |
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
| |
requires it.
MFC after: 2 weeks
Sponsored by: DARPA
Notes:
svn path=/head/; revision=361268
|
| |
|
|
|
|
|
|
|
|
| |
they need to be able to run net_receiver.py.
MFC after: 2 weeks
Sponsored by: DARPA
Notes:
svn path=/head/; revision=361267
|
| |
|
|
|
|
|
|
| |
PR: 246560
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361241
|
| |
|
|
|
|
|
|
|
|
| |
- sys.netinet.so_reuseport_lb_test.basic_ipv6
PR: 246560
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361237
|
| |
|
|
|
|
|
|
| |
PR: 244053
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361229
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
| |
Should be fixed by r360613
PR: 244737
Reported by: lwhsu
Notes:
svn path=/head/; revision=361129
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
PR: 246443
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361019
|
| |
|
|
|
|
|
|
| |
PR: 244229
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361018
|
| |
|
|
|
|
|
|
|
|
| |
This case timed out so often
PR: 246443
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=361017
|
| |
|
|
|
|
|
|
|
| |
Reviewed by: kp
MFC after: 2 weeks
Differential Revision: https://reviews.freebsd.org/D24809
Notes:
svn path=/head/; revision=360906
|
| |
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
| |
Differential Revision: https://reviews.freebsd.org/D24684
Notes:
svn path=/head/; revision=360685
|
| |
|
|
|
|
|
|
| |
PR: 245832
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=360626
|
| |
|
|
|
|
|
| |
MFC after: 2 weeks
Notes:
svn path=/head/; revision=360613
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
PR: 245931
Submitted by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=360509
|
| |
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
|
|
|
|
| |
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
|
| |
|
|
|
|
|
|
| |
PR: 245832
Sponsored by: The FreeBSD Foundation
Notes:
svn path=/head/; revision=360204
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- 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
|