Skip to content

Add kmod support on FreeBSD - #192

Open
ngie-eign wants to merge 1 commit into
freebsd:masterfrom
ngie-eign:add-kmod-support
Open

Add kmod support on FreeBSD#192
ngie-eign wants to merge 1 commit into
freebsd:masterfrom
ngie-eign:add-kmod-support

Conversation

@ngie-eign

Copy link
Copy Markdown
Contributor

This change adds kmod querying support for FreeBSD, allowing tests to require specific kmods using the require.kmods metadata in testcases, or the atf_require_kmod function in atf-c(3), atf-c++(3), and atf-sh(1).

This functionality has been stubbed out and is expected to fail/not be available on all other OSes, e.g., Linux, macOS, etc.

A corresponding paired change is required in order to make this functionality work with kyua. That change is coming soon.

This is a forward port of kmod support from [83a1ee57][1]. The original implementation was by Dag-Erling Smørgrav.

Co-Authored By: Dag-Erling Smørgrav das@FreeBSD.org
Signed-off-by: Enji Cooper ngie@FreeBSD.org
[1]: https://cgit.freebsd.org/src/commit/?id=83a1ee578c9d1ab7013e997289c7cd470c0e6902

This change adds kmod querying support for FreeBSD, allowing tests to
require specific kmods using the `require.kmods` metadata in testcases,
or the `atf_require_kmod` function in atf-c(3), atf-c++(3), and
atf-sh(1).

This functionality has been stubbed out and is expected to fail/not be
available on all other OSes, e.g., Linux, macOS, etc.

A corresponding paired change is required in order to make this
functionality work with kyua. That change is coming soon.

This is a forward port of kmod support from [freebsd/atf@83a1ee57][1]. The
original implementation was by Dag-Erling Smørgrav.

Co-Authored By:	Dag-Erling Smørgrav <das@FreeBSD.org>
Signed-off-by:	Enji Cooper <ngie@FreeBSD.org>
[1]: https://cgit.freebsd.org/src/commit/?id=83a1ee578c9d1ab7013e997289c7cd470c0e6902
@ngie-eign
ngie-eign requested a review from dag-erling September 4, 2026 03:59
@ngie-eign

Copy link
Copy Markdown
Contributor Author

@dag-erling : want to get your feedback since you provided the initial diff.

  • I added some __FreeBSD__ conditionals around the functionality.
  • I added some tests (the commit in FreeBSD didn't have tests).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant