src/tests/sys/netmap, branch releng/14.4FreeBSD source treehttps://cgit-dev.freebsd.org/src/atom?h=releng%2F14.42025-04-06T18:05:05Znetmap: fix unit tests2025-04-06T18:05:05ZVincenzo Maffionevmaffione@FreeBSD.org2023-12-29T07:46:46Zurn:sha1:984e3d813e5f1d2a0938bdf5c03712de0ab1099b
After ad874544d9f018bf8eef4053b5ca7b856c4674cb, interface name
validation has been removed, resulting in two unit tests failures.
Drop the failing tests since they no longer apply.
Reported by: markj
(cherry picked from commit ee5804da116f2107451c8b4376b69b3a64a630e8)
Remove $FreeBSD$: one-line sh pattern2023-08-16T17:55:03ZWarner Loshimp@FreeBSD.org2023-08-16T17:55:03Zurn:sha1:d0b2dbfa0ecf2bbc9709efc5e20baf8e4b44bbbf
Remove /^\s*#[#!]?\s*\$FreeBSD\$.*$\n/
Remove $FreeBSD$: two-line .h pattern2023-08-16T17:54:16ZWarner Loshimp@FreeBSD.org2023-08-16T17:54:16Zurn:sha1:b3e7694832e81d7a904a10f525f8797b753bf0d3
Remove /^\s*\*\n \*\s+\$FreeBSD\$$\n/
spdx: The BSD-2-Clause-FreeBSD identifier is obsolete, drop -FreeBSD2023-05-12T16:44:03ZWarner Loshimp@FreeBSD.org2023-05-10T15:40:58Zurn:sha1:4d846d260e2b9a3d4d0a701462568268cbfe7a5b
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
netmap: fix copyin/copyout of nmreq options list2023-03-21T23:23:18ZVincenzo Maffionevmaffione@FreeBSD.org2023-03-21T23:23:18Zurn:sha1:e2a431a0ffb6894220bdf5d8fc2ca2d0ca316e85
The previous code unsuccesfully attempted to report a precise error for
each option in the user list. Moreover, commit 253b2ec199b broke some
ctrl-api-test (see https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=260547).
With this patch we bail out as soon as an unrecoverable error is detected and
we properly check for copy boundaries. EOPNOTSUPP no longer immediately
returns an error, so that any other option in the list may be examined
by the caller code and a precise report of the (un)supported options can
be returned to the user.
With this patch, all ctrl-api-test unit tests pass again.
PR: 260547
Submitted by: giuseppe.lettieri@unipi.it
Reviewed by: vmaffione
MFC after: 14 days
netmap tests: Reset ctx->nmctx to NULL in nmreq_parsing.2022-12-05T00:29:25ZJohn Baldwinjhb@FreeBSD.org2022-12-05T00:29:25Zurn:sha1:f4db390553ba9db752865f5a42c36b062fdb03f2
This avoids leaking a pointer to the on-stack test_nmctx which
triggers a -Wdangling-pointer warning from GCC.
Reviewed by: imp, emaste
Differential Revision: https://reviews.freebsd.org/D37536
netmap: update unit tests with libnetmap tests2021-04-02T14:39:30ZVincenzo Maffionevmaffione@FreeBSD.org2021-04-02T14:39:30Zurn:sha1:36d6e65722ea515bf2d122d6e69096a5ff620a92netmap: several typo fixes2021-04-02T07:01:20ZVincenzo Maffionevmaffione@FreeBSD.org2021-04-02T07:01:20Zurn:sha1:45c67e8f6b56b9744f01142747fadf291fe3fad2
No functional changes intended.
Do a sweep and remove most WARNS=6 settings2020-10-01T01:10:51ZKyle Evanskevans@FreeBSD.org2020-10-01T01:10:51Zurn:sha1:7cc42f6d25ef2e19059d088fa7d4853fe9afefb5
Repeating the default WARNS here makes it slightly more difficult to
experiment with default WARNS changes, e.g. if we did something absolutely
bananas and introduced a WARNS=7 and wanted to try lifting the default to
that.
Drop most of them; there is one in the blake2 kernel module, but I suspect
it should be dropped -- the default WARNS in the rest of the build doesn't
currently apply to kernel modules, and I haven't put too much thought into
whether it makes sense to make it so.
netmap: add test cases for multiple host rings2019-04-13T12:50:47ZVincenzo Maffionevmaffione@FreeBSD.org2019-04-13T12:50:47Zurn:sha1:4f6858e8a941a040ae8df6932b4a12a12f4bd386
Extend the netmap unit tests with new test cases for the multiple-host-rings
feature introduced by r345269.
MFC after: 1 week