@@ -393,6 +393,7 @@ T: git://dpdk.org/next/dpdk-next-net
F: lib/librte_ethdev/
F: devtools/test-null.sh
F: doc/guides/prog_guide/switch_representation.rst
+F: app/test/test_ethdev*
Flow API
M: Ori Kam <orika@mellanox.com>
@@ -251,6 +251,9 @@ SRCS-$(CONFIG_RTE_LIBRTE_SECURITY) += test_security.c
SRCS-$(CONFIG_RTE_LIBRTE_IPSEC) += test_ipsec.c test_ipsec_perf.c
SRCS-$(CONFIG_RTE_LIBRTE_IPSEC) += test_ipsec_sad.c
+
+SRCS-$(CONFIG_RTE_LIBRTE_ETHER) += test_ethdev_link.c
+
ifeq ($(CONFIG_RTE_LIBRTE_IPSEC),y)
LDLIBS += -lrte_ipsec
endif
@@ -39,6 +39,7 @@ test_sources = files('commands.c',
'test_efd.c',
'test_efd_perf.c',
'test_errno.c',
+ 'test_ethdev_link.c',
'test_event_crypto_adapter.c',
'test_event_eth_rx_adapter.c',
'test_event_ring.c',
@@ -199,6 +200,7 @@ fast_tests = [
['eal_flags_misc_autotest', false],
['eal_fs_autotest', true],
['errno_autotest', true],
+ ['ethdev_link_status', true],
['event_ring_autotest', true],
['fib_autotest', true],
['fib6_autotest', true],
new file mode 100644
@@ -0,0 +1,306 @@
+/* SPDX-License-Identifier: BSD-3-Clause
+ * Copyright (c) 2020 Samsung Electronics Co., Ltd All Rights Reserved
+ */
+
+#include <rte_log.h>
+#include <rte_ethdev.h>
+
+#include <rte_test.h>
+#include "test.h"
+
+
+static int32_t
+test_link_status_up_default(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_2_5G,
+ .link_status = ETH_LINK_UP,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_FULL_DUPLEX
+ };
+ char text[128];
+
+ ret = rte_eth_link_to_str(text, 128, NULL, &link_status);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format default string\n");
+ printf("Default link up #1: %s\n", text);
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("Link up at 2.5 Gbit/s FDX Autoneg\n",
+ text, strlen(text), "Invalid default link status string");
+
+ link_status.link_duplex = ETH_LINK_HALF_DUPLEX;
+ link_status.link_autoneg = ETH_LINK_FIXED;
+ link_status.link_speed = ETH_SPEED_NUM_10M,
+ ret = rte_eth_link_to_str(text, 128, NULL, &link_status);
+ printf("Default link up #2: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format default string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("Link up at 10 Mbit/s HDX Fixed\n",
+ text, strlen(text), "Invalid default link status "
+ "string with HDX");
+
+ link_status.link_speed = ETH_SPEED_NUM_UNKNOWN,
+ ret = rte_eth_link_to_str(text, 128, NULL, &link_status);
+ printf("Default link up #3: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format default string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("Link up at Unknown speed HDX Fixed\n",
+ text, strlen(text), "Invalid default link status "
+ "string with HDX");
+ return TEST_SUCCESS;
+}
+
+static int32_t
+test_link_status_down_default(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_2_5G,
+ .link_status = ETH_LINK_DOWN,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_FULL_DUPLEX
+ };
+ char text[128];
+
+ ret = rte_eth_link_to_str(text, 128, NULL, &link_status);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format default string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("Link down\n",
+ text, strlen(text), "Invalid default link status string");
+
+ return TEST_SUCCESS;
+}
+
+static int32_t
+test_link_status_string_overflow(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_2_5G,
+ .link_status = ETH_LINK_UP,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_FULL_DUPLEX
+ };
+ char text[128];
+ int i = 0;
+
+ for (i = 0; i < 128; i++)
+ text[i] = 'Y';
+ text[127] = '\0';
+
+ ret = rte_eth_link_to_str(NULL, 2, "status %S, %G Gbits/s",
+ &link_status);
+ RTE_TEST_ASSERT(ret < 0, "Format string should fail, but it's ok\n");
+
+ ret = rte_eth_link_to_str(text, 2, "status %S, %G Gbits/s",
+ &link_status);
+ RTE_TEST_ASSERT(ret < 0, "Format string should fail, but it's ok\n");
+ RTE_TEST_ASSERT(text[2] == 'Y', "String1 overflow\n");
+
+ ret = rte_eth_link_to_str(text, 8, NULL,
+ &link_status);
+ RTE_TEST_ASSERT(ret < 0, "Default format string should fail,"
+ " but it's ok\n");
+ RTE_TEST_ASSERT(text[8] == 'Y', "String1 overflow\n");
+
+ ret = rte_eth_link_to_str(text, 10, NULL,
+ &link_status);
+ RTE_TEST_ASSERT(ret < 0, "Default format string should fail,"
+ " but it's ok\n");
+ RTE_TEST_ASSERT(text[10] == 'Y', "String1 overflow\n");
+
+ text[1] = 'Y';
+ ret = rte_eth_link_to_str(text, 1, "%S",
+ &link_status);
+ RTE_TEST_ASSERT(ret < 0, "Status string should fail, but it's ok\n");
+ RTE_TEST_ASSERT(text[1] == 'Y', "String1 overflow\n");
+
+ text[1] = 'Y';
+ ret = rte_eth_link_to_str(text, 1, "%s",
+ &link_status);
+ RTE_TEST_ASSERT(ret < 0, "Status string should fail, but it's ok\n");
+ RTE_TEST_ASSERT(text[1] == 'Y', "String1 overflow\n");
+
+
+ return TEST_SUCCESS;
+}
+
+static int32_t
+test_link_status_format(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_40G,
+ .link_status = ETH_LINK_UP,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_FULL_DUPLEX
+ };
+ char text[128];
+ int i = 0;
+
+ for (i = 0; i < 128; i++)
+ text[i] = 'Y';
+ text[127] = '\0';
+ printf("status format #1: %s\n", text);
+ ret = rte_eth_link_to_str(text, 128, "status = %S, duplex = %D\n",
+ &link_status);
+ printf("status format #2: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("status = Up, duplex = FDX\n",
+ text, strlen(text), "Invalid status string1.");
+
+ ret = rte_eth_link_to_str(text, 128, "%A", &link_status);
+ printf("status format #3: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("Autoneg",
+ text, strlen(text), "Invalid status string2.");
+
+ ret = rte_eth_link_to_str(text, 128,
+ "%G",
+ &link_status);
+ printf("status format #4: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("40.0",
+ text, strlen(text), "Invalid status string3.");
+
+ ret = rte_eth_link_to_str(text, 128,
+ "%d %M %",
+ &link_status);
+ printf("status format #5: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("%d 40000 %",
+ text, strlen(text), "Invalid status string4.");
+ return TEST_SUCCESS;
+}
+
+static int32_t
+test_link_status_return_value(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_40G,
+ .link_status = ETH_LINK_UP,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_FULL_DUPLEX
+ };
+ char text[128];
+ int i = 0;
+
+ for (i = 0; i < 128; i++)
+ text[i] = 'Y';
+ text[127] = '\0';
+ ret = rte_eth_link_to_str(text, 128, "status = %S, ",
+ &link_status);
+ printf("return value #1:ret=%u, text=%s\n", ret, text);
+ ret += rte_eth_link_to_str(text + ret, 128 - ret,
+ "%A",
+ &link_status);
+ printf("return value #2:ret=%u, text=%s\n", ret, text);
+ ret += rte_eth_link_to_str(text + ret, 128 - ret,
+ ", duplex = %D\n",
+ &link_status);
+ printf("return value #3:ret=%u, text=%s\n", ret, text);
+ ret += rte_eth_link_to_str(text + ret, 128 - ret,
+ "%M Mbits/s\n",
+ &link_status);
+ printf("return value #4:ret=%u, text=%s\n", ret, text);
+ RTE_TEST_ASSERT(ret > 0, "Failed to format string\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("status = Up, Autoneg, duplex = FDX\n"
+ "40000 Mbits/s\n",
+ text, strlen(text), "Invalid status string");
+
+ return TEST_SUCCESS;
+}
+
+static int32_t
+test_link_status_unknown_specifier(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_40G,
+ .link_status = ETH_LINK_UP,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_FULL_DUPLEX
+ };
+ char text[128];
+
+ ret = rte_eth_link_to_str(text, 128, "status = %",
+ &link_status);
+ RTE_TEST_ASSERT(ret > 0, "Status string1 is failed\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("status = %",
+ text, strlen(text), "Invalid status string1");
+
+ ret = rte_eth_link_to_str(text, 128,
+ ", duplex = %d\n",
+ &link_status);
+ RTE_TEST_ASSERT(ret > 0, "Status string2 is failed\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL(", duplex = %d\n",
+ text, strlen(text), "Invalid status string2");
+
+ ret = rte_eth_link_to_str(text, 128,
+ "% Mbits/s\n",
+ &link_status);
+ RTE_TEST_ASSERT(ret > 0, "Status string3 is failed\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("% Mbits/s\n",
+ text, strlen(text), "Invalid status string3");
+
+ ret = rte_eth_link_to_str(text, 128,
+ "%w Mbits/s\n",
+ &link_status);
+ RTE_TEST_ASSERT(ret > 0, "Status string4 should be ok, but it's fail\n");
+ TEST_ASSERT_BUFFERS_ARE_EQUAL("%w Mbits/s\n",
+ text, strlen(text), "Invalid status string4");
+ return TEST_SUCCESS;
+}
+
+static int32_t
+test_link_status_format_edges(void)
+{
+ int ret = 0;
+ struct rte_eth_link link_status = {
+ .link_speed = ETH_SPEED_NUM_UNKNOWN,
+ .link_status = ETH_LINK_DOWN,
+ .link_autoneg = ETH_LINK_AUTONEG,
+ .link_duplex = ETH_LINK_HALF_DUPLEX
+ };
+ char text[128];
+
+ ret = rte_eth_link_to_str(text, 4, "%S", &link_status);
+ printf("format edges #1: %s\n", text);
+ RTE_TEST_ASSERT(ret < 0, "It should fail. No space for "
+ "zero terminator\n");
+ ret = rte_eth_link_to_str(text, 6, "123%D", &link_status);
+ printf("format edges #2: %s\n", text);
+ RTE_TEST_ASSERT(ret < 0, "It should fail. No space for "
+ "zero terminator\n");
+ ret = rte_eth_link_to_str(text, 7, "%A", &link_status);
+ printf("format edges #3: %s\n", text);
+ RTE_TEST_ASSERT(ret < 0, "It should fail. No space for "
+ "zero terminator\n");
+ ret = rte_eth_link_to_str(text, 8, "%A", &link_status);
+ printf("format edges #4: %s\n", text);
+ RTE_TEST_ASSERT(ret > 0, "It should ok, but it fails\n");
+ return TEST_SUCCESS;
+}
+static struct unit_test_suite link_status_testsuite = {
+ .suite_name = "link status formatting",
+ .setup = NULL,
+ .teardown = NULL,
+ .unit_test_cases = {
+ TEST_CASE(test_link_status_up_default),
+ TEST_CASE(test_link_status_down_default),
+ TEST_CASE(test_link_status_string_overflow),
+ TEST_CASE(test_link_status_format),
+ TEST_CASE(test_link_status_format_edges),
+ TEST_CASE(test_link_status_unknown_specifier),
+ TEST_CASE(test_link_status_return_value),
+ TEST_CASES_END() /**< NULL terminate unit test array */
+ }
+};
+
+static int
+test_link_status(void)
+{
+ rte_log_set_global_level(RTE_LOG_DEBUG);
+ rte_log_set_level(RTE_LOGTYPE_EAL, RTE_LOG_DEBUG);
+
+ return unit_test_suite_runner(&link_status_testsuite);
+}
+
+REGISTER_TEST_COMMAND(ethdev_link_status, test_link_status);
@@ -2383,6 +2383,180 @@ rte_eth_link_get_nowait(uint16_t port_id, struct rte_eth_link *eth_link)
return 0;
}
+static int
+rte_eth_link_to_str_parser(char *str, size_t len, const char *const fmt,
+ const struct rte_eth_link *eth_link)
+{
+ size_t offset = 0;
+ const char *fmt_cur = fmt;
+ double gbits = (double)eth_link->link_speed / 1000.;
+ static const char autoneg_str[] = "Autoneg";
+ static const char fixed_str[] = "Fixed";
+ static const char fdx_str[] = "FDX";
+ static const char hdx_str[] = "HDX";
+ static const char unknown_str[] = "Unknown";
+ static const char up_str[] = "Up";
+ static const char down_str[] = "Down";
+ char gbits_str[20];
+ char mbits_str[20];
+
+ /* preformat complex formatting to easily concatinate it further */
+ snprintf(mbits_str, sizeof(mbits_str), "%u", eth_link->link_speed);
+ snprintf(gbits_str, sizeof(gbits_str), "%.1f", gbits);
+ /* init str before formatting */
+ str[0] = 0;
+ while (*fmt_cur) {
+ /* check str bounds */
+ if (offset >= len) {
+ str[len - 1] = '\0';
+ return -EINVAL;
+ }
+ if (*fmt_cur == '%') {
+ /* set null terminator to current position,
+ * it's required for strlcat
+ */
+ str[offset] = '\0';
+ switch (*++fmt_cur) {
+ /* Speed in Mbits/s */
+ case 'M':
+ if (eth_link->link_speed ==
+ ETH_SPEED_NUM_UNKNOWN)
+ offset = strlcat(str, unknown_str,
+ len);
+ else
+ offset = strlcat(str, mbits_str, len);
+ break;
+ /* Speed in Gbits/s */
+ case 'G':
+ if (eth_link->link_speed ==
+ ETH_SPEED_NUM_UNKNOWN)
+ offset = strlcat(str, unknown_str,
+ len);
+ else
+ offset = strlcat(str, gbits_str, len);
+ break;
+ /* Link status */
+ case 'S':
+ offset = strlcat(str, eth_link->link_status ?
+ up_str : down_str, len);
+ break;
+ /* Link autoneg */
+ case 'A':
+ offset = strlcat(str, eth_link->link_autoneg ?
+ autoneg_str : fixed_str, len);
+ break;
+ /* Link duplex */
+ case 'D':
+ offset = strlcat(str, eth_link->link_duplex ?
+ fdx_str : hdx_str, len);
+ break;
+ /* ignore unknown specifier
+ * just copy it to target str
+ */
+ default:
+ str[offset++] = '%';
+ fmt_cur--;
+ break;
+
+ }
+ if (offset >= len)
+ return -EINVAL;
+
+ } else {
+ str[offset++] = *fmt_cur;
+ }
+ fmt_cur++;
+ }
+ str[offset] = '\0';
+ return offset;
+}
+
+int
+rte_eth_link_printf(const char *const fmt,
+ const struct rte_eth_link *eth_link)
+{
+ char text[200];
+ int ret;
+
+ ret = rte_eth_link_to_str(text, 200, fmt, eth_link);
+ if (ret > 0)
+ printf("%s", text);
+ return ret;
+}
+
+int
+rte_eth_link_to_str(char *str, size_t len, const char *const fmt,
+ const struct rte_eth_link *eth_link)
+{
+ size_t offset = 0;
+ double gbits = (double)eth_link->link_speed / 1000.;
+ char speed_gbits_str[20];
+ char speed_mbits_str[20];
+ /* TBD: make it international? */
+ static const char link_down_str[] = "Link down\n";
+ static const char link_up_str[] = "Link up at ";
+ static const char unknown_speed_str[] = "Unknown speed ";
+ static const char mbits_str[] = "Mbit/s";
+ static const char gbits_str[] = "Gbit/s";
+ static const char fdx_str[] = "FDX ";
+ static const char hdx_str[] = "HDX ";
+ /* autoneg is latest param in default string, so add '\n' */
+ static const char autoneg_str[] = "Autoneg\n";
+ static const char fixed_str[] = "Fixed\n";
+
+ if (str == NULL || len == 0)
+ return -EINVAL;
+ /* default format string, if no fmt is specified */
+ if (fmt == NULL) {
+ if (eth_link->link_status == ETH_LINK_DOWN) {
+ if (sizeof(link_down_str) > len)
+ return -EINVAL;
+ return strlcpy(str, link_down_str, len);
+ }
+
+ /* preformat complex strings to easily concatinate it further */
+ snprintf(speed_mbits_str, sizeof(speed_mbits_str), "%u %s ",
+ eth_link->link_speed, mbits_str);
+ snprintf(speed_gbits_str, sizeof(speed_gbits_str), "%.1f %s ",
+ gbits, gbits_str);
+
+ offset = strlcpy(str, link_up_str, len);
+ /* reserve one byte to null terminator */
+ if (offset >= len)
+ return -EINVAL;
+ /* link speed */
+ if (eth_link->link_speed == ETH_SPEED_NUM_UNKNOWN) {
+ offset = strlcat(str, unknown_speed_str, len);
+ if (offset >= len)
+ return -EINVAL;
+ } else {
+ if (eth_link->link_speed < ETH_SPEED_NUM_1G) {
+ offset = strlcat(str, speed_mbits_str, len);
+ if (offset >= len)
+ return -EINVAL;
+ } else {
+ offset = strlcat(str, speed_gbits_str, len);
+ if (offset >= len)
+ return -EINVAL;
+ }
+ }
+ /* link duplex */
+ offset = strlcat(str, eth_link->link_duplex ?
+ fdx_str : hdx_str, len);
+ if (offset >= len)
+ return -EINVAL;
+ /* link autonegotiation */
+ offset = strlcat(str, eth_link->link_autoneg ?
+ autoneg_str : fixed_str, len);
+ if (offset >= len)
+ return -EINVAL;
+ /* Formatted status */
+ } else
+ offset = rte_eth_link_to_str_parser(str, len, fmt, eth_link);
+
+ return offset;
+}
+
int
rte_eth_stats_get(uint16_t port_id, struct rte_eth_stats *stats)
{
@@ -2295,6 +2295,62 @@ int rte_eth_link_get(uint16_t port_id, struct rte_eth_link *link);
*/
int rte_eth_link_get_nowait(uint16_t port_id, struct rte_eth_link *link);
+
+/**
+ * print formatted link status to stdout. This function treats all
+ * special values like ETH_SPEED_NUM_UNKNOWN, ETH_LINK_DOWN etc. and convert
+ * them to textual representation.
+ *
+ * @param fmt
+ * Format string which allows to format link status.
+ * If NULL is provided, default formatting will be applied.
+ * Following specifiers are available:
+ * - '%M' link speed in Mbits/s
+ * - '%G' link speed in Gbits/s
+ * - '%S' link status. e.g. Up or Down
+ * - '%A' link autonegotiation state
+ * - '%D' link duplex state
+ * @param eth_link
+ * Link status provided by rte_eth_link_get function
+ * @return
+ * - Number of bytes written to stdout.
+ * - (-EINVAL) if fmt is too long
+ *
+ */
+__rte_experimental
+int rte_eth_link_printf(const char *const fmt,
+ const struct rte_eth_link *eth_link);
+
+/**
+ * Format link status to textual representation. This function treats all
+ * special values like ETH_SPEED_NUM_UNKNOWN, ETH_LINK_DOWN etc. and convert
+ * them to textual representation.
+ *
+ * @param str
+ * A pointer to a string to be filled with textual representation of
+ * device status.
+ * @param len
+ * Length of available memory at 'str' string.
+ * @param fmt
+ * Format string which allows to format link status.
+ * If NULL is provided, default formatting will be applied.
+ * Following specifiers are available:
+ * - '%M' link speed in Mbits/s
+ * - '%G' link speed in Gbits/s
+ * - '%S' link status. e.g. Up or Down
+ * - '%A' link autonegotiation state
+ * - '%D' link duplex state
+ * @param eth_link
+ * Link status provided by rte_eth_link_get function
+ * @return
+ * - Number of bytes written to str array.
+ * - (-EINVAL) if size of target buffer is too small for the string
+ *
+ */
+__rte_experimental
+int rte_eth_link_to_str(char *str, size_t len, const char *const fmt,
+ const struct rte_eth_link *eth_link);
+
/**
* Retrieve the general I/O statistics of an Ethernet device.
*
@@ -241,4 +241,8 @@ EXPERIMENTAL {
__rte_ethdev_trace_rx_burst;
__rte_ethdev_trace_tx_burst;
rte_flow_get_aged_flows;
+
+ # added in 20.08
+ rte_eth_link_to_str;
+ rte_eth_link_printf;
};