DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] Aligning net/ethernet.h and rte_ether.h
@ 2015-12-02 19:45 Stephen Hemminger
  2015-12-02 21:57 ` Thomas Monjalon
  0 siblings, 1 reply; 2+ messages in thread
From: Stephen Hemminger @ 2015-12-02 19:45 UTC (permalink / raw)
  To: dev

The two header files net/ethenet.h and rte_ether.h are source
incompatiable right now. They both define a bunch of constants
and struct ether_addr; the effective values are the same but
the structure element name is different.

/usr/include/net/ether.h
/* This is a name for the 48 bit ethernet address available on many
   systems.  */
struct ether_addr
{
  u_int8_t ether_addr_octet[ETH_ALEN];
} __attribute__ ((__packed__));

lib/librte_ether/rte_ether.h
struct ether_addr {
	uint8_t addr_bytes[ETHER_ADDR_LEN]; /**< Address bytes in transmission order */
} __attribute__((__packed__));


I would like to just have rte_ether.h include netinet/ether.h
to get rid of the useless duplication, and fix all the code in DPDK.
But this will break out-of-tree source compatibility so best to
wait for DPDK 2.3. Is there a good place to put this in 2.2 release notes?

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] Aligning net/ethernet.h and rte_ether.h
  2015-12-02 19:45 [dpdk-dev] Aligning net/ethernet.h and rte_ether.h Stephen Hemminger
@ 2015-12-02 21:57 ` Thomas Monjalon
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Monjalon @ 2015-12-02 21:57 UTC (permalink / raw)
  To: Stephen Hemminger; +Cc: dev

2015-12-02 11:45, Stephen Hemminger:
> I would like to just have rte_ether.h include netinet/ether.h
> to get rid of the useless duplication, and fix all the code in DPDK.
> But this will break out-of-tree source compatibility so best to
> wait for DPDK 2.3. Is there a good place to put this in 2.2 release notes?

deprecation.rst?

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-12-02 21:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-12-02 19:45 [dpdk-dev] Aligning net/ethernet.h and rte_ether.h Stephen Hemminger
2015-12-02 21:57 ` Thomas Monjalon

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).