From nobody Tue Sep 27 13:34:57 2022 X-Original-To: dev-commits-src-main@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4McLGB1t60z4d1cy; Tue, 27 Sep 2022 13:34:58 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4McLGB1Jbyz3Wft; Tue, 27 Sep 2022 13:34:58 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1664285698; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=d1sJuPNgw64pCzshHGQyF2Sy8wPtrRJghBJ1PB+QriQ=; b=DqvjWojjCAZ+HDV9pbIvImhCQHzpPmsWdzD1m8LpYBkj0n6pBqEz7QZfub7aIrT8+rbavI fEDnfNWQQE8aRIrhCAbsE6j+ZcN+Nu2NG7Vy6D0RweAC5q4kKP9n2VuSqlBwEF0Sm8xkH4 /ek0cF//V5IDaKiKtJhs529K4Phq4tWTaf84otZXOTdBx+4HYpg2AHKvjLxHSjU2kXPEXb fATi7EqYkkOZm0cgbF83k0iTnE7jeech9a914RLg/uuN+3tMgMHn2Wfg6eTFregWp1TPHi NbA+ZpwLG5aZE28eLUwGqntUhY4imjud9tFXvIu7UmG2zo+lKNfvt9RVhDix+Q== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4McLG96vtGz17Zf; Tue, 27 Sep 2022 13:34:57 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 28RDYvQj090849; Tue, 27 Sep 2022 13:34:57 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 28RDYv39090848; Tue, 27 Sep 2022 13:34:57 GMT (envelope-from git) Date: Tue, 27 Sep 2022 13:34:57 GMT Message-Id: <202209271334.28RDYv39090848@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org From: "Alexander V. Chernikov" Subject: git: 04a32b802ec7 - main - if_epair: refactor interface creation and enqueue code. List-Id: Commit messages for the main branch of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-main@freebsd.org X-BeenThere: dev-commits-src-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: melifaro X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: 04a32b802ec718f8b9277f79e9b713d7aa06002c Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1664285698; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=d1sJuPNgw64pCzshHGQyF2Sy8wPtrRJghBJ1PB+QriQ=; b=ogLjXw4owNFOOstZtGhhyyx3Qtfd4Vi09iFrXVuapoJP3xq8ZsfxxOemN5zNZ+uNhbWm23 xJkoCrzfefWwo639loK7Q8ZDtylAK+SxOtjW30UH2fHyRjF7hdNEl/RpPgmYWwA5jtOUxz DITLnFbWQMGEmRg5yFhn+gj7Q3ib5ghWFM8gmBjDUcCY9EjGRBGh/4OyktePI1nFb80vr7 d7peeTKvVBFl2jLrisk+P5MJHYKtKjHQfKY5ylhmfIWnOwSFklfzG3+hTFee1O57BNBtcp vfIKNiNjfjIwOM+uVdx5Cupv0cvKibwzFHHU4F7DQqUNx5lUAVw8O9FjGhGPjg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1664285698; a=rsa-sha256; cv=none; b=eJnF4hv3gIfmoo43tsAw/dfJo1+sCSzvhFg/W/fnjbt3NqEmJoe0VHDFuiyqNrK+cPYuw5 RG4f0s4BnRw9p1jMEUGbKOUpFFVn5O2iv1rt981Vn/xiJZpy+s5Xm2L0Ppl0OFhYNzqDEM oLHOKjBGF7j9d0B7Hp5MQDKRimpAEYZgoQxlWzLDU/AeGv5zsvE9UP4ytyIOVWsVtTPDj0 dlw9C1JInxBGKElJnC08HpUcgFgLEVPiW6VTYkEAQ79aqinbaf8C7xHxpYeYFEKKAWCams s+xt7/ZSYhP41RAraZJ8/MezjcbnTd5WMbSGJ41IH7rQ8FBWLozq5wbubKtRqw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch main has been updated by melifaro: URL: https://cgit.FreeBSD.org/src/commit/?id=04a32b802ec718f8b9277f79e9b713d7aa06002c commit 04a32b802ec718f8b9277f79e9b713d7aa06002c Author: Alexander V. Chernikov AuthorDate: 2022-09-27 13:24:29 +0000 Commit: Alexander V. Chernikov CommitDate: 2022-09-27 13:34:19 +0000 if_epair: refactor interface creation and enqueue code. * Factor out queue selection (epair_select_queue()) and mbuf preparation (epair_prepare_mbuf()) from epair_menq(). It simplifies epair_menq() implementation and reduces the amount of dependencies on the neighbouring epair. * Use dedicated epair_set_state() instead of 2-lines copy-paste * Factor out unit selection code (epair_handle_unit()) from epair_clone_create(). It simplifies the clone creation logic. Reviewed By: kp Differential Revision: https://reviews.freebsd.org/D36689 --- sys/net/if_epair.c | 165 +++++++++++++++++++++++++++++++++-------------------- 1 file changed, 102 insertions(+), 63 deletions(-) diff --git a/sys/net/if_epair.c b/sys/net/if_epair.c index 6bf7481065cd..04b96ea49ec7 100644 --- a/sys/net/if_epair.c +++ b/sys/net/if_epair.c @@ -184,39 +184,13 @@ epair_tx_start_deferred(void *arg, int pending) if_rele(sc->ifp); } -static int -epair_menq(struct mbuf *m, struct epair_softc *osc) +static struct epair_queue * +epair_select_queue(struct epair_softc *sc, const struct mbuf *m) { - struct ifnet *ifp, *oifp; - int len, ret; - int ridx; - short mflags; - struct epair_queue *q = NULL; uint32_t bucket; #ifdef RSS struct ether_header *eh; -#endif - - /* - * I know this looks weird. We pass the "other sc" as we need that one - * and can get both ifps from it as well. - */ - oifp = osc->ifp; - ifp = osc->oifp; - - M_ASSERTPKTHDR(m); - epair_clear_mbuf(m); - if_setrcvif(m, oifp); - M_SETFIB(m, oifp->if_fib); - - /* Save values as once the mbuf is queued, it's not ours anymore. */ - len = m->m_pkthdr.len; - mflags = m->m_flags; - - MPASS(m->m_nextpkt == NULL); - MPASS((m->m_pkthdr.csum_flags & CSUM_SND_TAG) == 0); -#ifdef RSS ret = rss_m2bucket(m, &bucket); if (ret) { /* Actually hash the packet. */ @@ -238,11 +212,47 @@ epair_menq(struct mbuf *m, struct epair_softc *osc) break; } } - bucket %= osc->num_queues; + bucket %= sc->num_queues; #else bucket = 0; #endif - q = &osc->queues[bucket]; + return (&sc->queues[bucket]); +} + +static void +epair_prepare_mbuf(struct mbuf *m, struct ifnet *src_ifp) +{ + M_ASSERTPKTHDR(m); + epair_clear_mbuf(m); + if_setrcvif(m, src_ifp); + M_SETFIB(m, src_ifp->if_fib); + + MPASS(m->m_nextpkt == NULL); + MPASS((m->m_pkthdr.csum_flags & CSUM_SND_TAG) == 0); +} + +static void +epair_menq(struct mbuf *m, struct epair_softc *osc) +{ + struct ifnet *ifp, *oifp; + int len, ret; + int ridx; + short mflags; + + /* + * I know this looks weird. We pass the "other sc" as we need that one + * and can get both ifps from it as well. + */ + oifp = osc->ifp; + ifp = osc->oifp; + + epair_prepare_mbuf(m, oifp); + + /* Save values as once the mbuf is queued, it's not ours anymore. */ + len = m->m_pkthdr.len; + mflags = m->m_flags; + + struct epair_queue *q = epair_select_queue(osc, m); atomic_set_long(&q->state, (1 << BIT_MBUF_QUEUED)); ridx = atomic_load_int(&q->ridx); @@ -251,7 +261,7 @@ epair_menq(struct mbuf *m, struct epair_softc *osc) /* Ring is full. */ if_inc_counter(ifp, IFCOUNTER_OQDROPS, 1); m_freem(m); - return (0); + return; } if_inc_counter(ifp, IFCOUNTER_OPACKETS, 1); @@ -267,9 +277,7 @@ epair_menq(struct mbuf *m, struct epair_softc *osc) if_inc_counter(oifp, IFCOUNTER_IPACKETS, 1); if (!atomic_testandset_long(&q->state, BIT_QUEUE_TASK)) - taskqueue_enqueue(epair_tasks.tq[bucket], &q->tx_task); - - return (0); + taskqueue_enqueue(epair_tasks.tq[q->id], &q->tx_task); } static void @@ -304,7 +312,7 @@ epair_start(struct ifnet *ifp) continue; } - (void) epair_menq(m, sc); + epair_menq(m, sc); } } @@ -313,7 +321,6 @@ epair_transmit(struct ifnet *ifp, struct mbuf *m) { struct epair_softc *sc; struct ifnet *oifp; - int error; #ifdef ALTQ int len; short mflags; @@ -358,6 +365,7 @@ epair_transmit(struct ifnet *ifp, struct mbuf *m) #ifdef ALTQ len = m->m_pkthdr.len; mflags = m->m_flags; + int error = 0; /* Support ALTQ via the classic if_start() path. */ IF_LOCK(&ifp->if_snd); @@ -377,8 +385,8 @@ epair_transmit(struct ifnet *ifp, struct mbuf *m) IF_UNLOCK(&ifp->if_snd); #endif - error = epair_menq(m, oifp->if_softc); - return (error); + epair_menq(m, oifp->if_softc); + return (0); } static void @@ -607,15 +615,23 @@ epair_free_sc(struct epair_softc *sc) free(sc, M_EPAIR); } +static void +epair_set_state(struct ifnet *ifp, bool running) +{ + if (running) { + ifp->if_drv_flags |= IFF_DRV_RUNNING; + if_link_state_change(ifp, LINK_STATE_UP); + } else { + if_link_state_change(ifp, LINK_STATE_DOWN); + ifp->if_drv_flags &= ~IFF_DRV_RUNNING; + } +} + static int -epair_clone_create(struct if_clone *ifc, char *name, size_t len, - struct ifc_data *ifd, struct ifnet **ifpp) +epair_handle_unit(struct if_clone *ifc, char *name, size_t len, int *punit) { - struct epair_softc *sca, *scb; - struct ifnet *ifp; + int error = 0, unit, wildcard; char *dp; - int error, unit, wildcard; - uint8_t eaddr[ETHER_ADDR_LEN]; /* 00:00:00:00:00:00 */ /* Try to see if a special unit was requested. */ error = ifc_name2unit(name, &unit); @@ -633,29 +649,55 @@ epair_clone_create(struct if_clone *ifc, char *name, size_t len, */ for (dp = name; *dp != '\0'; dp++); if (wildcard) { - error = snprintf(dp, len - (dp - name), "%d", unit); - if (error > len - (dp - name) - 1) { + int slen = snprintf(dp, len - (dp - name), "%d", unit); + if (slen > len - (dp - name) - 1) { /* ifName too long. */ - ifc_free_unit(ifc, unit); - return (ENOSPC); + error = ENOSPC; + goto done; } - dp += error; + dp += slen; } if (len - (dp - name) - 1 < 1) { /* No space left for our [ab] suffix. */ - ifc_free_unit(ifc, unit); - return (ENOSPC); + error = ENOSPC; + goto done; } *dp = 'b'; /* Must not change dp so we can replace 'a' by 'b' later. */ *(dp+1) = '\0'; /* Check if 'a' and 'b' interfaces already exist. */ - if (ifunit(name) != NULL) - return (EEXIST); + if (ifunit(name) != NULL) { + error = EEXIST; + goto done; + } + *dp = 'a'; - if (ifunit(name) != NULL) - return (EEXIST); + if (ifunit(name) != NULL) { + error = EEXIST; + goto done; + } + *punit = unit; +done: + if (error != 0) + ifc_free_unit(ifc, unit); + + return (error); +} + +static int +epair_clone_create(struct if_clone *ifc, char *name, size_t len, + struct ifc_data *ifd, struct ifnet **ifpp) +{ + struct epair_softc *sca, *scb; + struct ifnet *ifp; + char *dp; + int error, unit; + uint8_t eaddr[ETHER_ADDR_LEN]; /* 00:00:00:00:00:00 */ + + error = epair_handle_unit(ifc, name, len, &unit); + if (error != 0) + return (error); /* Allocate memory for both [ab] interfaces */ sca = epair_alloc_sc(ifc); @@ -681,6 +723,7 @@ epair_clone_create(struct if_clone *ifc, char *name, size_t len, ether_ifattach(ifp, eaddr); /* Swap the name and finish initialization of interface b. */ + dp = name + strlen(name) - 1; *dp = 'b'; epair_setup_ifp(scb, name, unit); @@ -700,10 +743,8 @@ epair_clone_create(struct if_clone *ifc, char *name, size_t len, strlcpy(name, sca->ifp->if_xname, len); /* Tell the world, that we are ready to rock. */ - sca->ifp->if_drv_flags |= IFF_DRV_RUNNING; - if_link_state_change(sca->ifp, LINK_STATE_UP); - scb->ifp->if_drv_flags |= IFF_DRV_RUNNING; - if_link_state_change(scb->ifp, LINK_STATE_UP); + epair_set_state(sca->ifp, true); + epair_set_state(scb->ifp, true); *ifpp = sca->ifp; @@ -750,10 +791,8 @@ epair_clone_destroy(struct if_clone *ifc, struct ifnet *ifp, uint32_t flags) scb = oifp->if_softc; /* Frist get the interfaces down and detached. */ - if_link_state_change(ifp, LINK_STATE_DOWN); - ifp->if_drv_flags &= ~IFF_DRV_RUNNING; - if_link_state_change(oifp, LINK_STATE_DOWN); - oifp->if_drv_flags &= ~IFF_DRV_RUNNING; + epair_set_state(ifp, false); + epair_set_state(oifp, false); ether_ifdetach(ifp); ether_ifdetach(oifp);