From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas@monjalon.net>
Subject: Re: [PATCH v3] ip_frag: extend
	rte_ipv6_frag_get_ipv6_fragment_header()
Date: Sun, 28 Oct 2018 11:21:53 +0100
Message-ID: <1549031.gagrcBMtR4@xps>
References: <20180727135243.147744-1-doucette@bu.edu>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org, Gaetan Rivet <gaetan.rivet@6wind.com>,
 Konstantin Ananyev <konstantin.ananyev@intel.com>,
 Olivier Matz <olivier.matz@6wind.com>,
 Cristian Dumitrescu <cristian.dumitrescu@intel.com>, michel@digirati.com.br,
 Qiaobin Fu <qiaobinf@bu.edu>,
 Konstantin Ananyev <konstantin.ananyev@intel.com>
To: Cody Doucette <doucette@bu.edu>
Return-path: <dev-bounces@dpdk.org>
Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com
 [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id A29133195
 for <dev@dpdk.org>; Sun, 28 Oct 2018 11:21:50 +0100 (CET)
In-Reply-To: <20180727135243.147744-1-doucette@bu.edu>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <https://mails.dpdk.org/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://mails.dpdk.org/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <https://mails.dpdk.org/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

27/07/2018 15:52, Cody Doucette:
> Extend rte_ipv6_frag_get_ipv6_fragment_header() to skip over any
> other IPv6 extension headers when finding the fragment header.
> 
> According to RFC 8200, there is no guarantee that the IPv6
> Fragment extension header will come before any other extension
> header, even though it is recommended.
> 
> Signed-off-by: Cody Doucette <doucette@bu.edu>
> Signed-off-by: Qiaobin Fu <qiaobinf@bu.edu>
> Reviewed-by: Michel Machado <michel@digirati.com.br>
> ---
> v3:
> * Removed compilation flag D_XOPEN_SOURCE=700 from the
>   failsafe driver to allow compilation on freebsd.

How failsafe is related to ip_frag?


> v2:
> * Moved IPv6 extension header definitions to lib_net.
> 
>  drivers/net/failsafe/Makefile               |  1 -
>  drivers/net/failsafe/meson.build            |  1 -
>  examples/ip_reassembly/main.c               |  6 ++--
>  lib/librte_ip_frag/rte_ip_frag.h            | 23 ++++++-------
>  lib/librte_ip_frag/rte_ip_frag_version.map  |  1 +
>  lib/librte_ip_frag/rte_ipv6_fragmentation.c | 38 +++++++++++++++++++++
>  lib/librte_ip_frag/rte_ipv6_reassembly.c    |  4 +--
>  lib/librte_net/rte_ip.h                     | 27 +++++++++++++++
>  lib/librte_port/rte_port_ras.c              |  6 ++--

Changes in failsafe, rte_net and rte_port look like garbage.

Anyway, the ip_frag part requires some review.
+Cc Konstantin, the maintainer.