| Message ID | 1496941940-10298-3-git-send-email-ktraynor@redhat.com |
|---|---|
| State | Accepted |
| Headers | show |
On Thu, Jun 08, 2017 at 06:12:20PM +0100, Kevin Traynor wrote: > Currently ovs-appctl dpctl/show only shows the Rx checksum offload > status when true. Change to also show the status when false. > > CC: Sugesh Chandran <sugesh.chandran@intel.com> > Signed-off-by: Kevin Traynor <ktraynor@redhat.com> This patch looks obviously correct to me, so I applied it to master. I'll let others, who know DPDK better, review patch 1.
diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c index 79afda5..c4f32ac 100644 --- a/lib/netdev-dpdk.c +++ b/lib/netdev-dpdk.c @@ -1103,4 +1103,6 @@ netdev_dpdk_get_config(const struct netdev *netdev, struct smap *args) if (dev->hw_ol_features & NETDEV_RX_CHECKSUM_OFFLOAD) { smap_add(args, "rx_csum_offload", "true"); + } else { + smap_add(args, "rx_csum_offload", "false"); } }
Currently ovs-appctl dpctl/show only shows the Rx checksum offload status when true. Change to also show the status when false. CC: Sugesh Chandran <sugesh.chandran@intel.com> Signed-off-by: Kevin Traynor <ktraynor@redhat.com> --- lib/netdev-dpdk.c | 2 ++ 1 file changed, 2 insertions(+)