Hi Bruce, On 09/03/2014 05:49 PM, Bruce Richardson wrote: > Replace a reserved slot with the new packet type field used to identify > the type of the packet, i.e. what protocols are used. > > Signed-off-by: Bruce Richardson <bruce.richardson at intel.com> > --- > lib/librte_mbuf/rte_mbuf.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_mbuf/rte_mbuf.h b/lib/librte_mbuf/rte_mbuf.h > index f136d37..8d0c6fb 100644 > --- a/lib/librte_mbuf/rte_mbuf.h > +++ b/lib/librte_mbuf/rte_mbuf.h > @@ -146,7 +146,7 @@ struct rte_mbuf { > uint32_t reserved1; /**< Unused field. Required for padding */ > > /* remaining bytes are set on RX when pulling packet from descriptor */ > - uint16_t reserved2; /**< Unused field. Required for padding */ > + uint16_t packet_type; /**< Type of packet, e.g. protocols used */ > uint16_t data_len; /**< Amount of data in segment buffer. */ > uint32_t pkt_len; /**< Total pkt len: sum of all segments. */ > uint16_t l3_len:9; /**< L3 (IP) Header Length. */ >
This patch adds a new fields that nobody uses. So why should we add it ?