git: 137c43e13edb - stable/13 - dummynet: Remove unused code
Kristof Provost
kp at FreeBSD.org
Mon May 24 15:41:05 UTC 2021
The branch stable/13 has been updated by kp:
URL: https://cgit.FreeBSD.org/src/commit/?id=137c43e13edb1e9530c14697c4154ff6bca94414
commit 137c43e13edb1e9530c14697c4154ff6bca94414
Author: Kristof Provost <kp at FreeBSD.org>
AuthorDate: 2021-05-17 11:41:39 +0000
Commit: Kristof Provost <kp at FreeBSD.org>
CommitDate: 2021-05-24 15:38:22 +0000
dummynet: Remove unused code
We never set 'busy' and never dequeue from the pending mq. Remove this
code.
Reviewed by: ae
MFC after: 1 week
Sponsored by: Rubicon Communications, LLC ("Netgate")
Differential Revision: https://reviews.freebsd.org/D30313
(cherry picked from commit 02c44f40f984951fe34a1c5a43f40ff8147c52ca)
---
sys/netpfil/ipfw/ip_dn_io.c | 8 --------
sys/netpfil/ipfw/ip_dn_private.h | 7 -------
2 files changed, 15 deletions(-)
diff --git a/sys/netpfil/ipfw/ip_dn_io.c b/sys/netpfil/ipfw/ip_dn_io.c
index b439d2679f3c..4a65bd0ef798 100644
--- a/sys/netpfil/ipfw/ip_dn_io.c
+++ b/sys/netpfil/ipfw/ip_dn_io.c
@@ -870,14 +870,6 @@ dummynet_io(struct mbuf **m0, struct ip_fw_args *fwa)
/* we could actually tag outside the lock, but who cares... */
if (tag_mbuf(m, dir, fwa))
goto dropit;
- if (dn_cfg.busy) {
- /* if the upper half is busy doing something expensive,
- * lets queue the packet and move forward
- */
- mq_append(&dn_cfg.pending, m);
- m = *m0 = NULL; /* consumed */
- goto done; /* already active, nothing to do */
- }
/* XXX locate_flowset could be optimised with a direct ref. */
fs = dn_ht_find(dn_cfg.fshash, fs_id, 0, NULL);
if (fs == NULL)
diff --git a/sys/netpfil/ipfw/ip_dn_private.h b/sys/netpfil/ipfw/ip_dn_private.h
index 6e48bc5116a7..e6e699bf35b2 100644
--- a/sys/netpfil/ipfw/ip_dn_private.h
+++ b/sys/netpfil/ipfw/ip_dn_private.h
@@ -171,13 +171,6 @@ struct dn_parms {
int init_done;
- /* if the upper half is busy doing something long,
- * can set the busy flag and we will enqueue packets in
- * a queue for later processing.
- */
- int busy;
- struct mq pending;
-
#ifdef _KERNEL
/*
* This file is normally used in the kernel, unless we do
More information about the dev-commits-src-all
mailing list