Hi Qiming,

-----Original Message-----
From: Yang, Qiming
Sent: Friday, January 13, 2017 9:40 AM
To: De Lara Guarch, Pablo; dev@dpdk.org
Subject: RE: [dpdk-dev] [PATCH v5] app/testpmd: supported offload
capabilities query

Hi, Pablo

-----Original Message-----
From: De Lara Guarch, Pablo
Sent: Friday, January 13, 2017 4:43 PM
To: Yang, Qiming <redacted>; dev@dpdk.org
Cc: Yang, Qiming <redacted>
Subject: RE: [dpdk-dev] [PATCH v5] app/testpmd: supported offload
capabilities query

Hi Qiming,

quoted

-----Original Message-----
From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Qiming Yang
Sent: Thursday, January 12, 2017 3:26 AM
To: dev@dpdk.org
Cc: Yang, Qiming
Subject: [dpdk-dev] [PATCH v5] app/testpmd: supported offload
capabilities query

Add two new commands "show port cap <port>" and "show port cap

all"to

quoted

diaplay what offload capabilities supported in ports. It will not only
display all the capabilities of the port, but also the enabling
condition for each capability in the running time.

Signed-off-by: Qiming Yang <redacted>
Acked-by: Jingjing Wu <redacted>
Acked-by: Beilei Xing <redacted>
---
v2 changes:
* fixed the output style as Ferruh's patch show and add some
  description in docs for new functions.
v3 changes:
* add new command in cmd_help_long_parsed.
v4 changes:
* use 'cap' instead of 'capa'.
v5 changes:
* rebased, fixed the inappropriate expression and adjusted the
  output order.
---
---
 app/test-pmd/cmdline.c                      |  17 ++-
 app/test-pmd/config.c                       | 175
++++++++++++++++++++++++++++
 app/test-pmd/testpmd.h                      |   1 +
 doc/guides/testpmd_app_ug/testpmd_funcs.rst |  12 +-
 4 files changed, 195 insertions(+), 10 deletions(-)

diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c index

4e8b0d8..6fa1783 100644

--- a/app/test-pmd/cmdline.c
+++ b/app/test-pmd/cmdline.c

...

quoted

+
+	if (dev_info.rx_offload_capa & DEV_RX_OFFLOAD_TCP_CKSUM) {
+		printf("RX TCP checksum:               ");
+		if (dev->data->dev_conf.rxmode.hw_ip_checksum)
+			printf("on\n");
+		else
+			printf("off\n");
+	}
+
+	if (dev_info.rx_offload_capa &
DEV_RX_OFFLOAD_OUTER_IPV4_CKSUM)
+		printf("RX Outer IPv4 checksum:        ");

Missing on/off?
Qiming: I didn't find any switch for this feature in DPDK now. I'll fix the
format problem. Thank you.

I just saw your v6. So, what does this mean? If the device is capable of doing this,
but there is no way to enable/disable it, it would mean that it will be enabled or disabled by default, right?

I see that X550 NIC is the only one capable of this. Maybe Wenzhuo can help on clarifying this.

I would say that, if it is enabled when it is capable, then you should print "on",
But printing only "RX Outer IPv4 checksum" does not give any information (when all the rest of the capabilities show on/off).

Thanks,
Pablo