|WARNING| pw108248-108250 [PATCH] [3/3] common/cnxk: check SQ node before setting bp config

dpdklab at iol.unh.edu dpdklab at iol.unh.edu
Thu Feb 24 15:18:40 CET 2022


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

_apply patch failure_

Submitter: Satha Koteswara Rao Kottidi <skoteshwar at marvell.com>
Date: Thursday, February 24 2022 09:57:51 
Applied on: CommitID:ee05a93e1e6633d0fdec409faf09f12a2e05b991
Apply patch set 108248-108250 failed:

Checking patch drivers/common/cnxk/meson.build...
Hunk #1 succeeded at 43 (offset -1 lines).
Checking patch drivers/common/cnxk/roc_nix.h...
Hunk #1 succeeded at 551 (offset -19 lines).
error: while searching for:
int __roc_api roc_nix_tm_pfc_prepare_tree(struct roc_nix *roc_nix);
bool __roc_api roc_nix_tm_is_user_hierarchy_enabled(struct roc_nix *nix);
int __roc_api roc_nix_tm_tree_type_get(struct roc_nix *nix);

/* Ingress Policer API */
int __roc_api roc_nix_bpf_timeunit_get(struct roc_nix *roc_nix,

error: patch failed: drivers/common/cnxk/roc_nix.h:646
Checking patch drivers/common/cnxk/roc_nix_priv.h...
error: while searching for:
#define NIX_TM_CHAN_INVALID UINT16_MAX

/* TM flags */
#define NIX_TM_HIERARCHY_ENA BIT_ULL(0)
#define NIX_TM_TL1_NO_SP     BIT_ULL(1)
#define NIX_TM_TL1_ACCESS    BIT_ULL(2)

struct nix_tm_tb {
	/** Token bucket rate (bytes per second) */

error: patch failed: drivers/common/cnxk/roc_nix_priv.h:36
Hunk #2 succeeded at 166 (offset -17 lines).
error: while searching for:
int nix_tm_bp_config_set(struct roc_nix *roc_nix, uint16_t sq, uint16_t tc,
			 bool enable);
void nix_rq_vwqe_flush(struct roc_nix_rq *rq, uint16_t vwqe_interval);

/*
 * TM priv utils.

error: patch failed: drivers/common/cnxk/roc_nix_priv.h:384
Checking patch drivers/common/cnxk/roc_nix_tm.c...
Hunk #1 succeeded at 1531 (offset -161 lines).
Checking patch drivers/common/cnxk/roc_nix_tm_mark.c...
Checking patch drivers/common/cnxk/version.map...
Hunk #1 succeeded at 250 (offset -3 lines).
Applied patch drivers/common/cnxk/meson.build cleanly.
Applying patch drivers/common/cnxk/roc_nix.h with 1 reject...
Hunk #1 applied cleanly.
Rejected hunk #2.
Applying patch drivers/common/cnxk/roc_nix_priv.h with 2 rejects...
Rejected hunk #1.
Hunk #2 applied cleanly.
Rejected hunk #3.
Applied patch drivers/common/cnxk/roc_nix_tm.c cleanly.
Applied patch drivers/common/cnxk/roc_nix_tm_mark.c cleanly.
Applied patch drivers/common/cnxk/version.map cleanly.
diff a/drivers/common/cnxk/roc_nix.h b/drivers/common/cnxk/roc_nix.h	(rejected hunks)
@@ -646,6 +662,11 @@ int __roc_api roc_nix_tm_node_name_get(struct roc_nix *roc_nix,
 int __roc_api roc_nix_tm_pfc_prepare_tree(struct roc_nix *roc_nix);
 bool __roc_api roc_nix_tm_is_user_hierarchy_enabled(struct roc_nix *nix);
 int __roc_api roc_nix_tm_tree_type_get(struct roc_nix *nix);
+int __roc_api roc_nix_tm_mark_config(struct roc_nix *roc_nix,
+				     enum roc_nix_tm_mark type, int mark_yellow,
+				     int mark_red);
+uint64_t __roc_api roc_nix_tm_mark_format_get(struct roc_nix *roc_nix,
+					      uint64_t *flags);
 
 /* Ingress Policer API */
 int __roc_api roc_nix_bpf_timeunit_get(struct roc_nix *roc_nix,
diff a/drivers/common/cnxk/roc_nix_priv.h b/drivers/common/cnxk/roc_nix_priv.h	(rejected hunks)
@@ -36,9 +36,22 @@ struct nix_qint {
 #define NIX_TM_CHAN_INVALID UINT16_MAX
 
 /* TM flags */
-#define NIX_TM_HIERARCHY_ENA BIT_ULL(0)
-#define NIX_TM_TL1_NO_SP     BIT_ULL(1)
-#define NIX_TM_TL1_ACCESS    BIT_ULL(2)
+#define NIX_TM_HIERARCHY_ENA	BIT_ULL(0)
+#define NIX_TM_TL1_NO_SP	BIT_ULL(1)
+#define NIX_TM_TL1_ACCESS	BIT_ULL(2)
+#define NIX_TM_MARK_VLAN_DEI_EN BIT_ULL(3)
+#define NIX_TM_MARK_IP_DSCP_EN	BIT_ULL(4)
+#define NIX_TM_MARK_IP_ECN_EN	BIT_ULL(5)
+
+#define NIX_TM_MARK_EN_MASK                                                    \
+	(NIX_TM_MARK_IP_DSCP_EN | NIX_TM_MARK_IP_ECN_EN |                      \
+	 NIX_TM_MARK_VLAN_DEI_EN)
+
+#define NIX_TM_MARK_VLAN_DEI_SHIFT  0 /* Leave 16b for VLAN for FP logic */
+#define NIX_TM_MARK_IPV4_DSCP_SHIFT 16
+#define NIX_TM_MARK_IPV6_DSCP_SHIFT 24
+#define NIX_TM_MARK_IPV4_ECN_SHIFT  32
+#define NIX_TM_MARK_IPV6_ECN_SHIFT  40
 
 struct nix_tm_tb {
 	/** Token bucket rate (bytes per second) */
@@ -384,6 +400,7 @@ int nix_rq_cfg(struct dev *dev, struct roc_nix_rq *rq, uint16_t qints, bool cfg,
 int nix_tm_bp_config_set(struct roc_nix *roc_nix, uint16_t sq, uint16_t tc,
 			 bool enable);
 void nix_rq_vwqe_flush(struct roc_nix_rq *rq, uint16_t vwqe_interval);
+int nix_tm_mark_init(struct nix *nix);
 
 /*
  * TM priv utils.
Checking patch drivers/event/cnxk/cn10k_worker.h...
Hunk #1 succeeded at 490 (offset -9 lines).
Checking patch drivers/event/cnxk/cn9k_worker.h...
Hunk #1 succeeded at 764 (offset 2 lines).
Checking patch drivers/net/cnxk/cn10k_ethdev.c...
Hunk #1 succeeded at 107 (offset -3 lines).
Hunk #2 succeeded at 169 (offset -3 lines).
Hunk #3 succeeded at 207 (offset -3 lines).
Hunk #4 succeeded at 457 (offset -30 lines).
error: while searching for:
		cn10k_nix_rx_metadata_negotiate;
}

static void
npc_flow_ops_override(void)
{

error: patch failed: drivers/net/cnxk/cn10k_ethdev.c:501
Hunk #6 succeeded at 629 (offset -48 lines).
Checking patch drivers/net/cnxk/cn10k_ethdev.h...
Checking patch drivers/net/cnxk/cn10k_tx.h...
Checking patch drivers/net/cnxk/cn10k_tx_select.c...
Hunk #1 succeeded at 49 (offset -4 lines).
Checking patch drivers/net/cnxk/cn9k_ethdev.c...
Hunk #1 succeeded at 107 (offset -3 lines).
Hunk #2 succeeded at 168 (offset -3 lines).
Hunk #3 succeeded at 205 (offset -3 lines).
Hunk #4 succeeded at 455 (offset -24 lines).
error: while searching for:
	cnxk_eth_dev_ops.rx_metadata_negotiate = cn9k_nix_rx_metadata_negotiate;
}

static void
npc_flow_ops_override(void)
{

error: patch failed: drivers/net/cnxk/cn9k_ethdev.c:494
Hunk #6 succeeded at 628 (offset -41 lines).
Checking patch drivers/net/cnxk/cn9k_ethdev.h...
Checking patch drivers/net/cnxk/cn9k_tx.c...
Hunk #1 succeeded at 75 (offset -4 lines).
Checking patch drivers/net/cnxk/cn9k_tx.h...
Checking patch drivers/net/cnxk/cn9k_tx_select.c...
Hunk #1 succeeded at 45 (offset -4 lines).
Checking patch drivers/net/cnxk/cnxk_ethdev.h...
error: while searching for:

#define CNXK_NIX_PFC_CHAN_COUNT 16

struct cnxk_fc_cfg {
	enum rte_eth_fc_mode mode;
	uint8_t rx_pause;

error: patch failed: drivers/net/cnxk/cnxk_ethdev.h:139
error: while searching for:
	uint16_t flags;
	uint8_t ptype_disable;
	bool scalar_ena;
	bool ptp_en;
	bool rx_mark_update; /* Enable/Disable mark update to mbuf */


error: patch failed: drivers/net/cnxk/cnxk_ethdev.h:350
Hunk #4 succeeded at 446 (offset -25 lines).
Hunk #5 succeeded at 521 (offset -29 lines).
Checking patch drivers/net/cnxk/cnxk_tm.c...
Applied patch drivers/event/cnxk/cn10k_worker.h cleanly.
Applied patch drivers/event/cnxk/cn9k_worker.h cleanly.
Applying patch drivers/net/cnxk/cn10k_ethdev.c with 1 reject...
Hunk #1 applied cleanly.
Hunk #2 applied cleanly.
Hunk #3 applied cleanly.
Hunk #4 applied cleanly.
Rejected hunk #5.
Hunk #6 applied cleanly.
Applied patch drivers/net/cnxk/cn10k_ethdev.h cleanly.
Applied patch drivers/net/cnxk/cn10k_tx.h cleanly.
Applied patch drivers/net/cnxk/cn10k_tx_select.c cleanly.
Applying patch drivers/net/cnxk/cn9k_ethdev.c with 1 reject...
Hunk #1 applied cleanly.
Hunk #2 applied cleanly.
Hunk #3 applied cleanly.
Hunk #4 applied cleanly.
Rejected hunk #5.
Hunk #6 applied cleanly.
Applied patch drivers/net/cnxk/cn9k_ethdev.h cleanly.
Applied patch drivers/net/cnxk/cn9k_tx.c cleanly.
Applied patch drivers/net/cnxk/cn9k_tx.h cleanly.
Applied patch drivers/net/cnxk/cn9k_tx_select.c cleanly.
Applying patch drivers/net/cnxk/cnxk_ethdev.h with 2 rejects...
Hunk #1 applied cleanly.
Rejected hunk #2.
Rejected hunk #3.
Hunk #4 applied cleanly.
Hunk #5 applied cleanly.
Applied patch drivers/net/cnxk/cnxk_tm.c cleanly.
diff a/drivers/net/cnxk/cn10k_ethdev.c b/drivers/net/cnxk/cn10k_ethdev.c	(rejected hunks)
@@ -501,6 +622,22 @@
 		cn10k_nix_rx_metadata_negotiate;
 }
 
+/* Update platform specific tm ops */
+static void
+nix_tm_ops_override(void)
+{
+	static int init_once;
+
+	if (init_once)
+		return;
+	init_once = 1;
+
+	/* Update platform specific ops */
+	cnxk_tm_ops.mark_vlan_dei = cn10k_nix_tm_mark_vlan_dei;
+	cnxk_tm_ops.mark_ip_ecn = cn10k_nix_tm_mark_ip_ecn;
+	cnxk_tm_ops.mark_ip_dscp = cn10k_nix_tm_mark_ip_dscp;
+}
+
 static void
 npc_flow_ops_override(void)
 {
diff a/drivers/net/cnxk/cn9k_ethdev.c b/drivers/net/cnxk/cn9k_ethdev.c	(rejected hunks)
@@ -494,6 +614,22 @@
 	cnxk_eth_dev_ops.rx_metadata_negotiate = cn9k_nix_rx_metadata_negotiate;
 }
 
+/* Update platform specific eth dev ops */
+static void
+nix_tm_ops_override(void)
+{
+	static int init_once;
+
+	if (init_once)
+		return;
+	init_once = 1;
+
+	/* Update platform specific ops */
+	cnxk_tm_ops.mark_vlan_dei = cn9k_nix_tm_mark_vlan_dei;
+	cnxk_tm_ops.mark_ip_ecn = cn9k_nix_tm_mark_ip_ecn;
+	cnxk_tm_ops.mark_ip_dscp = cn9k_nix_tm_mark_ip_dscp;
+}
+
 static void
 npc_flow_ops_override(void)
 {
diff a/drivers/net/cnxk/cnxk_ethdev.h b/drivers/net/cnxk/cnxk_ethdev.h	(rejected hunks)
@@ -139,6 +140,15 @@
 
 #define CNXK_NIX_PFC_CHAN_COUNT 16
 
+#define CNXK_TM_MARK_VLAN_DEI BIT_ULL(0)
+#define CNXK_TM_MARK_IP_DSCP  BIT_ULL(1)
+#define CNXK_TM_MARK_IP_ECN   BIT_ULL(2)
+
+#define CNXK_TM_MARK_MASK                                                      \
+	(CNXK_TM_MARK_VLAN_DEI | CNXK_TM_MARK_IP_DSCP | CNXK_TM_MARK_IP_ECN)
+
+#define CNXK_TX_MARK_FMT_MASK (0xFFFFFFFFFFFFull)
+
 struct cnxk_fc_cfg {
 	enum rte_eth_fc_mode mode;
 	uint8_t rx_pause;
@@ -350,6 +360,7 @@ struct cnxk_eth_dev {
 	uint16_t flags;
 	uint8_t ptype_disable;
 	bool scalar_ena;
+	bool tx_mark;
 	bool ptp_en;
 	bool rx_mark_update; /* Enable/Disable mark update to mbuf */
 
Checking patch drivers/common/cnxk/roc_nix_tm.c...
error: drivers/common/cnxk/roc_nix_tm.c: does not match index
Checking patch drivers/common/cnxk/roc_nix_tm_ops.c...
error: while searching for:
			continue;

		rc = nix_tm_bp_config_set(roc_nix, sq->qid, 0, false);
		if (rc) {
			plt_err("Failed to disable backpressure, rc=%d", rc);
			goto cleanup;
		}

error: patch failed: drivers/common/cnxk/roc_nix_tm_ops.c:474
Applying patch drivers/common/cnxk/roc_nix_tm_ops.c with 1 reject...
Rejected hunk #1.
diff a/drivers/common/cnxk/roc_nix_tm_ops.c b/drivers/common/cnxk/roc_nix_tm_ops.c	(rejected hunks)
@@ -474,7 +474,7 @@
 			continue;
 
 		rc = nix_tm_bp_config_set(roc_nix, sq->qid, 0, false);
-		if (rc) {
+		if (rc && rc != -ENOENT) {
 			plt_err("Failed to disable backpressure, rc=%d", rc);
 			goto cleanup;
 		}

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

UNH-IOL DPDK Community Lab


More information about the test-report mailing list