dummynet: Remove unused code

[kp]

Summary

We never set 'busy' and never dequeue from the pending mq. Remove this
code.

Sponsored by: Rubicon Communications, LLC ("Netgate")

[ae]

Just want to note, I'm sorry for long delay, I moved temporary to another project and dummynet overhaul is not done yet. But I plan to publish the code somewhere on guthub.

[kp]

In D30313#680789, @ae wrote:

Just want to note, I'm sorry for long delay, I moved temporary to another project and dummynet overhaul is not done yet. But I plan to publish the code somewhere on guthub.

That'd be useful, yes, even if it's not in a working - or even compiling - state right now.

My immediate interest is in the bits of that work to move dummynet configuration into a separate utility, because that's a requirement for dummynet-in-pf. The rest of it is valuable as well, but more of a long term interest.

sys/netpfil/ipfw/ip_dn_io.c

cap_openlog(cap_channel_t *chan, const char *ident, int logopt, int facility)
 
         nvl = nvlist_create(0);
         nvlist_add_string(nvl, "cmd", "openlog");
         if (ident != NULL) {
                 nvlist_add_string(nvl, "ident", ident);
         }
         nvlist_add_number(nvl, "logopt", logopt);
         nvlist_add_number(nvl, "facility", facility);
+        if (logopt & LOG_PERROR) {
+                nvlist_add_descriptor(nvl, "stderr", STDERR_FILENO);
+        }
         nvl = cap_xfer_nvlist(chan, nvl);
         if (nvl == NULL) {
                 return;
         }
         nvlist_destroy(nvl);
 }
 
 void
cap_setlogmask(cap_channel_t *chan, int maskpri)
         return (omask);
 }
 
 /*
  * Service functions.
  */
 
 static char *LogTag;
+static int prev_stderr = -1;
 
 static void
 slog_vsyslog(const nvlist_t *limits __unused, const nvlist_t *nvlin,
     nvlist_t *nvlout __unused)
 {
 
         syslog(nvlist_get_number(nvlin, "priority"), "%s",
             nvlist_get_string(nvlin, "message"));
 }
 
 static void
 slog_openlog(const nvlist_t *limits __unused, const nvlist_t *nvlin,
     nvlist_t *nvlout __unused)
 {
         const char *ident;
+        uint64_t logopt;
+        int stderr_fd;
 
         ident = dnvlist_get_string(nvlin, "ident", NULL);
         if (ident != NULL) {
                 free(LogTag);
                 LogTag = strdup(ident);
         }
 
-        openlog(LogTag, nvlist_get_number(nvlin, "logopt"),
-            nvlist_get_number(nvlin, "facility"));
+        logopt = nvlist_get_number(nvlin, "logopt");
+        if (logopt & LOG_PERROR) {
+                stderr_fd = dnvlist_get_descriptor(nvlin, "stderr", -1);
+                if (prev_stderr == -1)
+                        prev_stderr = dup(STDERR_FILENO);
+                if (prev_stderr != -1)
+                        (void)dup2(stderr_fd, STDERR_FILENO);
+        } else if (prev_stderr != -1) {
+                (void)dup2(prev_stderr, STDERR_FILENO);
+                close(prev_stderr);
+                prev_stderr = -1;
         }
+        openlog(LogTag, logopt, nvlist_get_number(nvlin, "facility"));
+}
 
 static void
 slog_closelog(const nvlist_t *limits __unused, const nvlist_t *nvlin __unused,
     nvlist_t *nvlout __unused)
 {
 
         closelog();
 
         free(LogTag);
         LogTag = NULL;
+
+        if (prev_stderr != -1) {
+                (void)dup2(prev_stderr, STDERR_FILENO);
+                close(prev_stderr);
+                prev_stderr = -1;
         }
+}
 
 static void
 slog_setlogmask(const nvlist_t *limits __unused, const nvlist_t *nvlin,
     nvlist_t *nvlout)
 {
         int omask;
 
         omask = setlogmask(nvlist_get_number(nvlin, "maskpri"));
if (strcmp(cmd, "vsyslog") == 0) {
                 slog_setlogmask(limits, nvlin, nvlout);
         } else {
                 return (EINVAL);
         }
 
         return (0);
 }
 
-CREATE_SERVICE("system.syslog", NULL, syslog_command, CASPER_SERVICE_STDIO);
+CREATE_SERVICE("system.syslog", NULL, syslog_command, 0);

sys/netpfil/ipfw/ip_dn_private.h

cap_openlog(cap_channel_t *chan, const char *ident, int logopt, int facility)
 
         nvl = nvlist_create(0);
         nvlist_add_string(nvl, "cmd", "openlog");
         if (ident != NULL) {
                 nvlist_add_string(nvl, "ident", ident);
         }
         nvlist_add_number(nvl, "logopt", logopt);
         nvlist_add_number(nvl, "facility", facility);
+        if (logopt & LOG_PERROR) {
+                nvlist_add_descriptor(nvl, "stderr", STDERR_FILENO);
+        }
         nvl = cap_xfer_nvlist(chan, nvl);
         if (nvl == NULL) {
                 return;
         }
         nvlist_destroy(nvl);
 }
 
 void
cap_setlogmask(cap_channel_t *chan, int maskpri)
         return (omask);
 }
 
 /*
  * Service functions.
  */
 
 static char *LogTag;
+static int prev_stderr = -1;
 
 static void
 slog_vsyslog(const nvlist_t *limits __unused, const nvlist_t *nvlin,
     nvlist_t *nvlout __unused)
 {
 
         syslog(nvlist_get_number(nvlin, "priority"), "%s",
             nvlist_get_string(nvlin, "message"));
 }
 
 static void
 slog_openlog(const nvlist_t *limits __unused, const nvlist_t *nvlin,
     nvlist_t *nvlout __unused)
 {
         const char *ident;
+        uint64_t logopt;
+        int stderr_fd;
 
         ident = dnvlist_get_string(nvlin, "ident", NULL);
         if (ident != NULL) {
                 free(LogTag);
                 LogTag = strdup(ident);
         }
 
-        openlog(LogTag, nvlist_get_number(nvlin, "logopt"),
-            nvlist_get_number(nvlin, "facility"));
+        logopt = nvlist_get_number(nvlin, "logopt");
+        if (logopt & LOG_PERROR) {
+                stderr_fd = dnvlist_get_descriptor(nvlin, "stderr", -1);
+                if (prev_stderr == -1)
+                        prev_stderr = dup(STDERR_FILENO);
+                if (prev_stderr != -1)
+                        (void)dup2(stderr_fd, STDERR_FILENO);
+        } else if (prev_stderr != -1) {
+                (void)dup2(prev_stderr, STDERR_FILENO);
+                close(prev_stderr);
+                prev_stderr = -1;
         }
+        openlog(LogTag, logopt, nvlist_get_number(nvlin, "facility"));
+}
 
 static void
 slog_closelog(const nvlist_t *limits __unused, const nvlist_t *nvlin __unused,
     nvlist_t *nvlout __unused)
 {
 
         closelog();
 
         free(LogTag);
         LogTag = NULL;
+
+        if (prev_stderr != -1) {
+                (void)dup2(prev_stderr, STDERR_FILENO);
+                close(prev_stderr);
+                prev_stderr = -1;
         }
+}
 
 static void
 slog_setlogmask(const nvlist_t *limits __unused, const nvlist_t *nvlin,
     nvlist_t *nvlout)
 {
         int omask;
 
         omask = setlogmask(nvlist_get_number(nvlin, "maskpri"));
if (strcmp(cmd, "vsyslog") == 0) {
                 slog_setlogmask(limits, nvlin, nvlout);
         } else {
                 return (EINVAL);
         }
 
         return (0);
 }
 
-CREATE_SERVICE("system.syslog", NULL, syslog_command, CASPER_SERVICE_STDIO);
+CREATE_SERVICE("system.syslog", NULL, syslog_command, 0);