Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/34945/?format=api
{ "id": 34945, "url": "http://patches.dpdk.org/api/patches/34945/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1517819520-12673-2-git-send-email-beilei.xing@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<1517819520-12673-2-git-send-email-beilei.xing@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1517819520-12673-2-git-send-email-beilei.xing@intel.com", "date": "2018-02-05T08:31:58", "name": "[dpdk-dev,v4,1/4] net/i40e: add warnings when writing global registers", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "2c60c8d61e6f9063dd486190d5e680bfcca32b6a", "submitter": { "id": 410, "url": "http://patches.dpdk.org/api/people/410/?format=api", "name": "Xing, Beilei", "email": "beilei.xing@intel.com" }, "delegate": { "id": 24, "url": "http://patches.dpdk.org/api/users/24/?format=api", "username": "helin_zhang", "first_name": "Helin", "last_name": "Zhang", "email": "helin.zhang@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1517819520-12673-2-git-send-email-beilei.xing@intel.com/mbox/", "series": [], "comments": "http://patches.dpdk.org/api/patches/34945/comments/", "check": "fail", "checks": "http://patches.dpdk.org/api/patches/34945/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id B91231B2FE;\n\tMon, 5 Feb 2018 09:31:28 +0100 (CET)", "from mga09.intel.com (mga09.intel.com [134.134.136.24])\n\tby dpdk.org (Postfix) with ESMTP id 1A07D1B29D;\n\tMon, 5 Feb 2018 09:31:25 +0100 (CET)", "from orsmga006.jf.intel.com ([10.7.209.51])\n\tby orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t05 Feb 2018 00:31:23 -0800", "from unknown (HELO dpdk9.sh.intel.com) ([10.67.118.52])\n\tby orsmga006.jf.intel.com with ESMTP; 05 Feb 2018 00:31:22 -0800" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.46,464,1511856000\"; d=\"scan'208\";a=\"15896411\"", "From": "Beilei Xing <beilei.xing@intel.com>", "To": "jingjing.wu@intel.com,\n\tdev@dpdk.org,\n\tstable@dpdk.org", "Date": "Mon, 5 Feb 2018 16:31:58 +0800", "Message-Id": "<1517819520-12673-2-git-send-email-beilei.xing@intel.com>", "X-Mailer": "git-send-email 2.5.5", "In-Reply-To": "<1517819520-12673-1-git-send-email-beilei.xing@intel.com>", "References": "<1517574310-93096-1-git-send-email-beilei.xing@intel.com>\n\t<1517819520-12673-1-git-send-email-beilei.xing@intel.com>", "Subject": "[dpdk-dev] [PATCH v4 1/4] net/i40e: add warnings when writing\n\tglobal registers", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://dpdk.org/ml/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "Add warnings when writing global registers.\n\nSigned-off-by: Beilei Xing <beilei.xing@intel.com>\nCc: stable@dpdk.org\n---\n doc/guides/nics/i40e.rst | 12 ++++++++++++\n drivers/net/i40e/i40e_ethdev.c | 15 +++++++++++++++\n drivers/net/i40e/i40e_ethdev.h | 43 ++++++++++++++++++++++++++++++++++++++++++\n 3 files changed, 70 insertions(+)", "diff": "diff --git a/doc/guides/nics/i40e.rst b/doc/guides/nics/i40e.rst\nindex 5780268..68a546b 100644\n--- a/doc/guides/nics/i40e.rst\n+++ b/doc/guides/nics/i40e.rst\n@@ -459,3 +459,15 @@ Receive packets with Ethertype 0x88A8\n \n Due to the FW limitation, PF can receive packets with Ethertype 0x88A8\n only when floating VEB is disabled.\n+\n+Global configuration warning\n+~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n+\n+I40E PMD will set some global registers to enable some function or set some\n+configure. Then when using different ports of the same NIC with Linux kernel\n+and DPDK, the port with Linux kernel will be impacted by the port with DPDK.\n+For example, register I40E_GL_SWT_L2TAGCTRL is used to control L2 tag, i40e\n+PMD uses I40E_GL_SWT_L2TAGCTRL to set vlan TPID. If setting TPID in port A\n+with DPDK, then the configuration will also impact port B in the NIC with\n+kernel driver, which don't want to use the TPID.\n+So PMD reports warning to clarify what is changed by writing global register.\ndiff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c\nindex 0835c2d..44821f2 100644\n--- a/drivers/net/i40e/i40e_ethdev.c\n+++ b/drivers/net/i40e/i40e_ethdev.c\n@@ -738,6 +738,7 @@ static inline void i40e_GLQF_reg_init(struct i40e_hw *hw)\n \tI40E_WRITE_REG(hw, I40E_GLQF_ORT(33), 0x000000E0);\n \tI40E_WRITE_REG(hw, I40E_GLQF_ORT(34), 0x000000E3);\n \tI40E_WRITE_REG(hw, I40E_GLQF_ORT(35), 0x000000E6);\n+\ti40e_global_cfg_warning(I40E_WARNING_ENA_FLX_PLD);\n \n \t/*\n \t * Initialize registers for parsing packet type of QinQ\n@@ -747,6 +748,7 @@ static inline void i40e_GLQF_reg_init(struct i40e_hw *hw)\n \t */\n \tI40E_WRITE_REG(hw, I40E_GLQF_ORT(40), 0x00000029);\n \tI40E_WRITE_REG(hw, I40E_GLQF_PIT(9), 0x00009420);\n+\ti40e_global_cfg_warning(I40E_WARNING_QINQ_PARSER);\n }\n \n #define I40E_FLOW_CONTROL_ETHERTYPE 0x8808\n@@ -2800,6 +2802,8 @@ i40e_vlan_tpid_set(struct rte_eth_dev *dev,\n \tPMD_DRV_LOG(DEBUG, \"Debug write 0x%08\"PRIx64\" to \"\n \t\t \"I40E_GL_SWT_L2TAGCTRL[%d]\", reg_w, reg_id);\n \n+\ti40e_global_cfg_warning(I40E_WARNING_TPID);\n+\n \treturn ret;\n }\n \n@@ -3038,6 +3042,7 @@ i40e_flow_ctrl_set(struct rte_eth_dev *dev, struct rte_eth_fc_conf *fc_conf)\n \tI40E_WRITE_REG(hw, I40E_GLRPB_GLW,\n \t\t pf->fc_conf.low_water[I40E_MAX_TRAFFIC_CLASS]\n \t\t << I40E_KILOSHIFT);\n+\ti40e_global_cfg_warning(I40E_WARNING_FLOW_CTL);\n \n \tI40E_WRITE_FLUSH(hw);\n \n@@ -6875,6 +6880,7 @@ i40e_dev_set_gre_key_len(struct i40e_hw *hw, uint8_t len)\n \t\t\t\t\t\t reg, NULL);\n \t\tif (ret != 0)\n \t\t\treturn ret;\n+\t\ti40e_global_cfg_warning(I40E_WARNING_GRE_KEY_LEN);\n \t} else {\n \t\tret = 0;\n \t}\n@@ -7154,6 +7160,7 @@ i40e_set_hash_filter_global_config(struct i40e_hw *hw,\n \t\t} else {\n \t\t\ti40e_write_rx_ctl(hw, I40E_GLQF_HSYM(pctype), reg);\n \t\t}\n+\t\ti40e_global_cfg_warning(I40E_WARNING_HSYM);\n \t}\n \n \treg = i40e_read_rx_ctl(hw, I40E_GLQF_CTL);\n@@ -7178,6 +7185,7 @@ i40e_set_hash_filter_global_config(struct i40e_hw *hw,\n \t\tgoto out;\n \n \ti40e_write_rx_ctl(hw, I40E_GLQF_CTL, reg);\n+\ti40e_global_cfg_warning(I40E_WARNING_QF_CTL);\n \n out:\n \tI40E_WRITE_FLUSH(hw);\n@@ -7848,6 +7856,10 @@ i40e_filter_input_set_init(struct i40e_pf *pf)\n \t\tpf->hash_input_set[pctype] = input_set;\n \t\tpf->fdir.input_set[pctype] = input_set;\n \t}\n+\n+\ti40e_global_cfg_warning(I40E_WARNING_HASH_INSET);\n+\ti40e_global_cfg_warning(I40E_WARNING_FD_MSK);\n+\ti40e_global_cfg_warning(I40E_WARNING_HASH_MSK);\n }\n \n int\n@@ -7913,6 +7925,7 @@ i40e_hash_filter_inset_select(struct i40e_hw *hw,\n \ti40e_check_write_reg(hw, I40E_GLQF_HASH_INSET(1, pctype),\n \t\t\t (uint32_t)((inset_reg >>\n \t\t\t I40E_32_BIT_WIDTH) & UINT32_MAX));\n+\ti40e_global_cfg_warning(I40E_WARNING_HASH_INSET);\n \n \tfor (i = 0; i < num; i++)\n \t\ti40e_check_write_reg(hw, I40E_GLQF_HASH_MSK(i, pctype),\n@@ -7921,6 +7934,7 @@ i40e_hash_filter_inset_select(struct i40e_hw *hw,\n \tfor (i = num; i < I40E_INSET_MASK_NUM_REG; i++)\n \t\ti40e_check_write_reg(hw, I40E_GLQF_HASH_MSK(i, pctype),\n \t\t\t\t 0);\n+\ti40e_global_cfg_warning(I40E_WARNING_HASH_MSK);\n \tI40E_WRITE_FLUSH(hw);\n \n \tpf->hash_input_set[pctype] = input_set;\n@@ -7999,6 +8013,7 @@ i40e_fdir_filter_inset_select(struct i40e_pf *pf,\n \tfor (i = num; i < I40E_INSET_MASK_NUM_REG; i++)\n \t\ti40e_check_write_reg(hw, I40E_GLQF_FD_MSK(i, pctype),\n \t\t\t\t 0);\n+\ti40e_global_cfg_warning(I40E_WARNING_FD_MSK);\n \tI40E_WRITE_FLUSH(hw);\n \n \tpf->fdir.input_set[pctype] = input_set;\ndiff --git a/drivers/net/i40e/i40e_ethdev.h b/drivers/net/i40e/i40e_ethdev.h\nindex f283319..1d813ef 100644\n--- a/drivers/net/i40e/i40e_ethdev.h\n+++ b/drivers/net/i40e/i40e_ethdev.h\n@@ -569,6 +569,22 @@ struct i40e_adapter {\n \tstruct rte_timecounter tx_tstamp_tc;\n };\n \n+enum I40E_WARNING_IDX {\n+\tI40E_WARNING_DIS_FLX_PLD,\n+\tI40E_WARNING_ENA_FLX_PLD,\n+\tI40E_WARNING_QINQ_PARSER,\n+\tI40E_WARNING_QINQ_CLOUD_FILTER,\n+\tI40E_WARNING_TPID,\n+\tI40E_WARNING_FLOW_CTL,\n+\tI40E_WARNING_GRE_KEY_LEN,\n+\tI40E_WARNING_QF_CTL,\n+\tI40E_WARNING_HASH_INSET,\n+\tI40E_WARNING_HSYM,\n+\tI40E_WARNING_HASH_MSK,\n+\tI40E_WARNING_FD_MSK,\n+\tI40E_WARNING_RPL_CLD_FILTER,\n+};\n+\n int i40e_dev_switch_queues(struct i40e_pf *pf, bool on);\n int i40e_vsi_release(struct i40e_vsi *vsi);\n struct i40e_vsi *i40e_vsi_setup(struct i40e_pf *pf,\n@@ -703,6 +719,33 @@ i40e_calc_itr_interval(int16_t interval)\n \treturn interval / 2;\n }\n \n+static inline void\n+i40e_global_cfg_warning(enum I40E_WARNING_IDX idx)\n+{\n+\tconst char *warning;\n+\tstatic const char *const warning_list[] = {\n+\t\t[I40E_WARNING_DIS_FLX_PLD] = \"disable FDIR flexible payload\",\n+\t\t[I40E_WARNING_ENA_FLX_PLD] = \"enable FDIR flexible payload\",\n+\t\t[I40E_WARNING_QINQ_PARSER] = \"support QinQ parser\",\n+\t\t[I40E_WARNING_QINQ_CLOUD_FILTER] = \"support QinQ cloud filter\",\n+\t\t[I40E_WARNING_TPID] = \"support TPID configuration\",\n+\t\t[I40E_WARNING_FLOW_CTL] = \"configure water marker\",\n+\t\t[I40E_WARNING_GRE_KEY_LEN] = \"support GRE key length setting\",\n+\t\t[I40E_WARNING_QF_CTL] = \"support hash function setting\",\n+\t\t[I40E_WARNING_HASH_INSET] = \"configure hash input set\",\n+\t\t[I40E_WARNING_HSYM] = \"set symmetric hash\",\n+\t\t[I40E_WARNING_HASH_MSK] = \"configure hash mask\",\n+\t\t[I40E_WARNING_FD_MSK] = \"configure fdir mask\",\n+\t\t[I40E_WARNING_RPL_CLD_FILTER] = \"replace cloud filter\",\n+\t};\n+\n+\twarning = warning_list[idx];\n+\n+\tRTE_LOG(WARNING, PMD,\n+\t\t\"Global register is changed during %s\\n\",\n+\t\twarning);\n+}\n+\n #define I40E_VALID_FLOW(flow_type) \\\n \t((flow_type) == RTE_ETH_FLOW_FRAG_IPV4 || \\\n \t(flow_type) == RTE_ETH_FLOW_NONFRAG_IPV4_TCP || \\\n", "prefixes": [ "dpdk-dev", "v4", "1/4" ] }