[dpdk-dev] [PATCH v4 03/10] net: add ESP trailer structure definition
Olivier Matz
olivier.matz at 6wind.com
Thu Dec 27 11:13:49 CET 2018
Hi,
On Wed, Dec 19, 2018 at 09:32:09AM +0000, Akhil Goyal wrote:
>
>
> On 12/14/2018 9:53 PM, Konstantin Ananyev wrote:
> > Signed-off-by: Konstantin Ananyev <konstantin.ananyev at intel.com>
> > Acked-by: Mohammad Abdul Awal <mohammad.abdul.awal at intel.com>
> > Acked-by: Declan Doherty <declan.doherty at intel.com>
> > ---
> > lib/librte_net/rte_esp.h | 10 +++++++++-
> > 1 file changed, 9 insertions(+), 1 deletion(-)
> >
> > diff --git a/lib/librte_net/rte_esp.h b/lib/librte_net/rte_esp.h
> > index f77ec2eb2..8e1b3d2dd 100644
> > --- a/lib/librte_net/rte_esp.h
> > +++ b/lib/librte_net/rte_esp.h
> > @@ -11,7 +11,7 @@
> > * ESP-related defines
> > */
> >
> > -#include <stdint.h>
> > +#include <rte_byteorder.h>
> >
> > #ifdef __cplusplus
> > extern "C" {
> > @@ -25,6 +25,14 @@ struct esp_hdr {
> > rte_be32_t seq; /**< packet sequence number */
> > } __attribute__((__packed__));
> >
> > +/**
> > + * ESP Trailer
> > + */
> > +struct esp_tail {
> > + uint8_t pad_len; /**< number of pad bytes (0-255) */
> > + uint8_t next_proto; /**< IPv4 or IPv6 or next layer header */
> > +} __attribute__((__packed__));
> > +
> > #ifdef __cplusplus
> > }
> > #endif
> Acked-by: Akhil Goyal <akhil.goyal at nxp.com>
Is there a reason to pack the structure? I think it has no impact since
it is only composed of uint8_t, so it can be removed.
Thanks,
Olivier
More information about the dev
mailing list