Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/94858/?format=api
{ "id": 94858, "url": "https://patches.dpdk.org/api/patches/94858/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20210625093157.24436-4-ting.xu@intel.com/", "project": { "id": 1, "url": "https://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": "<20210625093157.24436-4-ting.xu@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20210625093157.24436-4-ting.xu@intel.com", "date": "2021-06-25T09:31:55", "name": "[v3,3/5] net/ice: support DCF link status event handling", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "bc243ba1ee447975544a326eecc5b3538f049671", "submitter": { "id": 1363, "url": "https://patches.dpdk.org/api/people/1363/?format=api", "name": "Xu, Ting", "email": "ting.xu@intel.com" }, "delegate": { "id": 1540, "url": "https://patches.dpdk.org/api/users/1540/?format=api", "username": "qzhan15", "first_name": "Qi", "last_name": "Zhang", "email": "qi.z.zhang@intel.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20210625093157.24436-4-ting.xu@intel.com/mbox/", "series": [ { "id": 17486, "url": "https://patches.dpdk.org/api/series/17486/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=17486", "date": "2021-06-25T09:31:52", "name": "Enable ETS-based Tx QoS for VF in DCF", "version": 3, "mbox": "https://patches.dpdk.org/series/17486/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/94858/comments/", "check": "success", "checks": "https://patches.dpdk.org/api/patches/94858/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id C5475A0547;\n\tFri, 25 Jun 2021 11:33:52 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id C626B410E6;\n\tFri, 25 Jun 2021 11:33:42 +0200 (CEST)", "from mga17.intel.com (mga17.intel.com [192.55.52.151])\n by mails.dpdk.org (Postfix) with ESMTP id D3ED34068A\n for <dev@dpdk.org>; Fri, 25 Jun 2021 11:33:34 +0200 (CEST)", "from orsmga001.jf.intel.com ([10.7.209.18])\n by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 25 Jun 2021 02:33:30 -0700", "from dpdk-xuting-main.sh.intel.com ([10.67.117.76])\n by orsmga001.jf.intel.com with ESMTP; 25 Jun 2021 02:33:28 -0700" ], "IronPort-SDR": [ "\n gIj8hnLgslPIy0hNmtCB7/V1KSMHLrkNXPrJhH4nRbRLlmnUloOrIRIO2Ud4kbzG787D7U6AHC\n uQvR1N0PQ0Tg==", "\n 3PfKSrzTbyAXK7dKhubm3hKsv8zOXSaJH0MTS9WYN9fj8UBRK1CLjC8oyWFKTsGZm3lFB5/jwd\n IXlNFR8FQg8g==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,10025\"; a=\"188022044\"", "E=Sophos;i=\"5.83,298,1616482800\"; d=\"scan'208\";a=\"188022044\"", "E=Sophos;i=\"5.83,298,1616482800\"; d=\"scan'208\";a=\"488117902\"" ], "X-ExtLoop1": "1", "From": "Ting Xu <ting.xu@intel.com>", "To": "dev@dpdk.org", "Cc": "qi.z.zhang@intel.com, jingjing.wu@intel.com, beilei.xing@intel.com,\n qiming.yang@intel.com", "Date": "Fri, 25 Jun 2021 17:31:55 +0800", "Message-Id": "<20210625093157.24436-4-ting.xu@intel.com>", "X-Mailer": "git-send-email 2.25.1", "In-Reply-To": "<20210625093157.24436-1-ting.xu@intel.com>", "References": "<20210601014034.36100-1-ting.xu@intel.com>\n <20210625093157.24436-1-ting.xu@intel.com>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v3 3/5] net/ice: support DCF link status event\n handling", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "When link status changes, DCF will receive virtchnl PF event message.\nAdd support to handle this event, change link status and update link\ninfo.\n\nSigned-off-by: Ting Xu <ting.xu@intel.com>\n---\n drivers/net/ice/ice_dcf.h | 6 ++++\n drivers/net/ice/ice_dcf_ethdev.c | 54 ++++++++++++++++++++++++++++++--\n drivers/net/ice/ice_dcf_parent.c | 51 ++++++++++++++++++++++++++++++\n 3 files changed, 108 insertions(+), 3 deletions(-)", "diff": "diff --git a/drivers/net/ice/ice_dcf.h b/drivers/net/ice/ice_dcf.h\nindex 0cb90b5e9f..587093b909 100644\n--- a/drivers/net/ice/ice_dcf.h\n+++ b/drivers/net/ice/ice_dcf.h\n@@ -60,6 +60,10 @@ struct ice_dcf_hw {\n \tuint16_t nb_msix;\n \tuint16_t rxq_map[16];\n \tstruct virtchnl_eth_stats eth_stats_offset;\n+\n+\t/* Link status */\n+\tbool link_up;\n+\tuint32_t link_speed;\n };\n \n int ice_dcf_execute_virtchnl_cmd(struct ice_dcf_hw *hw,\n@@ -77,5 +81,7 @@ int ice_dcf_disable_queues(struct ice_dcf_hw *hw);\n int ice_dcf_query_stats(struct ice_dcf_hw *hw,\n \t\t\tstruct virtchnl_eth_stats *pstats);\n int ice_dcf_add_del_all_mac_addr(struct ice_dcf_hw *hw, bool add);\n+int ice_dcf_link_update(struct rte_eth_dev *dev,\n+\t\t __rte_unused int wait_to_complete);\n \n #endif /* _ICE_DCF_H_ */\ndiff --git a/drivers/net/ice/ice_dcf_ethdev.c b/drivers/net/ice/ice_dcf_ethdev.c\nindex f73dc80bd9..0b40ebbec6 100644\n--- a/drivers/net/ice/ice_dcf_ethdev.c\n+++ b/drivers/net/ice/ice_dcf_ethdev.c\n@@ -881,11 +881,59 @@ ice_dcf_dev_close(struct rte_eth_dev *dev)\n \treturn 0;\n }\n \n-static int\n-ice_dcf_link_update(__rte_unused struct rte_eth_dev *dev,\n+int\n+ice_dcf_link_update(struct rte_eth_dev *dev,\n \t\t __rte_unused int wait_to_complete)\n {\n-\treturn 0;\n+\tstruct ice_dcf_adapter *ad = dev->data->dev_private;\n+\tstruct ice_dcf_hw *hw = &ad->real_hw;\n+\tstruct rte_eth_link new_link;\n+\n+\tmemset(&new_link, 0, sizeof(new_link));\n+\n+\t/* Only read status info stored in VF, and the info is updated\n+\t * when receive LINK_CHANGE event from PF by virtchnl.\n+\t */\n+\tswitch (hw->link_speed) {\n+\tcase 10:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_10M;\n+\t\tbreak;\n+\tcase 100:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_100M;\n+\t\tbreak;\n+\tcase 1000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_1G;\n+\t\tbreak;\n+\tcase 10000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_10G;\n+\t\tbreak;\n+\tcase 20000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_20G;\n+\t\tbreak;\n+\tcase 25000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_25G;\n+\t\tbreak;\n+\tcase 40000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_40G;\n+\t\tbreak;\n+\tcase 50000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_50G;\n+\t\tbreak;\n+\tcase 100000:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_100G;\n+\t\tbreak;\n+\tdefault:\n+\t\tnew_link.link_speed = ETH_SPEED_NUM_NONE;\n+\t\tbreak;\n+\t}\n+\n+\tnew_link.link_duplex = ETH_LINK_FULL_DUPLEX;\n+\tnew_link.link_status = hw->link_up ? ETH_LINK_UP :\n+\t\t\t\t\t ETH_LINK_DOWN;\n+\tnew_link.link_autoneg = !(dev->data->dev_conf.link_speeds &\n+\t\t\t\tETH_LINK_SPEED_FIXED);\n+\n+\treturn rte_eth_linkstatus_set(dev, &new_link);\n }\n \n /* Add UDP tunneling port */\ndiff --git a/drivers/net/ice/ice_dcf_parent.c b/drivers/net/ice/ice_dcf_parent.c\nindex 19420a0f58..788f6dd2a0 100644\n--- a/drivers/net/ice/ice_dcf_parent.c\n+++ b/drivers/net/ice/ice_dcf_parent.c\n@@ -178,6 +178,44 @@ start_vsi_reset_thread(struct ice_dcf_hw *dcf_hw, bool vfr, uint16_t vf_id)\n \t}\n }\n \n+static uint32_t\n+ice_dcf_convert_link_speed(enum virtchnl_link_speed virt_link_speed)\n+{\n+\tuint32_t speed;\n+\n+\tswitch (virt_link_speed) {\n+\tcase VIRTCHNL_LINK_SPEED_100MB:\n+\t\tspeed = 100;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_1GB:\n+\t\tspeed = 1000;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_10GB:\n+\t\tspeed = 10000;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_40GB:\n+\t\tspeed = 40000;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_20GB:\n+\t\tspeed = 20000;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_25GB:\n+\t\tspeed = 25000;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_2_5GB:\n+\t\tspeed = 2500;\n+\t\tbreak;\n+\tcase VIRTCHNL_LINK_SPEED_5GB:\n+\t\tspeed = 5000;\n+\t\tbreak;\n+\tdefault:\n+\t\tspeed = 0;\n+\t\tbreak;\n+\t}\n+\n+\treturn speed;\n+}\n+\n void\n ice_dcf_handle_pf_event_msg(struct ice_dcf_hw *dcf_hw,\n \t\t\t uint8_t *msg, uint16_t msglen)\n@@ -196,6 +234,19 @@ ice_dcf_handle_pf_event_msg(struct ice_dcf_hw *dcf_hw,\n \t\tbreak;\n \tcase VIRTCHNL_EVENT_LINK_CHANGE:\n \t\tPMD_DRV_LOG(DEBUG, \"VIRTCHNL_EVENT_LINK_CHANGE event\");\n+\t\tdcf_hw->link_up = pf_msg->event_data.link_event.link_status;\n+\t\tif (dcf_hw->vf_res->vf_cap_flags &\n+\t\t\tVIRTCHNL_VF_CAP_ADV_LINK_SPEED) {\n+\t\t\tdcf_hw->link_speed =\n+\t\t\t\tpf_msg->event_data.link_event_adv.link_speed;\n+\t\t} else {\n+\t\t\tenum virtchnl_link_speed speed;\n+\t\t\tspeed = pf_msg->event_data.link_event.link_speed;\n+\t\t\tdcf_hw->link_speed = ice_dcf_convert_link_speed(speed);\n+\t\t}\n+\t\tice_dcf_link_update(dcf_hw->eth_dev, 0);\n+\t\trte_eth_dev_callback_process(dcf_hw->eth_dev,\n+\t\t\tRTE_ETH_EVENT_INTR_LSC, NULL);\n \t\tbreak;\n \tcase VIRTCHNL_EVENT_PF_DRIVER_CLOSE:\n \t\tPMD_DRV_LOG(DEBUG, \"VIRTCHNL_EVENT_PF_DRIVER_CLOSE event\");\n", "prefixes": [ "v3", "3/5" ] }