git: cc7a40b2b54a - stable/15 - ixgbe: fix host interface timeout detection

Kevin Bowling <[email protected]>
Newsgroups gmane.os.freebsd.devel.cvs.src
Message-ID <6a767a83.3f31b.47d6e6f0__25086.8381861196$1786149625$gmane$org@gitrepo.freebsd.org>
The branch stable/15 has been updated by kbowling:

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

commit cc7a40b2b54aef5aa6fcc34f4448ecf0fd1bbd5d
Author:     Kevin Bowling <[email protected]>
AuthorDate: 2026-07-28 11:21:14 +0000
Commit:     Kevin Bowling <[email protected]>
CommitDate: 2026-08-08 00:36:51 +0000

    ixgbe: fix host interface timeout detection
    
    The host-interface polling loop was scaled from milliseconds to
    microseconds, but its terminal test was left using the unscaled timeout.
    Completion at that intermediate iteration can be reported as a timeout,
    while actual expiry is not recognized and can accept stale status.
    
    Test against the scaled loop bound used by the polling loop.
    
    Fixes:          f46d75c90f5f ("ixgbe: improve MDIO performance by reducing semaphore/IPC delays")
    
    (cherry picked from commit db2bf4553ce32fdcae00f6e7392a6c2010247dd6)
---
 sys/dev/ixgbe/ixgbe_common.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/ixgbe/ixgbe_common.c b/sys/dev/ixgbe/ixgbe_common.c
index 6495bfdecc60..439b861a17e9 100644
--- a/sys/dev/ixgbe/ixgbe_common.c
+++ b/sys/dev/ixgbe/ixgbe_common.c
@@ -4646,7 +4646,7 @@ s32 ixgbe_hic_unlocked(struct ixgbe_hw *hw, u32 *buffer, u32 length,
 		return IXGBE_SUCCESS;
 
 	/* Check command completion */
-	if ((timeout && i == timeout) ||
+	if ((timeout && i == timeout * 1000) ||
 	    !(IXGBE_READ_REG(hw, IXGBE_HICR) & IXGBE_HICR_SV)) {
 		ERROR_REPORT1(IXGBE_ERROR_CAUTION,
 			      "Command has failed with no status valid.\n");
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.