2015-07-13 06:42, Neil Horman: > On Mon, Jul 13, 2015 at 11:26:25AM +0100, John McNamara wrote: > > Fix for ABI breakage introduced in LRO addition. Moves > > lro bitfield to the end of the struct/member. > > > > Fixes: 8eecb3295aed (ixgbe: add LRO support) > > > > Signed-off-by: John McNamara <john.mcnamara at intel.com> > > --- > > lib/librte_ether/rte_ethdev.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/lib/librte_ether/rte_ethdev.h b/lib/librte_ether/rte_ethdev.h > > index 79bde89..1c3ace1 100644 > > --- a/lib/librte_ether/rte_ethdev.h > > +++ b/lib/librte_ether/rte_ethdev.h > > @@ -1578,9 +1578,9 @@ struct rte_eth_dev_data { > > uint8_t port_id; /**< Device [external] port identifier. */ > > uint8_t promiscuous : 1, /**< RX promiscuous mode ON(1) / OFF(0). */ > > scattered_rx : 1, /**< RX of scattered packets is ON(1) / > > OFF(0) */ > > - lro : 1, /**< RX LRO is ON(1) / OFF(0) */ > > all_multicast : 1, /**< RX all multicast mode ON(1) / OFF(0). */ > > - dev_started : 1; /**< Device state: STARTED(1) / STOPPED(0). > > */ > > + dev_started : 1, /**< Device state: STARTED(1) / STOPPED(0). > > */ > > + lro : 1; /**< RX LRO is ON(1) / OFF(0) */ > > }; > > > > /** > I presume the ABI checker stopped complaining about this with the patch, yes? > > Also, it would be great if someone could check this on ppc or a ppc cross > compile, as I recall bitfields follow endianess order.
+ Chao, IBM POWER maintainer.