Retire devctl_notify_f()

devctl_notify_f isn't needed, so retire it. The flags argument is now
unused, so rather than keep it around, retire it. Convert all old
users of it to devctl_notify(). This path no longer sleeps, so is safe
to call from any context. Since it doesn't sleep, it doesn't need to
know if it is OK to sleep or not.

Reviewed by: markj@
Differential Revision: https://reviews.freebsd.org/D26140

1 files changed, 2 insertions, 9 deletions

diff --git a/sys/kern/subr_bus.c b/sys/kern/subr_bus.c
index 2232799ebb92..503e73ff59b0 100644
--- a/sys/kern/subr_bus.c
+++ b/sys/kern/subr_bus.c
@@ -651,8 +651,8 @@ devctl_queue(struct dev_event_info *dei)
  * @brief Send a 'notification' to userland, using standard ways
  */
 void
-devctl_notify_f(const char *system, const char *subsystem, const char *type,
-    const char *data, int flags __unused)
+devctl_notify(const char *system, const char *subsystem, const char *type,
+    const char *data)
 {
 	struct dev_event_info *dei;
 	struct sbuf sb;
@@ -679,13 +679,6 @@ devctl_notify_f(const char *system, const char *subsystem, const char *type,
 		devctl_queue(dei);
 }
 
-void
-devctl_notify(const char *system, const char *subsystem, const char *type,
-    const char *data)
-{
-	devctl_notify_f(system, subsystem, type, data, M_NOWAIT);
-}
-
 /*
  * Common routine that tries to make sending messages as easy as possible.
  * We allocate memory for the data, copy strings into that, but do not