[dpdk-dev] [PATCH 04/11] lib/librte_net: add ESP header to generic flow steering

[Boris Pismenny]

Hi Hemant,

> On 9/15/2017 07:51 AM, Hemant Agrawal wrote:
> Hi Boris,
> 
> On 9/14/2017 1:56 PM, Akhil Goyal wrote:
> > From: Boris Pismenny <borisp at mellanox.com>
> >
> > The ESP header is required for IPsec crypto actions.
> >
> 
> > Signed-off-by: Boris Pismenny <borisp at mellanox.com>
> > Signed-off-by: Aviad Yehezkel <aviadye at mellanox.com>
> > ---
> >  doc/api/doxy-api-index.md   |  1 +
> >  lib/librte_ether/rte_flow.h | 26 ++++++++++++++++++++
> >  lib/librte_net/Makefile     |  2 +-
> >  lib/librte_net/rte_esp.h    | 60
> +++++++++++++++++++++++++++++++++++++++++++++
> >  4 files changed, 88 insertions(+), 1 deletion(-)  create mode 100644
> > lib/librte_net/rte_esp.h
> >
> > diff --git a/doc/api/doxy-api-index.md b/doc/api/doxy-api-index.md
> > index a3d39fc..23bafbc 100644
> > --- a/doc/api/doxy-api-index.md
> > +++ b/doc/api/doxy-api-index.md
> > @@ -107,6 +107,7 @@ The public API headers are grouped by topics:
> >    [LPM IPv6 route]     (@ref rte_lpm6.h),
> >    [ACL]                (@ref rte_acl.h),
> >    [EFD]                (@ref rte_efd.h)
> > +  [ESP]                (@ref rte_esp.h)
> >
> >  - **QoS**:
> >    [metering]           (@ref rte_meter.h),
> > diff --git a/lib/librte_ether/rte_flow.h b/lib/librte_ether/rte_flow.h
> > index bba6169..ea08af6 100644
> > --- a/lib/librte_ether/rte_flow.h
> > +++ b/lib/librte_ether/rte_flow.h
> > @@ -50,6 +50,7 @@
> >  #include <rte_tcp.h>
> >  #include <rte_udp.h>
> >  #include <rte_byteorder.h>
> > +#include <rte_esp.h>
> >
> >  #ifdef __cplusplus
> >  extern "C" {
> > @@ -309,6 +310,13 @@ enum rte_flow_item_type {
> >  	 * See struct rte_flow_item_fuzzy.
> >  	 */
> >  	RTE_FLOW_ITEM_TYPE_FUZZY,
> > +
> > +	/**
> > +	 * Matches a ESP header.
> > +	 *
> > +	 * See struct rte_flow_item_esp.
> > +	 */
> > +	RTE_FLOW_ITEM_TYPE_ESP,
> >  };
> 
> will you please also add changes for rte_flow documentation in prog_guide?
> 

Sure, I'll send the patches in the next couple of days.