> -----Original Message----- > From: Stephen Hemminger <step...@networkplumber.org> > Sent: Saturday, September 5, 2020 02:56 > To: Wang, Haiyue <haiyue.w...@intel.com> > Cc: dev@dpdk.org; Olivier Matz <olivier.m...@6wind.com> > Subject: Re: [dpdk-dev] [PATCH v1] net: expand the data size of L3 length to > 16 bits > > On Fri, 4 Sep 2020 13:40:20 +0800 > Haiyue Wang <haiyue.w...@intel.com> wrote: > > > diff --git a/lib/librte_net/rte_net.h b/lib/librte_net/rte_net.h > > index 94b06d9ee..992fb088a 100644 > > --- a/lib/librte_net/rte_net.h > > +++ b/lib/librte_net/rte_net.h > > @@ -20,11 +20,11 @@ extern "C" { > > */ > > struct rte_net_hdr_lens { > > uint8_t l2_len; > > - uint8_t l3_len; > > + uint16_t l3_len; > > uint8_t l4_len; > > Please reorder these to avoid creating a structure hole.
Done in v2, I use the bit field like rte_mbuf, and keep the member's original order.