From nobody Sun Aug 2 23:10:34 2026 X-Original-To: dev-commits-src-all@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 4hCwWv062lz6mYvR for ; Sun, 02 Aug 2026 23:10:35 +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 "YR1" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 4hCwWt4qLsz48xP for ; Sun, 02 Aug 2026 23:10:34 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1785712234; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=gHqPS9oDEelj2hoPGx0d7idywvxOk4+tDe9Jz2XHBS4=; b=iKlXZJy/ZXNGF46PUgiw9QfWhoff57i3FrXk+R4CEeLzf7i/r1ihAkgyEifV7EzyozsP3r 6rid9QLjZOvTv6Sx/l7+OkFgoakZsIIR9qYzpXHFA75ThXfTMH/KwSR3uVx0JYUdlzQDTz nb3TF9hk5g/sSdYpfBy4dRDx6ONcRjE32xGdj6AgIGf9nwfgVMEOfygMGZdOWGDqeFRmWY WEiTh8OceYUPktR/aC7dV+DfB28amteD5wYced16n3ijDbfNrd64ZkpnaigGx8kb1eG4uB V3AekMPyqlrImzgkDzbx5fgNbUgd890pYa80LDaaBJ9z8JUq/JDlL4Iz7ABvGA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1785712234; a=rsa-sha256; cv=none; b=aJu5G3xV0tytfPnW4MVJo+NvonQx1qZqXuFFxvmyHry6XkwJnhknNHYog9UpzqrpWVdxrD J/MyC9QTY7/ZhqTOBnwOPkyZMPKYNMdSH2+cyiB99veHohQaP0tR4viFJy+VIrSBJJTTNN yc9t98cFjSxSjz8axzX7zpfXUXxPvxYqEp3g+LERUV1AbnURiHIfWYIT4juJHGHoYJW1yZ 4OQZxiD8ljCVFPu0q87ifjcODXupFMYt7gfcu5R8jL2Gx4SfqTQcUzx6QRcNKL1FSw81Sl kdrIm5hhXeY2fI+d1JqkGls89rldmSnTr6EukrAleoJPLN4+AjsTa3IG0oSaXw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1785712234; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=gHqPS9oDEelj2hoPGx0d7idywvxOk4+tDe9Jz2XHBS4=; b=uTNgn887Ac9KSdNHzeflJjQswrq5fWfLGXuOdjbPgjucmjr1Tr8JQw8DwZdkBWXH7fB8jc UpRtJxYlgHpKVuiberrcjQQzA45KoWo3dnEXvJ+cThClmZwM26OErckniCY07Gwh0777F2 Fi2Yad6rD/VJwOsReVd+NfK/hrhf0VTtWrOEaazVWTErmJodAsigv8Fu4dB5ZSKnJ+KwkE STy3rlgHJGLpofJWXCueBe+2v/9RqUCd9h4RbGDq8WL+i6d8+U1bkQ6uwnxIJIQoZlMRJ1 qoXNZiJSyc6hHN8HxCKQyRvw2P/lwWkrD1Wd4Q2yzmPZp5xYRibsSoIWblis7Q== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) by mxrelay.nyi.freebsd.org (Postfix) with ESMTP id 4hCwWt3mkJzbZ7 for ; Sun, 02 Aug 2026 23:10:34 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from git (uid 1279) (envelope-from git@FreeBSD.org) id 425f9 by gitrepo.freebsd.org (DragonFly Mail Agent v0.13+ on gitrepo.freebsd.org); Sun, 02 Aug 2026 23:10:34 +0000 To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Cc: Nick Price From: Adrian Chadd Subject: git: ae7e2c9170f6 - main - aq(4): clean up diagnostics and remove dead code List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: X-BeenThere: dev-commits-src-all@freebsd.org Sender: owner-dev-commits-src-all@FreeBSD.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: adrian X-Git-Repository: src X-Git-Refname: refs/heads/main X-Git-Reftype: branch X-Git-Commit: ae7e2c9170f6f7e39ab3132eb72c89f9f6e3a4d6 Auto-Submitted: auto-generated Date: Sun, 02 Aug 2026 23:10:34 +0000 Message-Id: <6a6fce6a.425f9.54d210b4@gitrepo.freebsd.org> The branch main has been updated by adrian: URL: https://cgit.FreeBSD.org/src/commit/?id=ae7e2c9170f6f7e39ab3132eb72c89f9f6e3a4d6 commit ae7e2c9170f6f7e39ab3132eb72c89f9f6e3a4d6 Author: Nick Price AuthorDate: 2026-08-02 22:44:43 +0000 Commit: Adrian Chadd CommitDate: 2026-08-02 22:44:43 +0000 aq(4): clean up diagnostics and remove dead code Non-functional cleanup, no change in behavior. device_printf() already prefixes each line with the device name, so the inline "atlantic:" token in the status and error messages produced a doubled prefix and diverged from the trace macros; remove it so all output carries one uniform "aqN:" prefix. Compile the RX/TX descriptor tracers only when AQ_CFG_DEBUG_LVL > 2 and make them no-op macros otherwise, so the default build no longer pays a cross-TU call plus argument evaluation per descriptor. Drop enum aq_dev_state, struct aq_rx_filters, and struct aq_vlan_tag, which have no remaining references now that VLAN state lives in a bitstr_t. Replace the four identical aq_sysctl_print_{tx,rx}_{head,tail} handlers, each carrying a dead write path on a read-only oid, with one aq_sysctl_print_ring_ptr that selects the accessor from arg2. Reduce the thermal and PHY-recovery comments to single terse lines that keep the load-bearing register numbers and the A1-vs-A2 recovery difference. Signed-off-by: Nick Price Reviewed by: adrian Differential Revision: https://reviews.freebsd.org/D58436 --- sys/dev/aq/aq_dbg.c | 3 ++ sys/dev/aq/aq_dbg.h | 6 +++ sys/dev/aq/aq_device.h | 18 ------- sys/dev/aq/aq_irq.c | 4 +- sys/dev/aq/aq_main.c | 126 +++++++++++++++++-------------------------------- 5 files changed, 53 insertions(+), 104 deletions(-) diff --git a/sys/dev/aq/aq_dbg.c b/sys/dev/aq/aq_dbg.c index 44391322143d..078d561f5f9e 100644 --- a/sys/dev/aq/aq_dbg.c +++ b/sys/dev/aq/aq_dbg.c @@ -50,6 +50,8 @@ __FBSDID("$FreeBSD$"); (BIT(BIT_BEGIN - BIT_END + 1) -1)) #define __field(TYPE, VAR) TYPE VAR; + +#if AQ_CFG_DEBUG_LVL > 2 void trace_aq_tx_descr(struct aq_hw *hw, int ring_idx, unsigned int pointer, volatile uint64_t descr[2]) @@ -192,3 +194,4 @@ trace_aq_tx_context_descr(struct aq_hw *hw, int ring_idx, unsigned int pointer, __entry->des_typ); #endif } +#endif diff --git a/sys/dev/aq/aq_dbg.h b/sys/dev/aq/aq_dbg.h index 148916e3cbda..805ac3f05f44 100644 --- a/sys/dev/aq/aq_dbg.h +++ b/sys/dev/aq/aq_dbg.h @@ -150,11 +150,17 @@ enum aq_debug_category #define trace_detail(_hw, _cat, _fmt, args...) \ aq_trace_base(_hw, lvl_detail, _cat, _fmt, ##args) +#if AQ_CFG_DEBUG_LVL > 2 void trace_aq_tx_descr(struct aq_hw *hw, int ring_idx, unsigned int pointer, volatile uint64_t descr[2]); void trace_aq_rx_descr(struct aq_hw *hw, int ring_idx, unsigned int pointer, volatile uint64_t descr[2]); void trace_aq_tx_context_descr(struct aq_hw *hw, int ring_idx, unsigned int pointer, volatile uint64_t descr[2]); +#else +#define trace_aq_tx_descr(...) ((void)0) +#define trace_aq_rx_descr(...) ((void)0) +#define trace_aq_tx_context_descr(...) ((void)0) +#endif #endif // AQ_DBG_H diff --git a/sys/dev/aq/aq_device.h b/sys/dev/aq/aq_device.h index 1ace227974a5..07482e5402ca 100644 --- a/sys/dev/aq/aq_device.h +++ b/sys/dev/aq/aq_device.h @@ -92,24 +92,6 @@ struct aq_stats { uint64_t bbtc; }; -enum aq_dev_state { - AQ_DEV_STATE_UNLOAD, - AQ_DEV_STATE_PCI_STOP, - AQ_DEV_STATE_DOWN, - AQ_DEV_STATE_UP, -}; - -struct aq_rx_filters { - unsigned int rule_cnt; - struct aq_rx_filter_vlan vlan_filters[AQ_HW_VLAN_MAX_FILTERS]; - struct aq_rx_filter_l2 etype_filters[AQ_HW_ETYPE_MAX_FILTERS]; -}; - -struct aq_vlan_tag { - SLIST_ENTRY(aq_vlan_tag) next; - uint16_t tag; -}; - struct aq_dev { device_t dev; if_ctx_t ctx; diff --git a/sys/dev/aq/aq_irq.c b/sys/dev/aq/aq_irq.c index 325a957b1e8f..dc1b0ac351ff 100644 --- a/sys/dev/aq/aq_irq.c +++ b/sys/dev/aq/aq_irq.c @@ -219,7 +219,7 @@ aq_if_update_admin_status(if_ctx_t ctx) struct aq_hw_fc_info fc_neg; aq_hw_get_link_state(hw, &link_speed, &fc_neg); if (link_speed && !aq_dev->linkup) { /* link was DOWN */ - device_printf(aq_dev->dev, "atlantic: link UP: speed=%d\n", link_speed); + device_printf(aq_dev->dev, "link UP: speed=%d\n", link_speed); aq_dev->linkup = 1; aq_dev->phy_fault_last = 0; @@ -234,7 +234,7 @@ aq_if_update_admin_status(if_ctx_t ctx) /* update ITR settings according new link speed */ aq_hw_interrupt_moderation_set(hw); } else if (link_speed == 0U && aq_dev->linkup) { /* link was UP */ - device_printf(aq_dev->dev, "atlantic: link DOWN\n"); + device_printf(aq_dev->dev, "link DOWN\n"); aq_dev->linkup = 0; diff --git a/sys/dev/aq/aq_main.c b/sys/dev/aq/aq_main.c index 4fe9aad42a71..445d02da3257 100644 --- a/sys/dev/aq/aq_main.c +++ b/sys/dev/aq/aq_main.c @@ -629,7 +629,7 @@ aq_if_tx_queues_alloc(if_ctx_t ctx, caddr_t *vaddrs, uint64_t *paddrs, M_AQ, M_NOWAIT | M_ZERO); if (!ring){ rc = ENOMEM; - device_printf(softc->dev, "atlantic: tx_ring malloc fail\n"); + device_printf(softc->dev, "tx_ring malloc fail\n"); goto fail; } ring->tx_descs = (volatile struct aq_tx_desc*)vaddrs[i]; @@ -644,7 +644,7 @@ aq_if_tx_queues_alloc(if_ctx_t ctx, caddr_t *vaddrs, uint64_t *paddrs, rc = aq_ring_stats_alloc(ring); if (rc != 0) { device_printf(softc->dev, - "atlantic: tx_ring stats alloc fail\n"); + "tx_ring stats alloc fail\n"); goto fail; } } @@ -675,7 +675,7 @@ aq_if_rx_queues_alloc(if_ctx_t ctx, caddr_t *vaddrs, uint64_t *paddrs, if (!ring){ rc = ENOMEM; device_printf(softc->dev, - "atlantic: rx_ring malloc fail\n"); + "rx_ring malloc fail\n"); goto fail; } @@ -690,7 +690,7 @@ aq_if_rx_queues_alloc(if_ctx_t ctx, caddr_t *vaddrs, uint64_t *paddrs, rc = aq_ring_stats_alloc(ring); if (rc != 0) { device_printf(softc->dev, - "atlantic: rx_ring stats alloc fail\n"); + "rx_ring stats alloc fail\n"); goto fail; } } @@ -756,7 +756,7 @@ aq_if_init(if_ctx_t ctx) err = aq_hw_init(&softc->hw, softc->hw.mac_addr, softc->msix, softc->scctx->isc_intr == IFLIB_INTR_MSIX); if (err != 0) { - device_printf(softc->dev, "atlantic: aq_hw_init: %d\n", err); + device_printf(softc->dev, "aq_hw_init: %d\n", err); AQ_DBG_EXIT(err); return; } @@ -774,12 +774,12 @@ aq_if_init(if_ctx_t ctx) err = aq_ring_tx_init(&softc->hw, ring); if (err) { device_printf(softc->dev, - "atlantic: aq_ring_tx_init: %d\n", err); + "aq_ring_tx_init: %d\n", err); } err = aq_ring_tx_start(hw, ring); if (err != 0) { device_printf(softc->dev, - "atlantic: aq_ring_tx_start: %d\n", err); + "aq_ring_tx_start: %d\n", err); } } for (i = 0; i < softc->rx_rings_count; i++) { @@ -788,12 +788,12 @@ aq_if_init(if_ctx_t ctx) err = aq_ring_rx_init(&softc->hw, ring); if (err) { device_printf(softc->dev, - "atlantic: aq_ring_rx_init: %d\n", err); + "aq_ring_rx_init: %d\n", err); } err = aq_ring_rx_start(hw, ring); if (err != 0) { device_printf(softc->dev, - "atlantic: aq_ring_rx_start: %d\n", err); + "aq_ring_rx_start: %d\n", err); } aq_if_rx_queue_intr_enable(ctx, i); } @@ -1325,80 +1325,38 @@ aq_sysctl_print_rss_config(SYSCTL_HANDLER_ARGS) return (0); } -static int -aq_sysctl_print_tx_head(SYSCTL_HANDLER_ARGS) -{ - struct aq_ring *ring = arg1; - int error = 0; - unsigned int val; - - if (!ring) - return (0); - - val = tdm_tx_desc_head_ptr_get(&ring->dev->hw, ring->index); - - error = sysctl_handle_int(oidp, &val, 0, req); - if (error || !req->newptr) - return (error); - - return (0); -} - -static int -aq_sysctl_print_tx_tail(SYSCTL_HANDLER_ARGS) -{ - struct aq_ring *ring = arg1; - int error = 0; - unsigned int val; - - if (!ring) - return (0); - - val = reg_tx_dma_desc_tail_ptr_get(&ring->dev->hw, ring->index); - - error = sysctl_handle_int(oidp, &val, 0, req); - if (error || !req->newptr) - return (error); - - return (0); -} - -static int -aq_sysctl_print_rx_head(SYSCTL_HANDLER_ARGS) -{ - struct aq_ring *ring = arg1; - int error = 0; - unsigned int val; - - if (!ring) - return (0); - - val = rdm_rx_desc_head_ptr_get(&ring->dev->hw, ring->index); - - error = sysctl_handle_int(oidp, &val, 0, req); - if (error || !req->newptr) - return (error); - - return (0); -} +enum aq_ring_ptr { + AQ_RING_TX_HEAD, + AQ_RING_TX_TAIL, + AQ_RING_RX_HEAD, + AQ_RING_RX_TAIL, +}; static int -aq_sysctl_print_rx_tail(SYSCTL_HANDLER_ARGS) +aq_sysctl_print_ring_ptr(SYSCTL_HANDLER_ARGS) { - struct aq_ring *ring = arg1; - int error = 0; - unsigned int val; + struct aq_ring *ring = arg1; + unsigned int val; - if (!ring) + if (ring == NULL) return (0); - val = reg_rx_dma_desc_tail_ptr_get(&ring->dev->hw, ring->index); - - error = sysctl_handle_int(oidp, &val, 0, req); - if (error || !req->newptr) - return (error); + switch (arg2) { + case AQ_RING_TX_HEAD: + val = tdm_tx_desc_head_ptr_get(&ring->dev->hw, ring->index); + break; + case AQ_RING_TX_TAIL: + val = reg_tx_dma_desc_tail_ptr_get(&ring->dev->hw, ring->index); + break; + case AQ_RING_RX_HEAD: + val = rdm_rx_desc_head_ptr_get(&ring->dev->hw, ring->index); + break; + default: /* AQ_RING_RX_TAIL */ + val = reg_rx_dma_desc_tail_ptr_get(&ring->dev->hw, ring->index); + break; + } - return (0); + return (sysctl_handle_int(oidp, &val, 0, req)); } static int @@ -1467,11 +1425,11 @@ aq_add_stats_sysctls(struct aq_dev *softc) SYSCTL_ADD_COUNTER_U64(ctx, queue_list, OID_AUTO, "tx_bytes", CTLFLAG_RD, &(ring->stats.tx_bytes), "TX Octets"); SYSCTL_ADD_PROC(ctx, queue_list, OID_AUTO, "tx_head", - CTLTYPE_UINT | CTLFLAG_RD, ring, 0, - aq_sysctl_print_tx_head, "IU", "ring head pointer"); + CTLTYPE_UINT | CTLFLAG_RD, ring, AQ_RING_TX_HEAD, + aq_sysctl_print_ring_ptr, "IU", "ring head pointer"); SYSCTL_ADD_PROC(ctx, queue_list, OID_AUTO, "tx_tail", - CTLTYPE_UINT | CTLFLAG_RD, ring, 0, - aq_sysctl_print_tx_tail, "IU", "ring tail pointer"); + CTLTYPE_UINT | CTLFLAG_RD, ring, AQ_RING_TX_TAIL, + aq_sysctl_print_ring_ptr, "IU", "ring tail pointer"); } for (int i = 0; i < softc->rx_rings_count; i++) { @@ -1490,11 +1448,11 @@ aq_add_stats_sysctls(struct aq_dev *softc) SYSCTL_ADD_COUNTER_U64(ctx, queue_list, OID_AUTO, "irq", CTLFLAG_RD, &(ring->stats.irq), "RX interrupts"); SYSCTL_ADD_PROC(ctx, queue_list, OID_AUTO, "rx_head", - CTLTYPE_UINT | CTLFLAG_RD, ring, 0, - aq_sysctl_print_rx_head, "IU", "ring head pointer"); + CTLTYPE_UINT | CTLFLAG_RD, ring, AQ_RING_RX_HEAD, + aq_sysctl_print_ring_ptr, "IU", "ring head pointer"); SYSCTL_ADD_PROC(ctx, queue_list, OID_AUTO, "rx_tail", - CTLTYPE_UINT | CTLFLAG_RD, ring, 0, - aq_sysctl_print_rx_tail, "IU", " ring tail pointer"); + CTLTYPE_UINT | CTLFLAG_RD, ring, AQ_RING_RX_TAIL, + aq_sysctl_print_ring_ptr, "IU", "ring tail pointer"); } stat_node = SYSCTL_ADD_NODE(ctx, child, OID_AUTO, "mac",