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.