DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] vhost: support Explicit Congestion Notification
@ 2017-11-22  3:19 Jiayu Hu
  2018-01-11 14:27 ` Yuanhan Liu
  0 siblings, 1 reply; 2+ messages in thread
From: Jiayu Hu @ 2017-11-22  3:19 UTC (permalink / raw)
  To: dev; +Cc: yliu, jianfeng.tan, Jiayu Hu

In virtio, Explicit Congestion Notification (ECN) includes two parts:
guest ECN and host ECN. Guest ECN means the frontend can handle TSO
packets which have ECN set, and host ECN means the backend can handle
TSO packets which have ECN set.

The ECN features are rarely used. However, virtio-net enables them by
default, and vhost-net support both. To make live migration from
vhost-net to vhost-user possible, this patch announces to support
guest and host ECN in vhost-user.

Signed-off-by: Jiayu Hu <jiayu.hu@intel.com>
---
 lib/librte_vhost/vhost.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/librte_vhost/vhost.h b/lib/librte_vhost/vhost.h
index 1cc81c1..5d0ed88 100644
--- a/lib/librte_vhost/vhost.h
+++ b/lib/librte_vhost/vhost.h
@@ -206,10 +206,12 @@ struct vhost_msg {
 				(1ULL << VHOST_USER_F_PROTOCOL_FEATURES) | \
 				(1ULL << VIRTIO_NET_F_HOST_TSO4) | \
 				(1ULL << VIRTIO_NET_F_HOST_TSO6) | \
+				(1ULL << VIRTIO_NET_F_HOST_ECN) | \
 				(1ULL << VIRTIO_NET_F_CSUM)    | \
 				(1ULL << VIRTIO_NET_F_GUEST_CSUM) | \
 				(1ULL << VIRTIO_NET_F_GUEST_TSO4) | \
 				(1ULL << VIRTIO_NET_F_GUEST_TSO6) | \
+				(1ULL << VIRTIO_NET_F_GUEST_ECN) | \
 				(1ULL << VIRTIO_RING_F_INDIRECT_DESC) | \
 				(1ULL << VIRTIO_NET_F_MTU) | \
 				(1ULL << VIRTIO_F_IOMMU_PLATFORM))
-- 
2.7.4

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] vhost: support Explicit Congestion Notification
  2017-11-22  3:19 [dpdk-dev] [PATCH] vhost: support Explicit Congestion Notification Jiayu Hu
@ 2018-01-11 14:27 ` Yuanhan Liu
  0 siblings, 0 replies; 2+ messages in thread
From: Yuanhan Liu @ 2018-01-11 14:27 UTC (permalink / raw)
  To: Jiayu Hu; +Cc: dev, jianfeng.tan

On Wed, Nov 22, 2017 at 11:19:42AM +0800, Jiayu Hu wrote:
> In virtio, Explicit Congestion Notification (ECN) includes two parts:
> guest ECN and host ECN. Guest ECN means the frontend can handle TSO
> packets which have ECN set, and host ECN means the backend can handle
> TSO packets which have ECN set.
> 
> The ECN features are rarely used. However, virtio-net enables them by
> default, and vhost-net support both. To make live migration from
> vhost-net to vhost-user possible, this patch announces to support
> guest and host ECN in vhost-user.
> 
> Signed-off-by: Jiayu Hu <jiayu.hu@intel.com>

Applied to dpdk-next-virtio.

Thanks.

	--yliu

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2018-01-11 14:27 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-22  3:19 [dpdk-dev] [PATCH] vhost: support Explicit Congestion Notification Jiayu Hu
2018-01-11 14:27 ` Yuanhan Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).