Page MenuHomeFreeBSD
Differential D53617

virtual_oss(8): Use kldload(2) instead of system(3)
ClosedPublic
Actions

Authored by christos on Nov 6 2025, 3:16 PM.
Tags
None
Referenced Files
F163293531: D53617.id.diff
Tue, Jul 21, 7:36 PM
F163285245: D53617.diff
Tue, Jul 21, 5:58 PM
Unknown Object (File)
Mon, Jul 20, 4:26 PM
Unknown Object (File)
Sat, Jul 18, 1:08 PM
Unknown Object (File)
Wed, Jul 15, 6:54 AM
Unknown Object (File)
Wed, Jul 15, 6:53 AM
Unknown Object (File)
Thu, Jul 9, 7:34 PM
Unknown Object (File)
Wed, Jul 8, 1:06 PM
View All 56 Files
Subscribers
imp

Details

Reviewers
markj
emaste
imp
Commits
rG4804bb0d4588: virtual_oss(8): Use kldload(2) instead of system(3)
rG7bedc4634d89: virtual_oss(8): Use kldload(2) instead of system(3)
Summary

Sponsored by: The FreeBSD Foundation
MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 68460
Build 65343: arc lint + arc unit

Event Timeline

christos created this revision.Nov 6 2025, 3:16 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 6 2025, 3:16 PM
christos requested review of this revision.Nov 6 2025, 3:16 PM
Harbormaster completed remote builds in B68460: Diff 165957.Nov 6 2025, 3:16 PM
markj accepted this revision.Nov 6 2025, 3:38 PM
This revision is now accepted and ready to land.Nov 6 2025, 3:38 PM
emaste accepted this revision.Nov 6 2025, 3:42 PM

Tweak error text - let's be explicit since we no longer have the "kldload" text to give a hint that it's a kernel module

usr.sbin/virtual_oss/virtual_oss/main.c
2545
christos marked an inline comment as done.Nov 6 2025, 4:02 PM
imp accepted this revision.Nov 6 2025, 5:09 PM

whatever you think best for that message. This is good otherwise.

usr.sbin/virtual_oss/virtual_oss/main.c
2545

or "cuse.ko"

emaste added inline comments.Nov 6 2025, 5:10 PM
usr.sbin/virtual_oss/virtual_oss/main.c
2545

Yeah that would be good too

christos marked 2 inline comments as done.Nov 7 2025, 1:39 PM
Closed by commit rG7bedc4634d89: virtual_oss(8): Use kldload(2) instead of system(3) (authored by christos). · Explain WhyNov 11 2025, 12:14 PM
This revision was automatically updated to reflect the committed changes.
christos added a commit: rG7bedc4634d89: virtual_oss(8): Use kldload(2) instead of system(3).
christos added a commit: rG4804bb0d4588: virtual_oss(8): Use kldload(2) instead of system(3).Nov 18 2025, 12:53 AM

Revision Contents
Changeset List

PathSize
usr.sbin/
virtual_oss/
virtual_oss/
9 lines

Diff 165957

View Options

usr.sbin/virtual_oss/virtual_oss/main.c

Loading...