Date: Tue, 26 Jul 2022 19:33:29 GMT From: Marcin Wojtas <mw@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org Subject: git: aabc314c3fff - stable/12 - ena: Remove write-only datapath variable Message-ID: <202207261933.26QJXTK0033928@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by mw: URL: https://cgit.FreeBSD.org/src/commit/?id=aabc314c3fff131510452cd0c95932ffe73d46bb commit aabc314c3fff131510452cd0c95932ffe73d46bb Author: Michal Krawczyk <mk@semihalf.com> AuthorDate: 2022-06-10 09:18:11 +0000 Commit: Marcin Wojtas <mw@FreeBSD.org> CommitDate: 2022-07-26 19:33:04 +0000 ena: Remove write-only datapath variable The ena_qid variable value is never used. It can be safely removed. That also silences the compilation warning. Obtained from: Semihalf MFC after: 2 weeks Sponsored by: Amazon, Inc. (cherry picked from commit d5d5ea87236d8f839cad84296f23d285a696195c) --- sys/dev/ena/ena_datapath.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/sys/dev/ena/ena_datapath.c b/sys/dev/ena/ena_datapath.c index 729a50f59eeb..f23403b5089a 100644 --- a/sys/dev/ena/ena_datapath.c +++ b/sys/dev/ena/ena_datapath.c @@ -1101,7 +1101,6 @@ ena_start_xmit(struct ena_ring *tx_ring) { struct mbuf *mbuf; struct ena_adapter *adapter = tx_ring->adapter; - int ena_qid; int ret = 0; ENA_RING_MTX_ASSERT(tx_ring); @@ -1112,8 +1111,6 @@ ena_start_xmit(struct ena_ring *tx_ring) if (unlikely(!ENA_FLAG_ISSET(ENA_FLAG_LINK_UP, adapter))) return; - ena_qid = ENA_IO_TXQ_IDX(tx_ring->que->id); - while ((mbuf = drbr_peek(adapter->ifp, tx_ring->br)) != NULL) { ena_log_io(adapter->pdev, DBG, "\ndequeued mbuf %p with flags %#x and header csum flags %#jx\n",
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202207261933.26QJXTK0033928>