Page MenuHomeFreeBSD

kern: add devctl_notifyf()
AcceptedPublic

Authored by kevans on Mar 9 2022, 5:24 AM.
Tags
None
Referenced Files
F123653219: D34504.id103670.diff
Thu, Jul 17, 8:15 AM
Unknown Object (File)
Thu, Jul 10, 11:22 AM
Unknown Object (File)
Fri, Jul 4, 2:36 PM
Unknown Object (File)
Mon, Jun 23, 5:46 PM
Unknown Object (File)
Fri, Jun 20, 6:04 AM
Unknown Object (File)
Fri, Jun 20, 3:34 AM
Unknown Object (File)
Fri, Jun 20, 12:59 AM
Unknown Object (File)
Wed, Jun 18, 8:38 PM

Details

Reviewers
imp
bapt
Group Reviewers
manpages
Summary

This version is identical to devctl_notify(), except it fits slightly
less trivial needs by taking a data format string and varargs to go
with it. This lets us avoid extra allocations on top in various
callers, making them a bit less complicated.

Diff Detail

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

Event Timeline

debdrup added a subscriber: debdrup.

Manual page changes look good to me.

This revision is now accepted and ready to land.Mar 9 2022, 3:56 PM

Other than that, LGTM.

share/man/man9/devctl_notify.9
38

"userland", I think? I don't remember seeing it split.

Rebase past nlsysevent's devctl tap

Alright, hear me out, this isn't the ugliest thing I've ever written. This does
result in a functional difference for nlsysevent, in that we may not tap out an
nlsysevent if we couldn't allocate the sbuf/dei bits for whatever reason. I
don't think it's necessarily a bad difference, but it's a notable one.

This revision now requires review to proceed.May 7 2025, 1:05 AM

Tap in bapt, too, since this could slightly change some nlsysevent semantics in more edgey cases.

This revision is now accepted and ready to land.May 7 2025, 6:56 AM