Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 08 Aug 2026 00:44:11 +0000
From:      Kevin Bowling <kbowling@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 23c1d528bc0e - stable/14 - ixgbe: reject VF requests before CTS
Message-ID:  <6a767bdb.3fd9b.1725ed94@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch stable/14 has been updated by kbowling:

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

commit 23c1d528bc0e0fa7ae09a22aa2240aefa8092709
Author:     Kevin Bowling <kbowling@FreeBSD.org>
AuthorDate: 2026-07-28 11:19:47 +0000
Commit:     Kevin Bowling <kbowling@FreeBSD.org>
CommitDate: 2026-08-08 00:42:18 +0000

    ixgbe: reject VF requests before CTS
    
    A VF that sends a non-reset request before completing reset negotiation
    has not received CTS.  The PF ignores the request but currently reports
    success, leaving the VF with a false view of the programmed state.
    
    Return failure for the ignored request.  This restores the behavior lost
    when the mailbox helpers were renamed.
    
    Fixes:          36c516b31136 ("ixgbe: update if_sriov to use the new mailbox apis")
    
    (cherry picked from commit 9fc83caf48e710c3f6457cab4bdb5e6c76e8be51)
---
 sys/dev/ixgbe/if_sriov.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/sys/dev/ixgbe/if_sriov.c b/sys/dev/ixgbe/if_sriov.c
index bd27796ebcf3..8230464bd9bc 100644
--- a/sys/dev/ixgbe/if_sriov.c
+++ b/sys/dev/ixgbe/if_sriov.c
@@ -599,7 +599,7 @@ ixgbe_process_vf_msg(if_ctx_t ctx, struct ixgbe_vf *vf)
 	}
 
 	if (!(vf->flags & IXGBE_VF_CTS)) {
-		ixgbe_send_vf_success(sc, vf, msg[0]);
+		ixgbe_send_vf_failure(sc, vf, msg[0]);
 		return;
 	}
 


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6a767bdb.3fd9b.1725ed94>