DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: add VxLAN GRO to release notes
@ 2018-02-09  5:28 Jiayu Hu
  2018-02-09  9:56 ` Mcnamara, John
  0 siblings, 1 reply; 4+ messages in thread
From: Jiayu Hu @ 2018-02-09  5:28 UTC (permalink / raw)
  To: dev; +Cc: john.mcnamara, thomas, Jiayu Hu

Signed-off-by: Jiayu Hu <jiayu.hu@intel.com>
---
 doc/guides/rel_notes/release_18_02.rst | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/doc/guides/rel_notes/release_18_02.rst b/doc/guides/rel_notes/release_18_02.rst
index c42a1d6..c2078b5 100644
--- a/doc/guides/rel_notes/release_18_02.rst
+++ b/doc/guides/rel_notes/release_18_02.rst
@@ -230,6 +230,15 @@ New Features
   current build system using ``make``. For instructions on how to do a DPDK build
   using the new system, see the instructions in ``doc/build-sdk-meson.txt``.
 
+* **Add GRO support for VxLAN-tunneled packets.**
+
+  Add GRO support for VxLAN-tunneled packets. Supported VxLAN packets
+  must contain an outer IPv4 header and inner TCP/IPv4 headers. VxLAN
+  GRO doesn't check if input packets have correct checksums and doesn't
+  update checksums for output packets. Additionally, it assumes the
+  packets are complete (i.e., MF==0 && frag_off==0), when IP
+  fragmentation is possible (i.e., DF==0).
+
 .. note::
 
     This new build system support is incomplete at this point and is added
-- 
2.7.4

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

* Re: [dpdk-dev] [PATCH] doc: add VxLAN GRO to release notes
  2018-02-09  5:28 [dpdk-dev] [PATCH] doc: add VxLAN GRO to release notes Jiayu Hu
@ 2018-02-09  9:56 ` Mcnamara, John
  2018-02-09 10:02   ` Thomas Monjalon
  0 siblings, 1 reply; 4+ messages in thread
From: Mcnamara, John @ 2018-02-09  9:56 UTC (permalink / raw)
  To: Hu, Jiayu, dev; +Cc: thomas



> -----Original Message-----
> From: Hu, Jiayu
> Sent: Friday, February 9, 2018 5:29 AM
> To: dev@dpdk.org
> Cc: Mcnamara, John <john.mcnamara@intel.com>; thomas@monjalon.net; Hu,
> Jiayu <jiayu.hu@intel.com>
> Subject: [PATCH] doc: add VxLAN GRO to release notes
> 
> Signed-off-by: Jiayu Hu <jiayu.hu@intel.com>
> ---
>  doc/guides/rel_notes/release_18_02.rst | 9 +++++++++
>  1 file changed, 9 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/release_18_02.rst
> b/doc/guides/rel_notes/release_18_02.rst
> index c42a1d6..c2078b5 100644
> --- a/doc/guides/rel_notes/release_18_02.rst
> +++ b/doc/guides/rel_notes/release_18_02.rst
> @@ -230,6 +230,15 @@ New Features
>    current build system using ``make``. For instructions on how to do a
> DPDK build
>    using the new system, see the instructions in ``doc/build-sdk-
> meson.txt``.
> 
> +* **Add GRO support for VxLAN-tunneled packets.**
> +
> +  Add GRO support for VxLAN-tunneled packets. Supported VxLAN packets
> + must contain an outer IPv4 header and inner TCP/IPv4 headers. VxLAN
> + GRO doesn't check if input packets have correct checksums and doesn't
> + update checksums for output packets. Additionally, it assumes the
> + packets are complete (i.e., MF==0 && frag_off==0), when IP
> + fragmentation is possible (i.e., DF==0).
> +
>  .. note::
> 
>      This new build system support is incomplete at this point and is
> added

The text has been added between the previous section and a note belonging
to the previous section.

However, we can fix this in the final revision of the release notes, so:

Acked-by: John McNamara <john.mcnamara@intel.com>

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

* Re: [dpdk-dev] [PATCH] doc: add VxLAN GRO to release notes
  2018-02-09  9:56 ` Mcnamara, John
@ 2018-02-09 10:02   ` Thomas Monjalon
  2018-02-13 23:09     ` Thomas Monjalon
  0 siblings, 1 reply; 4+ messages in thread
From: Thomas Monjalon @ 2018-02-09 10:02 UTC (permalink / raw)
  To: Mcnamara, John; +Cc: Hu, Jiayu, dev, xiao.w.wang

09/02/2018 10:56, Mcnamara, John:
> The text has been added between the previous section and a note belonging
> to the previous section.
> 
> However, we can fix this in the final revision of the release notes, so:

I can fix it.
I also take care of keeping a logical order in release notes features.
For example, drivers are grouped together by class, and libraries are after.

General comment about release notes:
Please try to update release notes while updating the code.
It is better for 3 reasons:
	- less chance of forgetting
	- release notes is up-to-date in RC1
	- it is better linked in the git history

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

* Re: [dpdk-dev] [PATCH] doc: add VxLAN GRO to release notes
  2018-02-09 10:02   ` Thomas Monjalon
@ 2018-02-13 23:09     ` Thomas Monjalon
  0 siblings, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2018-02-13 23:09 UTC (permalink / raw)
  To: Hu, Jiayu; +Cc: dev, Mcnamara, John, xiao.w.wang

09/02/2018 11:02, Thomas Monjalon:
> 09/02/2018 10:56, Mcnamara, John:
> > The text has been added between the previous section and a note belonging
> > to the previous section.
> > 
> > However, we can fix this in the final revision of the release notes, so:
> 
> I can fix it.
> I also take care of keeping a logical order in release notes features.
> For example, drivers are grouped together by class, and libraries are after.

Applied, thanks

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

end of thread, other threads:[~2018-02-13 23:09 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-09  5:28 [dpdk-dev] [PATCH] doc: add VxLAN GRO to release notes Jiayu Hu
2018-02-09  9:56 ` Mcnamara, John
2018-02-09 10:02   ` Thomas Monjalon
2018-02-13 23:09     ` Thomas Monjalon

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).