Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 30 Jul 2026 05:06:04 +0000
From:      Kevin Bowling <kbowling@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: c637d474045a - main - igb: Stop writing the legacy TADV register
Message-ID:  <6a6adbbc.33207.ca912dd@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch main has been updated by kbowling:

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

commit c637d474045a41b1763c17a8b4b7763d4159504f
Author:     Kevin Bowling <kbowling@FreeBSD.org>
AuthorDate: 2026-07-29 04:23:30 +0000
Commit:     Kevin Bowling <kbowling@FreeBSD.org>
CommitDate: 2026-07-30 05:05:32 +0000

    igb: Stop writing the legacy TADV register
    
    TADV is an em-class interrupt delay register and is absent from the
    82575 and later register model. The igb attach path does not expose or
    initialize that control, but transmit initialization still wrote its
    zero valued storage into a reserved queue-window offset.
    
    Apply the same igb_mac_min boundary already used for TIDV and the
    absolute-delay sysctls.
    
    MFC after:      1 week
    Sponsored by:   BBOX.io
---
 sys/dev/e1000/if_em.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/e1000/if_em.c b/sys/dev/e1000/if_em.c
index 4f4eeee5da50..0f60178ae3c8 100644
--- a/sys/dev/e1000/if_em.c
+++ b/sys/dev/e1000/if_em.c
@@ -4208,7 +4208,7 @@ em_initialize_transmit_unit(if_ctx_t ctx)
 			sc->txd_cmd |= E1000_TXD_CMD_IDE;
 	}
 
-	if (hw->mac.type >= e1000_82540)
+	if (hw->mac.type >= e1000_82540 && hw->mac.type < igb_mac_min)
 		E1000_WRITE_REG(hw, E1000_TADV, sc->tx_abs_int_delay.value);
 
 	if (hw->mac.type == e1000_82571 || hw->mac.type == e1000_82572) {


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6a6adbbc.33207.ca912dd>