Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/11572/?format=api
{ "id": 11572, "url": "https://patches.dpdk.org/api/patches/11572/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/1458238145-7496-9-git-send-email-thomas.monjalon@6wind.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": "<1458238145-7496-9-git-send-email-thomas.monjalon@6wind.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1458238145-7496-9-git-send-email-thomas.monjalon@6wind.com", "date": "2016-03-17T18:09:05", "name": "[dpdk-dev,v11,8/8] ethdev: add 100G link speed", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "77b336c6020a240b21d3906fd9cf173e400f09e5", "submitter": { "id": 1, "url": "https://patches.dpdk.org/api/people/1/?format=api", "name": "Thomas Monjalon", "email": "thomas.monjalon@6wind.com" }, "delegate": { "id": 1, "url": "https://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/1458238145-7496-9-git-send-email-thomas.monjalon@6wind.com/mbox/", "series": [], "comments": "https://patches.dpdk.org/api/patches/11572/comments/", "check": "pending", "checks": "https://patches.dpdk.org/api/patches/11572/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 [IPv6:::1])\n\tby dpdk.org (Postfix) with ESMTP id 1D14E5A43;\n\tThu, 17 Mar 2016 19:11:08 +0100 (CET)", "from mail-wm0-f43.google.com (mail-wm0-f43.google.com\n\t[74.125.82.43]) by dpdk.org (Postfix) with ESMTP id 24F485A17\n\tfor <dev@dpdk.org>; Thu, 17 Mar 2016 19:10:59 +0100 (CET)", "by mail-wm0-f43.google.com with SMTP id p65so37771019wmp.1\n\tfor <dev@dpdk.org>; Thu, 17 Mar 2016 11:10:59 -0700 (PDT)", "from XPS13.localdomain (91.111.75.86.rev.sfr.net. [86.75.111.91])\n\tby smtp.gmail.com with ESMTPSA id\n\tu14sm31196179wmu.8.2016.03.17.11.10.56\n\t(version=TLSv1/SSLv3 cipher=OTHER);\n\tThu, 17 Mar 2016 11:10:58 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=6wind-com.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=k3mLuUmP7hRjHNZLaMDfeFDNyC9fgky8hkTq7OwH/VQ=;\n\tb=J44GFdDpmXUMvDmxVu63mF1Pap4F5txPl0er2BMpmgVadkyNbD1qWBiu/cKUUPOz7z\n\trQM7HgYM1q2IxvzcpKrMSA/ri/31G0PO5pyMHRlGt6K8jvUDo8o336nFb0RpSQRqa+o/\n\trv846ny7meLfMrfIV4m9n/v9WMyGCuYCc2IJUGfa5WhP3U0xa4Jl1ZNccRyMOOwmoRs9\n\ttGUv3KTG5DQ8OnE1qKtYp3GOt5VScQatPwYNzU6DAQhnZey7pkHEsSdjeAVz64CyOY/i\n\twf55RrHL3e5mtGUZV7/ZiYr7XatC5hQkzjSqfSartB5k41UNfb7x37ZtXQ7PY54It4yg\n\tw4sQ==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20130820;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=k3mLuUmP7hRjHNZLaMDfeFDNyC9fgky8hkTq7OwH/VQ=;\n\tb=SuC55Uox0NOr/tj7h5rowoapL1FwCGBDCd5bZGdPZw3oMPPOSjTFN8LX0unH7M6us/\n\tkj7GDx4wWP0LtdcDNbuQ6bu0JQuNyZ3otnuLApKorgz7XjEvbverJyJgKErvtoJONuGc\n\tJpvlifmJZO9P6jWRs0KBPr+irwkW5jOBVKWs4bZINNIZLzlrSQxbiZ1gaKqrvvZDuZeJ\n\tNyeDrPrPiCPLm+xY0lS/5eFU4LBPBJPaOEj7iJZtajHa52GB0py3LNV0rC8VhIMYOsFA\n\to0OIJBRpfg34EvdDQq/c5oBY2pJDIoiBiH0frghpYDARB6hieHcjPh+VlF/CrCJiYGVA\n\t1TJA==", "X-Gm-Message-State": "AD7BkJLQIyd2EKiQYddxEzxLDm0g41m1LZFI2cLkw9wF52JVmWXpsclDtPbu3xX+U+3WOGAp", "X-Received": "by 10.28.105.77 with SMTP id e74mr12843027wmc.87.1458238258986; \n\tThu, 17 Mar 2016 11:10:58 -0700 (PDT)", "From": "Thomas Monjalon <thomas.monjalon@6wind.com>", "To": "marcdevel@gmail.com, bruce.richardson@intel.com, declan.doherty@intel.com,\n\tkonstantin.ananyev@intel.com, wenzhuo.lu@intel.com,\n\thelin.zhang@intel.com, jing.d.chen@intel.com, harish.patil@qlogic.com,\n\trahul.lakkireddy@chelsio.com, johndale@cisco.com, vido@cesnet.cz,\n\tadrien.mazarguil@6wind.com, alejandro.lucero@netronome.com", "Cc": "dev@dpdk.org", "Date": "Thu, 17 Mar 2016 19:09:05 +0100", "Message-Id": "<1458238145-7496-9-git-send-email-thomas.monjalon@6wind.com>", "X-Mailer": "git-send-email 2.7.0", "In-Reply-To": "<1458238145-7496-1-git-send-email-thomas.monjalon@6wind.com>", "References": "<1457992546-32230-1-git-send-email-thomas.monjalon@6wind.com>\n\t<1458238145-7496-1-git-send-email-thomas.monjalon@6wind.com>", "Subject": "[dpdk-dev] [PATCH v11 8/8] ethdev: add 100G link speed", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "patches and discussions about DPDK <dev.dpdk.org>", "List-Unsubscribe": "<http://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": "<http://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": "The link speed configuration is now done with bitmaps so 100G speed\nrequires only a new bit flag.\nThe actual link speed is a number so its size must be increased from\n16-bit to 32-bit.\n\nSigned-off-by: Marc Sune <marcdevel@gmail.com>\nTested-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com>\nSigned-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>\n---\n app/test-pmd/cmdline.c | 12 +++++++-----\n doc/guides/nics/szedata2.rst | 6 ------\n doc/guides/rel_notes/release_16_04.rst | 5 +++++\n doc/guides/testpmd_app_ug/testpmd_funcs.rst | 2 +-\n drivers/net/fm10k/fm10k_ethdev.c | 2 +-\n drivers/net/mlx5/mlx5_ethdev.c | 2 +-\n drivers/net/nfp/nfp_net.c | 2 +-\n drivers/net/szedata2/rte_eth_szedata2.c | 9 ++-------\n lib/librte_ether/rte_ethdev.c | 2 ++\n lib/librte_ether/rte_ethdev.h | 4 +++-\n 10 files changed, 23 insertions(+), 23 deletions(-)", "diff": "diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c\nindex 3bc7bb4..3337b7b 100644\n--- a/app/test-pmd/cmdline.c\n+++ b/app/test-pmd/cmdline.c\n@@ -549,7 +549,7 @@ static void cmd_help_long_parsed(void *parsed_result,\n \t\t\t\" Detach physical or virtual dev by port_id\\n\\n\"\n \n \t\t\t\"port config (port_id|all)\"\n-\t\t\t\" speed (10|100|1000|10000|40000|auto)\"\n+\t\t\t\" speed (10|100|1000|10000|40000|100000|auto)\"\n \t\t\t\" duplex (half|full|auto)\\n\"\n \t\t\t\" Set speed and duplex for all ports or port_id\\n\\n\"\n \n@@ -1017,6 +1017,8 @@ parse_and_check_speed_duplex(char *speedstr, char *duplexstr, uint32_t *speed)\n \t\t\t*speed = ETH_LINK_SPEED_10G;\n \t\t} else if (!strcmp(speedstr, \"40000\")) {\n \t\t\t*speed = ETH_LINK_SPEED_40G;\n+\t\t} else if (!strcmp(speedstr, \"100000\")) {\n+\t\t\t*speed = ETH_LINK_SPEED_100G;\n \t\t} else if (!strcmp(speedstr, \"auto\")) {\n \t\t\t*speed = ETH_LINK_SPEED_AUTONEG;\n \t\t} else {\n@@ -1064,7 +1066,7 @@ cmdline_parse_token_string_t cmd_config_speed_all_item1 =\n \tTOKEN_STRING_INITIALIZER(struct cmd_config_speed_all, item1, \"speed\");\n cmdline_parse_token_string_t cmd_config_speed_all_value1 =\n \tTOKEN_STRING_INITIALIZER(struct cmd_config_speed_all, value1,\n-\t\t\t\t\t\t\"10#100#1000#10000#40000#auto\");\n+\t\t\t\t\t\t\"10#100#1000#10000#40000#100000#auto\");\n cmdline_parse_token_string_t cmd_config_speed_all_item2 =\n \tTOKEN_STRING_INITIALIZER(struct cmd_config_speed_all, item2, \"duplex\");\n cmdline_parse_token_string_t cmd_config_speed_all_value2 =\n@@ -1074,7 +1076,7 @@ cmdline_parse_token_string_t cmd_config_speed_all_value2 =\n cmdline_parse_inst_t cmd_config_speed_all = {\n \t.f = cmd_config_speed_all_parsed,\n \t.data = NULL,\n-\t.help_str = \"port config all speed 10|100|1000|10000|40000|auto duplex \"\n+\t.help_str = \"port config all speed 10|100|1000|10000|40000|100000|auto duplex \"\n \t\t\t\t\t\t\t\"half|full|auto\",\n \t.tokens = {\n \t\t(void *)&cmd_config_speed_all_port,\n@@ -1138,7 +1140,7 @@ cmdline_parse_token_string_t cmd_config_speed_specific_item1 =\n \t\t\t\t\t\t\t\t\"speed\");\n cmdline_parse_token_string_t cmd_config_speed_specific_value1 =\n \tTOKEN_STRING_INITIALIZER(struct cmd_config_speed_specific, value1,\n-\t\t\t\t\t\t\"10#100#1000#10000#40000#auto\");\n+\t\t\t\t\t\t\"10#100#1000#10000#40000#100000#auto\");\n cmdline_parse_token_string_t cmd_config_speed_specific_item2 =\n \tTOKEN_STRING_INITIALIZER(struct cmd_config_speed_specific, item2,\n \t\t\t\t\t\t\t\t\"duplex\");\n@@ -1149,7 +1151,7 @@ cmdline_parse_token_string_t cmd_config_speed_specific_value2 =\n cmdline_parse_inst_t cmd_config_speed_specific = {\n \t.f = cmd_config_speed_specific_parsed,\n \t.data = NULL,\n-\t.help_str = \"port config X speed 10|100|1000|10000|40000|auto duplex \"\n+\t.help_str = \"port config X speed 10|100|1000|10000|40000|100000|auto duplex \"\n \t\t\t\t\t\t\t\"half|full|auto\",\n \t.tokens = {\n \t\t(void *)&cmd_config_speed_specific_port,\ndiff --git a/doc/guides/nics/szedata2.rst b/doc/guides/nics/szedata2.rst\nindex 77c15b3..741b400 100644\n--- a/doc/guides/nics/szedata2.rst\n+++ b/doc/guides/nics/szedata2.rst\n@@ -148,9 +148,3 @@ Example output:\n TX threshold registers: pthresh=0 hthresh=0 wthresh=0\n TX RS bit threshold=0 - TXQ flags=0x0\n testpmd>\n-\n-.. note::\n-\n- Link speed API currently supports speeds up to 40 Gbps.\n- Therefore there is used 10G constant for 100 Gbps cards until the link speed\n- API is not changed.\ndiff --git a/doc/guides/rel_notes/release_16_04.rst b/doc/guides/rel_notes/release_16_04.rst\nindex 6540941..e6a9144 100644\n--- a/doc/guides/rel_notes/release_16_04.rst\n+++ b/doc/guides/rel_notes/release_16_04.rst\n@@ -279,6 +279,11 @@ EAL\n Drivers\n ~~~~~~~\n \n+* **ethdev: Fixed overflow for 100Gbps.**\n+\n+ 100Gbps in Mbps (100000) was exceeding 16-bit max value of ``link_speed``\n+ in ``rte_eth_link``.\n+\n * **ethdev: Fixed byte order consistency between fdir flow and mask.**\n \n Fixed issue in ethdev library that the structure for setting\ndiff --git a/doc/guides/testpmd_app_ug/testpmd_funcs.rst b/doc/guides/testpmd_app_ug/testpmd_funcs.rst\nindex efaf2b5..cb49495 100644\n--- a/doc/guides/testpmd_app_ug/testpmd_funcs.rst\n+++ b/doc/guides/testpmd_app_ug/testpmd_funcs.rst\n@@ -1155,7 +1155,7 @@ port config - speed\n \n Set the speed and duplex mode for all ports or a specific port::\n \n- testpmd> port config (port_id|all) speed (10|100|1000|10000|auto) \\\n+ testpmd> port config (port_id|all) speed (10|100|1000|10000|40000|100000|auto) \\\n duplex (half|full|auto)\n \n port config - queues/descriptors\ndiff --git a/drivers/net/fm10k/fm10k_ethdev.c b/drivers/net/fm10k/fm10k_ethdev.c\nindex 2a1c222..056f3aa 100644\n--- a/drivers/net/fm10k/fm10k_ethdev.c\n+++ b/drivers/net/fm10k/fm10k_ethdev.c\n@@ -1413,7 +1413,7 @@ fm10k_dev_infos_get(struct rte_eth_dev *dev,\n \n \tdev_info->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_2_5G |\n \t\t\tETH_LINK_SPEED_10G | ETH_LINK_SPEED_25G |\n-\t\t\tETH_LINK_SPEED_40G;\n+\t\t\tETH_LINK_SPEED_40G | ETH_LINK_SPEED_100G;\n }\n \n static int\ndiff --git a/drivers/net/mlx5/mlx5_ethdev.c b/drivers/net/mlx5/mlx5_ethdev.c\nindex 3487538..1f9ba98 100644\n--- a/drivers/net/mlx5/mlx5_ethdev.c\n+++ b/drivers/net/mlx5/mlx5_ethdev.c\n@@ -524,7 +524,7 @@ mlx5_dev_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *info)\n \tinfo->reta_size = priv->ind_table_max_size;\n \tinfo->speed_capa = ETH_LINK_SPEED_1G | ETH_LINK_SPEED_10G |\n \t\t\tETH_LINK_SPEED_25G | ETH_LINK_SPEED_40G |\n-\t\t\tETH_LINK_SPEED_50G;\n+\t\t\tETH_LINK_SPEED_50G | ETH_LINK_SPEED_100G;\n \tpriv_unlock(priv);\n }\n \ndiff --git a/drivers/net/nfp/nfp_net.c b/drivers/net/nfp/nfp_net.c\nindex c9e1dc5..fbf6737 100644\n--- a/drivers/net/nfp/nfp_net.c\n+++ b/drivers/net/nfp/nfp_net.c\n@@ -1062,7 +1062,7 @@ nfp_net_infos_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info)\n \tdev_info->reta_size = NFP_NET_CFG_RSS_ITBL_SZ;\n \tdev_info->hash_key_size = NFP_NET_CFG_RSS_KEY_SZ;\n \n-\tdev_info->speed_capa = ETH_LINK_SPEED_50G;\n+\tdev_info->speed_capa = ETH_LINK_SPEED_50G | ETH_LINK_SPEED_100G;\n }\n \n static uint32_t\ndiff --git a/drivers/net/szedata2/rte_eth_szedata2.c b/drivers/net/szedata2/rte_eth_szedata2.c\nindex 0708e4a..916333f 100644\n--- a/drivers/net/szedata2/rte_eth_szedata2.c\n+++ b/drivers/net/szedata2/rte_eth_szedata2.c\n@@ -1036,6 +1036,7 @@ eth_dev_info(struct rte_eth_dev *dev,\n \tdev_info->max_rx_queues = internals->max_rx_queues;\n \tdev_info->max_tx_queues = internals->max_tx_queues;\n \tdev_info->min_rx_bufsize = 0;\n+\tdev_info->speed_capa = ETH_LINK_SPEED_100G;\n }\n \n static void\n@@ -1155,13 +1156,7 @@ eth_link_update(struct rte_eth_dev *dev,\n \t\tlink.link_speed = ETH_SPEED_NUM_40G;\n \t\tbreak;\n \tcase SZEDATA2_LINK_SPEED_100G:\n-\t\t/*\n-\t\t * TODO\n-\t\t * If link_speed value from rte_eth_link structure\n-\t\t * will be changed to support 100Gbps speed change\n-\t\t * this value to 100G.\n-\t\t */\n-\t\tlink.link_speed = ETH_SPEED_NUM_10G;\n+\t\tlink.link_speed = ETH_SPEED_NUM_100G;\n \t\tbreak;\n \tdefault:\n \t\tlink.link_speed = ETH_SPEED_NUM_10G;\ndiff --git a/lib/librte_ether/rte_ethdev.c b/lib/librte_ether/rte_ethdev.c\nindex 4dbea4e..62937cb 100644\n--- a/lib/librte_ether/rte_ethdev.c\n+++ b/lib/librte_ether/rte_ethdev.c\n@@ -892,6 +892,8 @@ rte_eth_speed_bitflag(uint32_t speed, int duplex)\n \t\treturn ETH_LINK_SPEED_50G;\n \tcase ETH_SPEED_NUM_56G:\n \t\treturn ETH_LINK_SPEED_56G;\n+\tcase ETH_SPEED_NUM_100G:\n+\t\treturn ETH_LINK_SPEED_100G;\n \tdefault:\n \t\treturn 0;\n \t}\ndiff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h\nindex 4b37cd0..73ded7c 100644\n--- a/lib/librte_ether/rte_ethdev.h\n+++ b/lib/librte_ether/rte_ethdev.h\n@@ -259,6 +259,7 @@ struct rte_eth_stats {\n #define ETH_LINK_SPEED_40G (1 << 11) /**< 40 Gbps */\n #define ETH_LINK_SPEED_50G (1 << 12) /**< 50 Gbps */\n #define ETH_LINK_SPEED_56G (1 << 13) /**< 56 Gbps */\n+#define ETH_LINK_SPEED_100G (1 << 14) /**< 100 Gbps */\n \n /**\n * Ethernet numeric link speeds in Mbps\n@@ -275,12 +276,13 @@ struct rte_eth_stats {\n #define ETH_SPEED_NUM_40G 40000 /**< 40 Gbps */\n #define ETH_SPEED_NUM_50G 50000 /**< 50 Gbps */\n #define ETH_SPEED_NUM_56G 56000 /**< 56 Gbps */\n+#define ETH_SPEED_NUM_100G 100000 /**< 100 Gbps */\n \n /**\n * A structure used to retrieve link-level information of an Ethernet port.\n */\n struct rte_eth_link {\n-\tuint16_t link_speed; /**< ETH_SPEED_NUM_ */\n+\tuint32_t link_speed; /**< ETH_SPEED_NUM_ */\n \tuint16_t link_duplex : 1; /**< ETH_LINK_[HALF/FULL]_DUPLEX */\n \tuint16_t link_autoneg : 1; /**< ETH_LINK_SPEED_[AUTONEG/FIXED] */\n \tuint16_t link_status : 1; /**< ETH_LINK_[DOWN/UP] */\n", "prefixes": [ "dpdk-dev", "v11", "8/8" ] }