[dpdk-dev] [PATCH] bnx2x: tx_start_bd->vlan_or_ethertype is le16
Thomas Monjalon
thomas.monjalon at 6wind.com
Wed Dec 2 02:04:14 CET 2015
2015-12-01 18:58, Charles Williams:
> On Wed, 2015-12-02 at 00:34 +0100, Thomas Monjalon wrote:
> > 2015-12-01 14:37, Stephen Hemminger:
> > > Harish Patil <harish.patil at qlogic.com> wrote:
> > > > >2015-11-03 12:26, Chas Williams:
> > > > >> --- a/drivers/net/bnx2x/bnx2x.c
> > > > >> +++ b/drivers/net/bnx2x/bnx2x.c
> > > > >> - tx_start_bd->vlan_or_ethertype = eh->ether_type;
> > > > >> + tx_start_bd->vlan_or_ethertype
> > > > >> + = rte_cpu_to_le_16(rte_be_to_cpu_16(eh->ether_type));
> > > >
> > > > Minor question - any specific reason to use rte_be_to_cpu_16() on
> > > > ether_type alone before converting from native order to le16?
> > >
> > > ether_type is in network byte order (big endian)
> > > and hardware wants little endian. On x86 the second step is a nop.
> >
> > Doesn't it deserve a macro rte_ntole16()?
> > It may be in lib/librte_eal/common/include/generic/rte_byteorder.h.
>
> I looked I didn't see anything. This value, according to the linux
> driver, wants to be little endian regardless of the host endian.
Yes, that's why I suggest to create some macros to do this kind of conversion.
Example: rte_ntole16 means "network to little endian 16-bit".
Do you think it would be clearer to use?
More information about the dev
mailing list