|SUCCESS|dpdk-next-net-brcm|ad22730ab1| Intel-Testing

[sys_stv at intel.com]

Test-Label: intel-Testing
Test-Status: SUCCESS
_Testing PASS_

DPDK git repo: dpdk-next-net-brcm
commit ad22730ab1586f576631563a13fe7ec4ac672222
Author: Chenna Arnoori <chenna.arnoori at broadcom.com>
Date:   Tue Jun 16 10:01:29 2026 -0600

    net/bnxt: avoid link flap on flow control set
    
    When OVS-DPDK reconciles port state, it calls flow_ctrl_get followed
    by flow_ctrl_set if the returned configuration differs from its
    database. The driver was reporting autoneg=1 whenever the firmware
    had auto_pause set (including the AUTONEG_PAUSE bit 0x4), even
    though no pause autoneg was actually requested. This mismatch
    caused OVS to repeatedly call flow_ctrl_set, which triggered a
    full link reconfig with PHY reset, flapping the link every time
    any interface change occurred on the system.
    
    Two problems were fixed. First, flow_ctrl_set was clearing all
    autoneg bits instead of only the flow-control autoneg bit,
    which also wiped the speed autoneg state. Second, the pause
    set path was abandoning its own HWRM request and calling the
    full link config function, which built a separate request
    without the pause fields that set_pause_common would have
    computed.
    
    Port the kernel bnxt_en driver's approach: add a
    set_link_common helper that merges link and speed fields into
    an existing HWRM request, and rework set_pause to build a
    single combined request with both pause and link fields when
    a full reconfig is needed. When only pause changes without an
    auto-to-force transition, a pause-only PHY config is sent
    without a full link reprogram.
    
    When a reconfig is required, set_link_common reproduces the
    behaviour of the full link config path: it validates the
    requested speeds, honours the speeds-v2 (P7) field layout, and
    keeps the 10GBase-T must-autoneg guard, so the combined request
    programs the same fields the standalone link config would have.
    
    Fixes: 8aaf473bbed6 ("net/bnxt: add flow control operations")
    Cc: stable at dpdk.org
    Signed-off-by: Chenna Arnoori <chenna.arnoori at broadcom.com>

Testing Summary : 18 Case Done, 18 Successful, 0 Failures      

TestPlan:
	pf_smoke: http://git.dpdk.org/tools/dts/tree/test_plans/pf_smoke_test_plan.rst
	vf_smoke: http://git.dpdk.org/tools/dts/tree/test_plans/vf_smoke_test_plan.rst
	virtio_smoke: http://git.dpdk.org/tools/dts/tree/test_plans/virtio_smoke_test_plan.rst

TestSuite:
	pf_smoke: http://git.dpdk.org/tools/dts/tree/tests/TestSuite_pf_smoke.py
	vf_smoke: http://git.dpdk.org/tools/dts/tree/tests/TestSuite_vf_smoke.py
	virtio_smoke: http://git.dpdk.org/tools/dts/tree/tests/TestSuite_virtio_smoke.py


OS : Ubuntu 24.04.4 LTS       
Kernel : 6.8.0-87-generic         
GCC : 13.3.0-6ubuntu2~24.04.1        
NIC : Ethernet Controller E810-C for SFP         
Target : x86_64-native-linuxapp-gcc      

	Test result details:
	+-------------+---------------------------+-------+
	| suite       | case                      | status|
	+-------------+---------------------------+-------+
	| asan_smoke  | test_rxtx_with_ASan_enable| passed|
	| pf_smoke    | test_pf_jumbo_frames      | passed|
	| pf_smoke    | test_pf_rss               | passed|
	| pf_smoke    | test_pf_tx_rx_queue       | passed|
	| vf_smoke    | test_vf_jumbo_frames      | passed|
	| vf_smoke    | test_vf_rss               | passed|
	| vf_smoke    | test_vf_tx_rx_queue       | passed|
	| virtio_smoke| test_virtio_loopback      | passed|
	| virtio_smoke| test_virtio_pvp           | passed|
	+-------------+---------------------------+-------+


OS : Ubuntu 24.04.4 LTS       
Kernel : 6.8.0-71-generic         
GCC : 13.3.0-6ubuntu2~24.04.1        
NIC : Ethernet Controller XL710 for 40GbE QSFP+         
Target : x86_64-native-linuxapp-gcc      

	Test result details:
	+-------------+---------------------------+-------+
	| suite       | case                      | status|
	+-------------+---------------------------+-------+
	| asan_smoke  | test_rxtx_with_ASan_enable| passed|
	| pf_smoke    | test_pf_jumbo_frames      | passed|
	| pf_smoke    | test_pf_rss               | passed|
	| pf_smoke    | test_pf_tx_rx_queue       | passed|
	| vf_smoke    | test_vf_rss               | passed|
	| vf_smoke    | test_vf_tx_rx_queue       | passed|
	| vf_smoke    | test_vf_jumbo_frames      | n/a   |
	| virtio_smoke| test_virtio_loopback      | passed|
	| virtio_smoke| test_virtio_pvp           | passed|
	+-------------+---------------------------+-------+


DPDK STV team