|SUCCESS|dpdk|0e03ab647d| Intel-Testing

[sys_stv@intel.com]

[-- Attachment #1: Type: text/plain, Size: 4399 bytes --]


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

DPDK git repo: dpdk
commit 0e03ab647d07cd985a7cac36cefff5195cc3a07d
Author: Yang Ming <mosesyyoung@gmail.com>
Date:   Sat Jul 19 23:32:26 2025 +0800

    crypto/ipsec_mb: fix QP release in secondary
    
    When a secondary process tries to release a queue pair (QP) that
    does not belong to it, error logs occur:
    CRYPTODEV: ipsec_mb_ipc_request() line 373: Unable to release
    qp_id=0
    EAL: Message data is too long
    EAL: Fail to handle message: ipsec_mb_mp_msg
    EAL: Fail to recv reply for request /tmp/dpdk/l2hi/mp_socket:
    ipsec_mb_mp_msg
    
    From the code path, cryptodev->data is allocated in the primary
    via rte_cryptodev_data_alloc() (inside
    ipsec_mb_create-->rte_cryptodev_pmd_create
    -->rte_cryptodev_pmd_allocate-->rte_cryptodev_data_alloc).
    This memory is placed in a shared memzone
    (rte_cryptodev_data_%u), so both primary and secondary processes
    reference the same cryptodev->data, including nb_queue_pairs and
    queue_pairs[].
    
    As a result, when the secondary process exits, ipsec_mb_remove()
    is called (inside
    rte_eal_cleanup-->eal_bus_cleanup-->vdev_cleanup
    -->rte_vdev_driver-->ipsec_mb_remove-->ipsec_mb_qp_release
    -->ipsec_mb_secondary_qp_op) and it loops through all queue
    pairs using:
    for (qp_id = 0; qp_id < cryptodev->data->nb_queue_pairs; qp_id++)
         ipsec_mb_qp_release(cryptodev, qp_id);
    
    This causes the secondary to attempt releasing queue pairs it
    doesn't own, triggering the error logs mentioned above.
    
    This patch ensures that a secondary process only frees a QP if
    it actually owns it, preventing conflicts and resolving the
    issue.
    
    Fixes: b35848bc01f6 ("crypto/ipsec_mb: add multi-process IPC request handler")
    Cc: stable@dpdk.org
    
    Signed-off-by: Yang Ming <mosesyyoung@gmail.com>
    Acked-by: Anatoly Burakov <anatoly.burakov@intel.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 22.04.5 LTS       
Kernel : 6.8.0-40-generic         
GCC : 11.4.0-1ubuntu1~22.04.2        
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      | blocked|
	| vf_smoke    | test_vf_rss               | blocked|
	| vf_smoke    | test_vf_tx_rx_queue       | blocked|
	| virtio_smoke| test_virtio_loopback      | blocked|
	| virtio_smoke| test_virtio_pvp           | blocked|
	+-------------+---------------------------+--------+


OS : Ubuntu 22.04.5 LTS       
Kernel : 5.15.0-119-generic         
GCC : 11.4.0-1ubuntu1~22.04.2        
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