git: 27b2aa49385d - main - altq: Remove unused arguments from altq_attach()
Kristof Provost
kp at FreeBSD.org
Mon Jan 25 19:59:04 UTC 2021
The branch main has been updated by kp:
URL: https://cgit.FreeBSD.org/src/commit/?id=27b2aa49385dc39959f772b76abec1601d4b9ab2
commit 27b2aa49385dc39959f772b76abec1601d4b9ab2
Author: Kristof Provost <kp at FreeBSD.org>
AuthorDate: 2021-01-23 09:37:13 +0000
Commit: Kristof Provost <kp at FreeBSD.org>
CommitDate: 2021-01-25 18:58:22 +0000
altq: Remove unused arguments from altq_attach()
Minor cleanup, no functional change.
Reviewed by: donner@
Differential Revision: https://reviews.freebsd.org/D28304
---
sys/net/altq/altq_cbq.c | 2 +-
sys/net/altq/altq_codel.c | 2 +-
sys/net/altq/altq_fairq.c | 2 +-
sys/net/altq/altq_hfsc.c | 2 +-
sys/net/altq/altq_priq.c | 2 +-
sys/net/altq/altq_subr.c | 12 ++----------
sys/net/altq/if_altq.h | 11 ++---------
7 files changed, 9 insertions(+), 24 deletions(-)
diff --git a/sys/net/altq/altq_cbq.c b/sys/net/altq/altq_cbq.c
index 6c6557725354..0e267fcb49e2 100644
--- a/sys/net/altq/altq_cbq.c
+++ b/sys/net/altq/altq_cbq.c
@@ -216,7 +216,7 @@ cbq_pfattach(struct pf_altq *a)
return (EINVAL);
s = splnet();
error = altq_attach(&ifp->if_snd, ALTQT_CBQ, a->altq_disc,
- cbq_enqueue, cbq_dequeue, cbq_request, NULL, NULL);
+ cbq_enqueue, cbq_dequeue, cbq_request);
splx(s);
return (error);
}
diff --git a/sys/net/altq/altq_codel.c b/sys/net/altq/altq_codel.c
index cca75a70b4c6..b9950ed94d2b 100644
--- a/sys/net/altq/altq_codel.c
+++ b/sys/net/altq/altq_codel.c
@@ -85,7 +85,7 @@ codel_pfattach(struct pf_altq *a)
return (EINVAL);
return (altq_attach(&ifp->if_snd, ALTQT_CODEL, a->altq_disc,
- codel_enqueue, codel_dequeue, codel_request, NULL, NULL));
+ codel_enqueue, codel_dequeue, codel_request));
}
int
diff --git a/sys/net/altq/altq_fairq.c b/sys/net/altq/altq_fairq.c
index a8f96daeed36..b4eeb1ddf9b4 100644
--- a/sys/net/altq/altq_fairq.c
+++ b/sys/net/altq/altq_fairq.c
@@ -142,7 +142,7 @@ fairq_pfattach(struct pf_altq *a)
return (EINVAL);
error = altq_attach(&ifp->if_snd, ALTQT_FAIRQ, a->altq_disc,
- fairq_enqueue, fairq_dequeue, fairq_request, NULL, NULL);
+ fairq_enqueue, fairq_dequeue, fairq_request);
return (error);
}
diff --git a/sys/net/altq/altq_hfsc.c b/sys/net/altq/altq_hfsc.c
index 2dbf571cdbb9..c3e84b345509 100644
--- a/sys/net/altq/altq_hfsc.c
+++ b/sys/net/altq/altq_hfsc.c
@@ -150,7 +150,7 @@ hfsc_pfattach(struct pf_altq *a)
return (EINVAL);
s = splnet();
error = altq_attach(&ifp->if_snd, ALTQT_HFSC, a->altq_disc,
- hfsc_enqueue, hfsc_dequeue, hfsc_request, NULL, NULL);
+ hfsc_enqueue, hfsc_dequeue, hfsc_request);
splx(s);
return (error);
}
diff --git a/sys/net/altq/altq_priq.c b/sys/net/altq/altq_priq.c
index e0c3e0f6ee80..f0627e2611a7 100644
--- a/sys/net/altq/altq_priq.c
+++ b/sys/net/altq/altq_priq.c
@@ -87,7 +87,7 @@ priq_pfattach(struct pf_altq *a)
return (EINVAL);
s = splnet();
error = altq_attach(&ifp->if_snd, ALTQT_PRIQ, a->altq_disc,
- priq_enqueue, priq_dequeue, priq_request, NULL, NULL);
+ priq_enqueue, priq_dequeue, priq_request);
splx(s);
return (error);
}
diff --git a/sys/net/altq/altq_subr.c b/sys/net/altq/altq_subr.c
index b35637820a47..dff589607ac8 100644
--- a/sys/net/altq/altq_subr.c
+++ b/sys/net/altq/altq_subr.c
@@ -168,15 +168,13 @@ altq_lookup(name, type)
}
int
-altq_attach(ifq, type, discipline, enqueue, dequeue, request, clfier, classify)
+altq_attach(ifq, type, discipline, enqueue, dequeue, request)
struct ifaltq *ifq;
int type;
void *discipline;
int (*enqueue)(struct ifaltq *, struct mbuf *, struct altq_pktattr *);
struct mbuf *(*dequeue)(struct ifaltq *, int);
int (*request)(struct ifaltq *, int, void *);
- void *clfier;
- void *(*classify)(void *, struct mbuf *, int);
{
IFQ_LOCK(ifq);
if (!ALTQ_IS_READY(ifq)) {
@@ -189,8 +187,6 @@ altq_attach(ifq, type, discipline, enqueue, dequeue, request, clfier, classify)
ifq->altq_enqueue = enqueue;
ifq->altq_dequeue = dequeue;
ifq->altq_request = request;
- ifq->altq_clfier = clfier;
- ifq->altq_classify = classify;
ifq->altq_flags &= (ALTQF_CANTCHANGE|ALTQF_ENABLED);
IFQ_UNLOCK(ifq);
return 0;
@@ -220,8 +216,6 @@ altq_detach(ifq)
ifq->altq_enqueue = NULL;
ifq->altq_dequeue = NULL;
ifq->altq_request = NULL;
- ifq->altq_clfier = NULL;
- ifq->altq_classify = NULL;
ifq->altq_flags &= ALTQF_CANTCHANGE;
IFQ_UNLOCK(ifq);
@@ -250,8 +244,6 @@ altq_enable(ifq)
ASSERT(ifq->ifq_len == 0);
ifq->ifq_drv_maxlen = 0; /* disable bulk dequeue */
ifq->altq_flags |= ALTQF_ENABLED;
- if (ifq->altq_clfier != NULL)
- ifq->altq_flags |= ALTQF_CLASSIFY;
splx(s);
IFQ_UNLOCK(ifq);
@@ -273,7 +265,7 @@ altq_disable(ifq)
s = splnet();
IFQ_PURGE_NOLOCK(ifq);
ASSERT(ifq->ifq_len == 0);
- ifq->altq_flags &= ~(ALTQF_ENABLED|ALTQF_CLASSIFY);
+ ifq->altq_flags &= ~(ALTQF_ENABLED);
splx(s);
IFQ_UNLOCK(ifq);
diff --git a/sys/net/altq/if_altq.h b/sys/net/altq/if_altq.h
index 8d0463e09b3e..e0237ce77fda 100644
--- a/sys/net/altq/if_altq.h
+++ b/sys/net/altq/if_altq.h
@@ -63,10 +63,6 @@ struct ifaltq {
struct mbuf *(*altq_dequeue)(struct ifaltq *, int);
int (*altq_request)(struct ifaltq *, int, void *);
- /* classifier fields */
- void *altq_clfier; /* classifier-specific use */
- void *(*altq_classify)(void *, struct mbuf *, int);
-
/* token bucket regulator */
struct tb_regulator *altq_tbr;
@@ -127,7 +123,7 @@ struct tb_regulator {
/* if_altqflags */
#define ALTQF_READY 0x01 /* driver supports alternate queueing */
#define ALTQF_ENABLED 0x02 /* altq is in use */
-#define ALTQF_CLASSIFY 0x04 /* classify packets */
+/* ALTQF_CLASSIFY 0x04 obsolete classify packets */
#define ALTQF_CNDTNING 0x08 /* altq traffic conditioning is enabled */
#define ALTQF_DRIVER1 0x40 /* driver specific */
@@ -147,7 +143,6 @@ struct tb_regulator {
#else
#define ALTQ_IS_ENABLED(ifq) 0
#endif
-#define ALTQ_NEEDS_CLASSIFY(ifq) ((ifq)->altq_flags & ALTQF_CLASSIFY)
#define ALTQ_IS_CNDTNING(ifq) ((ifq)->altq_flags & ALTQF_CNDTNING)
#define ALTQ_SET_CNDTNING(ifq) ((ifq)->altq_flags |= ALTQF_CNDTNING)
@@ -169,9 +164,7 @@ extern int altq_attach(struct ifaltq *, int, void *,
int (*)(struct ifaltq *, struct mbuf *,
struct altq_pktattr *),
struct mbuf *(*)(struct ifaltq *, int),
- int (*)(struct ifaltq *, int, void *),
- void *,
- void *(*)(void *, struct mbuf *, int));
+ int (*)(struct ifaltq *, int, void *));
extern int altq_detach(struct ifaltq *);
extern int altq_enable(struct ifaltq *);
extern int altq_disable(struct ifaltq *);
More information about the dev-commits-src-main
mailing list