|WARNING| pw109520 [PATCH v2] net/ice: optimize max queue number calculation

|WARNING| pw109520 [PATCH v2] net/ice: optimize max queue number calculation

[checkpatch]

Test-Label: checkpatch
Test-Status: WARNING
http://dpdk.org/patch/109520

_coding style issues_

Must be a reply to the first patch (--in-reply-to).

|WARNING| pw109520 [PATCH] [v2] net/ice: optimize max queue number calculation

[dpdklab]

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

Test-Label: iol-testing
Test-Status: WARNING
http://dpdk.org/patch/109520

_apply patch failure_

Submitter: Qi Zhang <qi.z.zhang@intel.com>
Date: Friday, April 08 2022 11:23:43 
Applied on: CommitID:b35c4b0aa2bcd242d8b1989acaa41fed154045c7
Apply patch set 109520 failed:

Checking patch drivers/net/ice/ice_ethdev.c...
error: while searching for:
		return -ENOTSUP;
	}

	vsi->nb_qps = RTE_MIN(vsi->nb_qps, ICE_MAX_Q_PER_TC);
	fls = (vsi->nb_qps == 0) ? 0 : rte_fls_u32(vsi->nb_qps) - 1;
	/* Adjust the queue number to actual queues that can be applied */
	vsi->nb_qps = (vsi->nb_qps == 0) ? 0 : 0x1 << fls;

	qp_idx = 0;
	/* Set tc and queue mapping with VSI */

error: patch failed: drivers/net/ice/ice_ethdev.c:819
Applying patch drivers/net/ice/ice_ethdev.c with 1 reject...
Rejected hunk #1.
diff a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c	(rejected hunks)
@@ -819,10 +819,26 @@ ice_vsi_config_tc_queue_mapping(struct ice_vsi *vsi,
 		return -ENOTSUP;
 	}
 
-	vsi->nb_qps = RTE_MIN(vsi->nb_qps, ICE_MAX_Q_PER_TC);
-	fls = (vsi->nb_qps == 0) ? 0 : rte_fls_u32(vsi->nb_qps) - 1;
-	/* Adjust the queue number to actual queues that can be applied */
-	vsi->nb_qps = (vsi->nb_qps == 0) ? 0 : 0x1 << fls;
+	/* vector 0 is reserved and 1 vector for ctrl vsi */
+	if (vsi->adapter->hw.func_caps.common_cap.num_msix_vectors < 2)
+		vsi->nb_qps = 0;
+	else
+		vsi->nb_qps = RTE_MIN
+			((uint16_t)vsi->adapter->hw.func_caps.common_cap.num_msix_vectors - 2,
+			RTE_MIN(vsi->nb_qps, ICE_MAX_Q_PER_TC));
+
+	/* nb_qps(hex)  -> fls */
+	/* 0000		-> 0 */
+	/* 0001		-> 0 */
+	/* 0002		-> 1 */
+	/* 0003 ~ 0004	-> 2 */
+	/* 0005 ~ 0008	-> 3 */
+	/* 0009 ~ 0010	-> 4 */
+	/* 0011 ~ 0020	-> 5 */
+	/* 0021 ~ 0040	-> 6 */
+	/* 0041 ~ 0080	-> 7 */
+	/* 0081 ~ 0100	-> 8 */
+	fls = (vsi->nb_qps == 0) ? 0 : rte_fls_u32(vsi->nb_qps - 1);
 
 	qp_idx = 0;
 	/* Set tc and queue mapping with VSI */

https://lab.dpdk.org/results/dashboard/patchsets/21737/

UNH-IOL DPDK Community Lab