Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 1 Sep 2021 13:28:32 GMT
From:      Kristof Provost <kp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 4585629b74e7 - stable/12 - if_bridge: add ALTQ support
Message-ID:  <202109011328.181DSWrc047713@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by kp:

URL: https://cgit.FreeBSD.org/src/commit/?id=4585629b74e7d98a0712238baff02940b54d7dee

commit 4585629b74e7d98a0712238baff02940b54d7dee
Author:     Luiz Otavio O Souza <loos@FreeBSD.org>
AuthorDate: 2021-08-25 09:07:44 +0000
Commit:     Kristof Provost <kp@FreeBSD.org>
CommitDate: 2021-09-01 07:39:40 +0000

    if_bridge: add ALTQ support
    
    Similar to the recent addition of ALTQ support to if_vlan.
    
    Reviewed by:    donner
    Obtained from:  pfsense
    MFC after:      1 week
    Sponsored by:   Rubicon Communications, LLC ("Netgate")
    Differential Revision:  https://reviews.freebsd.org/D31675
    
    (cherry picked from commit eb680a63de1dbf5c974f483975dcb2c60ec6fa08)
---
 sys/net/if_bridge.c | 47 +++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 47 insertions(+)

diff --git a/sys/net/if_bridge.c b/sys/net/if_bridge.c
index 8b25c28eb554..22bcc84d44ef 100644
--- a/sys/net/if_bridge.c
+++ b/sys/net/if_bridge.c
@@ -301,6 +301,10 @@ static void	bridge_init(void *);
 static void	bridge_dummynet(struct mbuf *, struct ifnet *);
 static void	bridge_stop(struct ifnet *, int);
 static int	bridge_transmit(struct ifnet *, struct mbuf *);
+#ifdef ALTQ
+static void	bridge_altq_start(if_t);
+static int	bridge_altq_transmit(if_t, struct mbuf *);
+#endif
 static void	bridge_qflush(struct ifnet *);
 static struct mbuf *bridge_input(struct ifnet *, struct mbuf *);
 static int	bridge_output(struct ifnet *, struct mbuf *, struct sockaddr *,
@@ -732,7 +736,15 @@ bridge_clone_create(struct if_clone *ifc, int unit, caddr_t params)
 	if_initname(ifp, bridge_name, unit);
 	ifp->if_flags = IFF_BROADCAST | IFF_SIMPLEX | IFF_MULTICAST;
 	ifp->if_ioctl = bridge_ioctl;
+#ifdef ALTQ
+	ifp->if_start = bridge_altq_start;
+	ifp->if_transmit = bridge_altq_transmit;
+	IFQ_SET_MAXLEN(&ifp->if_snd, ifqmaxlen);
+	ifp->if_snd.ifq_drv_maxlen = 0;
+	IFQ_SET_READY(&ifp->if_snd);
+#else
 	ifp->if_transmit = bridge_transmit;
+#endif
 	ifp->if_qflush = bridge_qflush;
 	ifp->if_init = bridge_init;
 	ifp->if_type = IFT_BRIDGE;
@@ -838,6 +850,9 @@ bridge_clone_destroy(struct ifnet *ifp)
 	BRIDGE_LIST_UNLOCK();
 
 	bstp_detach(&sc->sc_stp);
+#ifdef ALTQ
+	IFQ_PURGE(&ifp->if_snd);
+#endif
 	NET_EPOCH_EXIT_ET(et);
 
 	ether_ifdetach(ifp);
@@ -2256,6 +2271,38 @@ bridge_transmit(struct ifnet *ifp, struct mbuf *m)
 	return (error);
 }
 
+#ifdef ALTQ
+static void
+bridge_altq_start(if_t ifp)
+{
+	struct ifaltq *ifq = &ifp->if_snd;
+	struct mbuf *m;
+
+	IFQ_LOCK(ifq);
+	IFQ_DEQUEUE_NOLOCK(ifq, m);
+	while (m != NULL) {
+		bridge_transmit(ifp, m);
+		IFQ_DEQUEUE_NOLOCK(ifq, m);
+	}
+	IFQ_UNLOCK(ifq);
+}
+
+static int
+bridge_altq_transmit(if_t ifp, struct mbuf *m)
+{
+	int err;
+
+	if (ALTQ_IS_ENABLED(&ifp->if_snd)) {
+		IFQ_ENQUEUE(&ifp->if_snd, m, err);
+		if (err == 0)
+			bridge_altq_start(ifp);
+	} else
+		err = bridge_transmit(ifp, m);
+
+	return (err);
+}
+#endif	/* ALTQ */
+
 /*
  * The ifp->if_qflush entry point for if_bridge(4) is no-op.
  */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202109011328.181DSWrc047713>