| Message ID | 1453906781-9424-1-git-send-email-christophe.milard@linaro.org |
|---|---|
| State | New |
| Headers | show |
maxim: do you want a v2 or are you fixing this in the merge? On 27 January 2016 at 16:14, Savolainen, Petri (Nokia - FI/Espoo) < petri.savolainen@nokia.com> wrote: > For the git log readability better subject would be: > > "api: endian: rename endian types with odp_ prefix" > > > Maybe Maxim can edit that during merge. > > > > Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com> > > > > > -----Original Message----- > > From: EXT Christophe Milard [mailto:christophe.milard@linaro.org] > > Sent: Wednesday, January 27, 2016 5:00 PM > > To: anders.roxell@linaro.org; mike.holmes@linaro.org; > > bill.fischofer@linaro.org; petri.savolainen@linaro.org > > Cc: lng-odp@lists.linaro.org; Christophe Milard > > Subject: [API-NEXT PATCH] api: prefixing some standard types. > > > > The following types: uint16le_t, uint16be_t, uint32le_t, uint32be_t, > > uint64le_t, uint64be_t, uint16sum_t, uint32sum_t defined in the api and > > in platform/linux-generic/include/odp/plat/byteorder_types.h were not > > odp_* prefixed and could create name clash with an applications. > > This patch prefixes those with odp_*, suppressing "int" for shortness. > > e.g. uint16le_t becomes odp_u16le_t > > Also modifies files using these types, of course. > > > > Signed-off-by: Christophe Milard <christophe.milard@linaro.org> > > --- > > example/classifier/odp_classifier.c | 2 +- > > example/ipsec/odp_ipsec_cache.h | 2 +- > > example/ipsec/odp_ipsec_stream.c | 2 +- > > example/packet/odp_pktio.c | 2 +- > > helper/include/odp/helper/chksum.h | 4 +- > > helper/include/odp/helper/eth.h | 6 +- > > helper/include/odp/helper/icmp.h | 12 ++-- > > helper/include/odp/helper/ip.h | 22 ++++---- > > helper/include/odp/helper/ipsec.h | 10 ++-- > > helper/include/odp/helper/tcp.h | 64 > +++++++++++------ > > ----- > > helper/include/odp/helper/udp.h | 8 +-- > > include/odp/api/byteorder.h | 40 +++++++------- > > platform/linux-generic/include/odp/byteorder.h | 48 ++++++++-------- > > .../include/odp/plat/byteorder_types.h | 16 +++--- > > test/performance/odp_pktio_perf.c | 2 +- > > .../classification/odp_classification_common.c | 6 +- > > test/validation/pktio/pktio.c | 6 +- > > 17 files changed, 126 insertions(+), 126 deletions(-) > > > > diff --git a/example/classifier/odp_classifier.c > > b/example/classifier/odp_classifier.c > > index 5a8cd37..9e5bec9 100644 > > --- a/example/classifier/odp_classifier.c > > +++ b/example/classifier/odp_classifier.c > > @@ -667,7 +667,7 @@ static void swap_pkt_addrs(odp_packet_t pkt_tbl[], > > unsigned len) > > odph_ethhdr_t *eth; > > odph_ethaddr_t tmp_addr; > > odph_ipv4hdr_t *ip; > > - uint32be_t ip_tmp_addr; /* tmp ip addr */ > > + odp_u32be_t ip_tmp_addr; /* tmp ip addr */ > > unsigned i; > > > > for (i = 0; i < len; ++i) { > > diff --git a/example/ipsec/odp_ipsec_cache.h > > b/example/ipsec/odp_ipsec_cache.h > > index 91d9d7e..56be9d8 100644 > > --- a/example/ipsec/odp_ipsec_cache.h > > +++ b/example/ipsec/odp_ipsec_cache.h > > @@ -57,7 +57,7 @@ typedef struct ipsec_cache_entry_s { > > uint32_t esp_seq; /**< ESP TX sequence number > */ > > uint32_t ah_seq; /**< AH TX sequence number > */ > > uint8_t iv[MAX_IV_LEN]; /**< ESP IV storage */ > > - uint16be_t tun_hdr_id; /**< Tunnel header IP ID */ > > + odp_u16be_t tun_hdr_id; /**< Tunnel header IP ID */ > > } state; > > } ipsec_cache_entry_t; > > > > diff --git a/example/ipsec/odp_ipsec_stream.c > > b/example/ipsec/odp_ipsec_stream.c > > index 9c2722e..ff2ca33 100644 > > --- a/example/ipsec/odp_ipsec_stream.c > > +++ b/example/ipsec/odp_ipsec_stream.c > > @@ -36,7 +36,7 @@ > > * Stream packet header > > */ > > typedef struct ODP_PACKED stream_pkt_hdr_s { > > - uint64be_t magic; /**< Stream magic value for verification */ > > + odp_u64be_t magic; /**< Stream magic value for verification */ > > uint8_t data[0]; /**< Incrementing data stream */ > > } stream_pkt_hdr_t; > > > > diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c > > index 75e92ac..13f044f 100644 > > --- a/example/packet/odp_pktio.c > > +++ b/example/packet/odp_pktio.c > > @@ -502,7 +502,7 @@ static void swap_pkt_addrs(odp_packet_t pkt_tbl[], > > unsigned len) > > odph_ethhdr_t *eth; > > odph_ethaddr_t tmp_addr; > > odph_ipv4hdr_t *ip; > > - uint32be_t ip_tmp_addr; /* tmp ip addr */ > > + odp_u32be_t ip_tmp_addr; /* tmp ip addr */ > > unsigned i; > > > > for (i = 0; i < len; ++i) { > > diff --git a/helper/include/odp/helper/chksum.h > > b/helper/include/odp/helper/chksum.h > > index edb1c98..215917c 100644 > > --- a/helper/include/odp/helper/chksum.h > > +++ b/helper/include/odp/helper/chksum.h > > @@ -27,7 +27,7 @@ extern "C" { > > * > > * @return checksum value in host cpu order > > */ > > -static inline uint16sum_t odp_chksum(void *buffer, int len) > > +static inline odp_u16sum_t odp_chksum(void *buffer, int len) > > { > > uint16_t *buf = buffer; > > uint32_t sum = 0; > > @@ -43,7 +43,7 @@ static inline uint16sum_t odp_chksum(void *buffer, int > > len) > > sum += (sum >> 16); > > result = ~sum; > > > > - return (__odp_force uint16sum_t) result; > > + return (__odp_force odp_u16sum_t) result; > > } > > > > #ifdef __cplusplus > > diff --git a/helper/include/odp/helper/eth.h > > b/helper/include/odp/helper/eth.h > > index f1c164d..7c9c728 100644 > > --- a/helper/include/odp/helper/eth.h > > +++ b/helper/include/odp/helper/eth.h > > @@ -51,7 +51,7 @@ _ODP_STATIC_ASSERT(sizeof(odph_ethaddr_t) == > > ODPH_ETHADDR_LEN, "ODPH_ETHADDR_T__ > > typedef struct ODP_PACKED { > > odph_ethaddr_t dst; /**< Destination address */ > > odph_ethaddr_t src; /**< Source address */ > > - uint16be_t type; /**< Type */ > > + odp_u16be_t type; /**< Type */ > > } odph_ethhdr_t; > > > > /** @internal Compile time assert */ > > @@ -63,8 +63,8 @@ _ODP_STATIC_ASSERT(sizeof(odph_ethhdr_t) == > > ODPH_ETHHDR_LEN, "ODPH_ETHHDR_T__SIZ > > * @todo Check usage of tpid vs ethertype. Check outer VLAN TPID. > > */ > > typedef struct ODP_PACKED { > > - uint16be_t tpid; /**< Tag protocol ID (located after ethhdr.src) > > */ > > - uint16be_t tci; /**< Priority / CFI / VLAN ID */ > > + odp_u16be_t tpid; /**< Tag protocol ID (located after ethhdr.src) > > */ > > + odp_u16be_t tci; /**< Priority / CFI / VLAN ID */ > > } odph_vlanhdr_t; > > > > /** @internal Compile time assert */ > > diff --git a/helper/include/odp/helper/icmp.h > > b/helper/include/odp/helper/icmp.h > > index abcf818..7f5097b 100644 > > --- a/helper/include/odp/helper/icmp.h > > +++ b/helper/include/odp/helper/icmp.h > > @@ -33,16 +33,16 @@ extern "C" { > > typedef struct ODP_PACKED { > > uint8_t type; /**< message type */ > > uint8_t code; /**< type sub-code */ > > - uint16sum_t chksum; /**< checksum of icmp header */ > > + odp_u16sum_t chksum; /**< checksum of icmp header */ > > union { > > struct { > > - uint16be_t id; > > - uint16be_t sequence; > > + odp_u16be_t id; > > + odp_u16be_t sequence; > > } echo; /**< echo datagram */ > > - uint32be_t gateway; /**< gateway address */ > > + odp_u32be_t gateway; /**< gateway address */ > > struct { > > - uint16be_t __unused; > > - uint16be_t mtu; > > + odp_u16be_t __unused; > > + odp_u16be_t mtu; > > } frag; /**< path mtu discovery */ > > } un; /**< icmp sub header */ > > } odph_icmphdr_t; > > diff --git a/helper/include/odp/helper/ip.h > > b/helper/include/odp/helper/ip.h > > index 41408a3..2fa4aae 100644 > > --- a/helper/include/odp/helper/ip.h > > +++ b/helper/include/odp/helper/ip.h > > @@ -61,14 +61,14 @@ extern "C" { > > typedef struct ODP_PACKED { > > uint8_t ver_ihl; /**< Version / Header length */ > > uint8_t tos; /**< Type of service */ > > - uint16be_t tot_len; /**< Total length */ > > - uint16be_t id; /**< ID */ > > - uint16be_t frag_offset; /**< Fragmentation offset */ > > + odp_u16be_t tot_len; /**< Total length */ > > + odp_u16be_t id; /**< ID */ > > + odp_u16be_t frag_offset;/**< Fragmentation offset */ > > uint8_t ttl; /**< Time to live */ > > uint8_t proto; /**< Protocol */ > > - uint16sum_t chksum; /**< Checksum */ > > - uint32be_t src_addr; /**< Source address */ > > - uint32be_t dst_addr; /**< Destination address */ > > + odp_u16sum_t chksum; /**< Checksum */ > > + odp_u32be_t src_addr; /**< Source address */ > > + odp_u32be_t dst_addr; /**< Destination address */ > > } odph_ipv4hdr_t; > > > > /** @internal Compile time assert */ > > @@ -83,11 +83,11 @@ _ODP_STATIC_ASSERT(sizeof(odph_ipv4hdr_t) == > > ODPH_IPV4HDR_LEN, "ODPH_IPV4HDR_T__ > > */ > > static inline int odph_ipv4_csum_valid(odp_packet_t pkt) > > { > > - uint16be_t res = 0; > > + odp_u16be_t res = 0; > > uint16_t *w; > > int nleft = sizeof(odph_ipv4hdr_t); > > odph_ipv4hdr_t ip; > > - uint16be_t chksum; > > + odp_u16be_t chksum; > > > > if (!odp_packet_l3_offset(pkt)) > > return 0; > > @@ -113,7 +113,7 @@ static inline int odph_ipv4_csum_valid(odp_packet_t > > pkt) > > * > > * @return IPv4 checksum in host cpu order, or 0 on failure > > */ > > -static inline uint16sum_t odph_ipv4_csum_update(odp_packet_t pkt) > > +static inline odp_u16sum_t odph_ipv4_csum_update(odp_packet_t pkt) > > { > > uint16_t *w; > > odph_ipv4hdr_t *ip; > > @@ -138,8 +138,8 @@ static inline uint16sum_t > > odph_ipv4_csum_update(odp_packet_t pkt) > > * IPv6 header > > */ > > typedef struct ODP_PACKED { > > - uint32be_t ver_tc_flow; /**< Version / Traffic class / Flow label > > */ > > - uint16be_t payload_len; /**< Payload length */ > > + odp_u32be_t ver_tc_flow; /**< Version / Traffic class / Flow label > > */ > > + odp_u16be_t payload_len; /**< Payload length */ > > uint8_t next_hdr; /**< Next header */ > > uint8_t hop_limit; /**< Hop limit */ > > uint8_t src_addr[16]; /**< Source address */ > > diff --git a/helper/include/odp/helper/ipsec.h > > b/helper/include/odp/helper/ipsec.h > > index 2565f74..2cc2403 100644 > > --- a/helper/include/odp/helper/ipsec.h > > +++ b/helper/include/odp/helper/ipsec.h > > @@ -35,8 +35,8 @@ extern "C" { > > * IPSec ESP header > > */ > > typedef struct ODP_PACKED { > > - uint32be_t spi; /**< Security Parameter Index */ > > - uint32be_t seq_no; /**< Sequence Number */ > > + odp_u32be_t spi; /**< Security Parameter Index */ > > + odp_u32be_t seq_no; /**< Sequence Number */ > > uint8_t iv[0]; /**< Initialization vector */ > > } odph_esphdr_t; > > > > @@ -61,9 +61,9 @@ _ODP_STATIC_ASSERT(sizeof(odph_esptrl_t) == > > ODPH_ESPTRL_LEN, "ODPH_ESPTRL_T__SIZ > > typedef struct ODP_PACKED { > > uint8_t next_header; /**< Next header protocol */ > > uint8_t ah_len; /**< AH header length */ > > - uint16be_t pad; /**< Padding (must be 0) */ > > - uint32be_t spi; /**< Security Parameter Index */ > > - uint32be_t seq_no; /**< Sequence Number */ > > + odp_u16be_t pad; /**< Padding (must be 0) */ > > + odp_u32be_t spi; /**< Security Parameter Index */ > > + odp_u32be_t seq_no; /**< Sequence Number */ > > uint8_t icv[0]; /**< Integrity Check Value */ > > } odph_ahhdr_t; > > > > diff --git a/helper/include/odp/helper/tcp.h > > b/helper/include/odp/helper/tcp.h > > index 42f0cbe..eb0a268 100644 > > --- a/helper/include/odp/helper/tcp.h > > +++ b/helper/include/odp/helper/tcp.h > > @@ -30,54 +30,54 @@ extern "C" { > > > > /** TCP header */ > > typedef struct ODP_PACKED { > > - uint16be_t src_port; /**< Source port */ > > - uint16be_t dst_port; /**< Destination port */ > > - uint32be_t seq_no; /**< Sequence number */ > > - uint32be_t ack_no; /**< Acknowledgment number */ > > + odp_u16be_t src_port; /**< Source port */ > > + odp_u16be_t dst_port; /**< Destination port */ > > + odp_u32be_t seq_no; /**< Sequence number */ > > + odp_u32be_t ack_no; /**< Acknowledgment number */ > > union { > > - uint16be_t doffset_flags; > > + odp_u16be_t doffset_flags; > > #if defined(ODP_BIG_ENDIAN_BITFIELD) > > struct { > > - uint16be_t rsvd1:8; > > - uint16be_t flags:8; /**< TCP flags as a byte */ > > + odp_u16be_t rsvd1:8; > > + odp_u16be_t flags:8; /**< TCP flags as a byte */ > > }; > > struct { > > - uint16be_t hl:4; /**< Hdr len, in words */ > > - uint16be_t rsvd3:4; /**< Reserved */ > > - uint16be_t cwr:1; > > - uint16be_t ece:1; > > - uint16be_t urg:1; > > - uint16be_t ack:1; > > - uint16be_t psh:1; > > - uint16be_t rst:1; > > - uint16be_t syn:1; > > - uint16be_t fin:1; > > + odp_u16be_t hl:4; /**< Hdr len, in words */ > > + odp_u16be_t rsvd3:4; /**< Reserved */ > > + odp_u16be_t cwr:1; > > + odp_u16be_t ece:1; > > + odp_u16be_t urg:1; > > + odp_u16be_t ack:1; > > + odp_u16be_t psh:1; > > + odp_u16be_t rst:1; > > + odp_u16be_t syn:1; > > + odp_u16be_t fin:1; > > }; > > #elif defined(ODP_LITTLE_ENDIAN_BITFIELD) > > struct { > > - uint16be_t flags:8; > > - uint16be_t rsvd1:8; /**< TCP flags as a byte */ > > + odp_u16be_t flags:8; > > + odp_u16be_t rsvd1:8; /**< TCP flags as a byte */ > > }; > > struct { > > - uint16be_t rsvd3:4; /**< Reserved */ > > - uint16be_t hl:4; /**< Hdr len, in words */ > > - uint16be_t fin:1; > > - uint16be_t syn:1; > > - uint16be_t rst:1; > > - uint16be_t psh:1; > > - uint16be_t ack:1; > > - uint16be_t urg:1; > > - uint16be_t ece:1; > > - uint16be_t cwr:1; > > + odp_u16be_t rsvd3:4; /**< Reserved */ > > + odp_u16be_t hl:4; /**< Hdr len, in words */ > > + odp_u16be_t fin:1; > > + odp_u16be_t syn:1; > > + odp_u16be_t rst:1; > > + odp_u16be_t psh:1; > > + odp_u16be_t ack:1; > > + odp_u16be_t urg:1; > > + odp_u16be_t ece:1; > > + odp_u16be_t cwr:1; > > }; > > > > #else > > #error "Endian BitField order not defined!" > > #endif > > }; > > - uint16be_t window; /**< Window size */ > > - uint16be_t cksm; /**< Checksum */ > > - uint16be_t urgptr; /**< Urgent pointer */ > > + odp_u16be_t window; /**< Window size */ > > + odp_u16be_t cksm; /**< Checksum */ > > + odp_u16be_t urgptr; /**< Urgent pointer */ > > } odph_tcphdr_t; > > > > /** > > diff --git a/helper/include/odp/helper/udp.h > > b/helper/include/odp/helper/udp.h > > index 93b342d..88a77f8 100644 > > --- a/helper/include/odp/helper/udp.h > > +++ b/helper/include/odp/helper/udp.h > > @@ -30,10 +30,10 @@ extern "C" { > > > > /** UDP header */ > > typedef struct ODP_PACKED { > > - uint16be_t src_port; /**< Source port */ > > - uint16be_t dst_port; /**< Destination port */ > > - uint16be_t length; /**< UDP datagram length in bytes > (header+data) > > */ > > - uint16be_t chksum; /**< UDP header and data checksum (0 if not > > used)*/ > > + odp_u16be_t src_port; /**< Source port */ > > + odp_u16be_t dst_port; /**< Destination port */ > > + odp_u16be_t length; /**< UDP datagram length in bytes > > (header+data) */ > > + odp_u16be_t chksum; /**< UDP header and data checksum (0 if not > > used)*/ > > } odph_udphdr_t; > > > > /** > > diff --git a/include/odp/api/byteorder.h b/include/odp/api/byteorder.h > > index a7b3647..a12a729 100644 > > --- a/include/odp/api/byteorder.h > > +++ b/include/odp/api/byteorder.h > > @@ -41,28 +41,28 @@ extern "C" { > > */ > > > > /** > > - * @typedef uint16le_t > > + * @typedef odp_u16le_t > > * unsigned 16bit little endian > > * > > - * @typedef uint16be_t > > + * @typedef odp_u16be_t > > * unsigned 16bit big endian > > * > > - * @typedef uint32le_t > > + * @typedef odp_u32le_t > > * unsigned 32bit little endian > > * > > - * @typedef uint32be_t > > + * @typedef odp_u32be_t > > * unsigned 32bit big endian > > * > > - * @typedef uint64le_t > > + * @typedef odp_u64le_t > > * unsigned 64bit little endian > > * > > - * @typedef uint64be_t > > + * @typedef odp_u64be_t > > * unsigned 64bit big endian > > * > > - * @typedef uint16sum_t > > + * @typedef odp_u16sum_t > > * unsigned 16bit bitwise > > * > > - * @typedef uint32sum_t > > + * @typedef odp_u32sum_t > > * unsigned 32bit bitwise > > */ > > > > @@ -75,21 +75,21 @@ extern "C" { > > * @param be16 big endian 16bit > > * @return cpu native uint16_t > > */ > > -uint16_t odp_be_to_cpu_16(uint16be_t be16); > > +uint16_t odp_be_to_cpu_16(odp_u16be_t be16); > > > > /** > > * Convert 32bit big endian to cpu native uint32_t > > * @param be32 big endian 32bit > > * @return cpu native uint32_t > > */ > > -uint32_t odp_be_to_cpu_32(uint32be_t be32); > > +uint32_t odp_be_to_cpu_32(odp_u32be_t be32); > > > > /** > > * Convert 64bit big endian to cpu native uint64_t > > * @param be64 big endian 64bit > > * @return cpu native uint64_t > > */ > > -uint64_t odp_be_to_cpu_64(uint64be_t be64); > > +uint64_t odp_be_to_cpu_64(odp_u64be_t be64); > > > > > > /* > > @@ -101,21 +101,21 @@ uint64_t odp_be_to_cpu_64(uint64be_t be64); > > * @param cpu16 uint16_t in cpu native format > > * @return big endian 16bit > > */ > > -uint16be_t odp_cpu_to_be_16(uint16_t cpu16); > > +odp_u16be_t odp_cpu_to_be_16(uint16_t cpu16); > > > > /** > > * Convert cpu native uint32_t to 32bit big endian > > * @param cpu32 uint32_t in cpu native format > > * @return big endian 32bit > > */ > > -uint32be_t odp_cpu_to_be_32(uint32_t cpu32); > > +odp_u32be_t odp_cpu_to_be_32(uint32_t cpu32); > > > > /** > > * Convert cpu native uint64_t to 64bit big endian > > * @param cpu64 uint64_t in cpu native format > > * @return big endian 64bit > > */ > > -uint64be_t odp_cpu_to_be_64(uint64_t cpu64); > > +odp_u64be_t odp_cpu_to_be_64(uint64_t cpu64); > > > > > > /* > > @@ -127,21 +127,21 @@ uint64be_t odp_cpu_to_be_64(uint64_t cpu64); > > * @param le16 little endian 16bit > > * @return cpu native uint16_t > > */ > > -uint16_t odp_le_to_cpu_16(uint16le_t le16); > > +uint16_t odp_le_to_cpu_16(odp_u16le_t le16); > > > > /** > > * Convert 32bit little endian to cpu native uint32_t > > * @param le32 little endian 32bit > > * @return cpu native uint32_t > > */ > > -uint32_t odp_le_to_cpu_32(uint32le_t le32); > > +uint32_t odp_le_to_cpu_32(odp_u32le_t le32); > > > > /** > > * Convert 64bit little endian to cpu native uint64_t > > * @param le64 little endian 64bit > > * @return cpu native uint64_t > > */ > > -uint64_t odp_le_to_cpu_64(uint64le_t le64); > > +uint64_t odp_le_to_cpu_64(odp_u64le_t le64); > > > > > > /* > > @@ -153,21 +153,21 @@ uint64_t odp_le_to_cpu_64(uint64le_t le64); > > * @param cpu16 uint16_t in cpu native format > > * @return little endian 16bit > > */ > > -uint16le_t odp_cpu_to_le_16(uint16_t cpu16); > > +odp_u16le_t odp_cpu_to_le_16(uint16_t cpu16); > > > > /** > > * Convert cpu native uint32_t to 32bit little endian > > * @param cpu32 uint32_t in cpu native format > > * @return little endian 32bit > > */ > > -uint32le_t odp_cpu_to_le_32(uint32_t cpu32); > > +odp_u32le_t odp_cpu_to_le_32(uint32_t cpu32); > > > > /** > > * Convert cpu native uint64_t to 64bit little endian > > * @param cpu64 uint64_t in cpu native format > > * @return little endian 64bit > > */ > > -uint64le_t odp_cpu_to_le_64(uint64_t cpu64); > > +odp_u64le_t odp_cpu_to_le_64(uint64_t cpu64); > > > > /** > > * @} > > diff --git a/platform/linux-generic/include/odp/byteorder.h > > b/platform/linux-generic/include/odp/byteorder.h > > index 173b047..ee60d02 100644 > > --- a/platform/linux-generic/include/odp/byteorder.h > > +++ b/platform/linux-generic/include/odp/byteorder.h > > @@ -27,7 +27,7 @@ extern "C" { > > * @{ > > */ > > > > -static inline uint16_t odp_be_to_cpu_16(uint16be_t be16) > > +static inline uint16_t odp_be_to_cpu_16(odp_u16be_t be16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return __odp_builtin_bswap16((__odp_force uint16_t)be16); > > @@ -36,7 +36,7 @@ static inline uint16_t odp_be_to_cpu_16(uint16be_t > be16) > > #endif > > } > > > > -static inline uint32_t odp_be_to_cpu_32(uint32be_t be32) > > +static inline uint32_t odp_be_to_cpu_32(odp_u32be_t be32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return __builtin_bswap32((__odp_force uint32_t)be32); > > @@ -45,7 +45,7 @@ static inline uint32_t odp_be_to_cpu_32(uint32be_t > be32) > > #endif > > } > > > > -static inline uint64_t odp_be_to_cpu_64(uint64be_t be64) > > +static inline uint64_t odp_be_to_cpu_64(odp_u64be_t be64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return __builtin_bswap64((__odp_force uint64_t)be64); > > @@ -55,35 +55,35 @@ static inline uint64_t odp_be_to_cpu_64(uint64be_t > > be64) > > } > > > > > > -static inline uint16be_t odp_cpu_to_be_16(uint16_t cpu16) > > +static inline odp_u16be_t odp_cpu_to_be_16(uint16_t cpu16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint16be_t)__odp_builtin_bswap16(cpu16); > > + return (__odp_force odp_u16be_t)__odp_builtin_bswap16(cpu16); > > #else > > - return (__odp_force uint16be_t)cpu16; > > + return (__odp_force odp_u16be_t)cpu16; > > #endif > > } > > > > -static inline uint32be_t odp_cpu_to_be_32(uint32_t cpu32) > > +static inline odp_u32be_t odp_cpu_to_be_32(uint32_t cpu32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint32be_t)__builtin_bswap32(cpu32); > > + return (__odp_force odp_u32be_t)__builtin_bswap32(cpu32); > > #else > > - return (__odp_force uint32be_t)cpu32; > > + return (__odp_force odp_u32be_t)cpu32; > > #endif > > } > > > > -static inline uint64be_t odp_cpu_to_be_64(uint64_t cpu64) > > +static inline odp_u64be_t odp_cpu_to_be_64(uint64_t cpu64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint64be_t)__builtin_bswap64(cpu64); > > + return (__odp_force odp_u64be_t)__builtin_bswap64(cpu64); > > #else > > - return (__odp_force uint64be_t)cpu64; > > + return (__odp_force odp_u64be_t)cpu64; > > #endif > > } > > > > > > -static inline uint16_t odp_le_to_cpu_16(uint16le_t le16) > > +static inline uint16_t odp_le_to_cpu_16(odp_u16le_t le16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return (__odp_force uint16_t)le16; > > @@ -92,7 +92,7 @@ static inline uint16_t odp_le_to_cpu_16(uint16le_t > le16) > > #endif > > } > > > > -static inline uint32_t odp_le_to_cpu_32(uint32le_t le32) > > +static inline uint32_t odp_le_to_cpu_32(odp_u32le_t le32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return (__odp_force uint32_t)le32; > > @@ -101,7 +101,7 @@ static inline uint32_t odp_le_to_cpu_32(uint32le_t > > le32) > > #endif > > } > > > > -static inline uint64_t odp_le_to_cpu_64(uint64le_t le64) > > +static inline uint64_t odp_le_to_cpu_64(odp_u64le_t le64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return (__odp_force uint64_t)le64; > > @@ -111,30 +111,30 @@ static inline uint64_t odp_le_to_cpu_64(uint64le_t > > le64) > > } > > > > > > -static inline uint16le_t odp_cpu_to_le_16(uint16_t cpu16) > > +static inline odp_u16le_t odp_cpu_to_le_16(uint16_t cpu16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint16le_t)cpu16; > > + return (__odp_force odp_u16le_t)cpu16; > > #else > > - return (__odp_force uint16le_t)__odp_builtin_bswap16(cpu16); > > + return (__odp_force odp_u16le_t)__odp_builtin_bswap16(cpu16); > > #endif > > } > > > > -static inline uint32le_t odp_cpu_to_le_32(uint32_t cpu32) > > +static inline odp_u32le_t odp_cpu_to_le_32(uint32_t cpu32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint32le_t)cpu32; > > + return (__odp_force odp_u32le_t)cpu32; > > #else > > - return (__odp_force uint32le_t)__builtin_bswap32(cpu32); > > + return (__odp_force odp_u32le_t)__builtin_bswap32(cpu32); > > #endif > > } > > > > -static inline uint64le_t odp_cpu_to_le_64(uint64_t cpu64) > > +static inline odp_u64le_t odp_cpu_to_le_64(uint64_t cpu64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint64le_t)cpu64; > > + return (__odp_force odp_u64le_t)cpu64; > > #else > > - return (__odp_force uint64le_t)__builtin_bswap64(cpu64); > > + return (__odp_force odp_u64le_t)__builtin_bswap64(cpu64); > > #endif > > } > > > > diff --git a/platform/linux-generic/include/odp/plat/byteorder_types.h > > b/platform/linux-generic/include/odp/plat/byteorder_types.h > > index cf917b1..0a8e409 100644 > > --- a/platform/linux-generic/include/odp/plat/byteorder_types.h > > +++ b/platform/linux-generic/include/odp/plat/byteorder_types.h > > @@ -67,17 +67,17 @@ extern "C" { > > #define ODP_BYTE_ORDER ODP_BIG_ENDIAN > > #endif > > > > -typedef uint16_t __odp_bitwise uint16le_t; > > -typedef uint16_t __odp_bitwise uint16be_t; > > +typedef uint16_t __odp_bitwise odp_u16le_t; > > +typedef uint16_t __odp_bitwise odp_u16be_t; > > > > -typedef uint32_t __odp_bitwise uint32le_t; > > -typedef uint32_t __odp_bitwise uint32be_t; > > +typedef uint32_t __odp_bitwise odp_u32le_t; > > +typedef uint32_t __odp_bitwise odp_u32be_t; > > > > -typedef uint64_t __odp_bitwise uint64le_t; > > -typedef uint64_t __odp_bitwise uint64be_t; > > +typedef uint64_t __odp_bitwise odp_u64le_t; > > +typedef uint64_t __odp_bitwise odp_u64be_t; > > > > -typedef uint16_t __odp_bitwise uint16sum_t; > > -typedef uint32_t __odp_bitwise uint32sum_t; > > +typedef uint16_t __odp_bitwise odp_u16sum_t; > > +typedef uint32_t __odp_bitwise odp_u32sum_t; > > > > /** > > * @} > > diff --git a/test/performance/odp_pktio_perf.c > > b/test/performance/odp_pktio_perf.c > > index 9fdc4cb..f85f882 100644 > > --- a/test/performance/odp_pktio_perf.c > > +++ b/test/performance/odp_pktio_perf.c > > @@ -145,7 +145,7 @@ typedef struct { > > } thread_args_t; > > > > typedef struct { > > - uint32be_t magic; /* Packet header magic number */ > > + odp_u32be_t magic; /* Packet header magic number */ > > } pkt_head_t; > > > > /* Pool from which transmitted packets are allocated */ > > diff --git a/test/validation/classification/odp_classification_common.c > > b/test/validation/classification/odp_classification_common.c > > index bb937dc..70d21a2 100644 > > --- a/test/validation/classification/odp_classification_common.c > > +++ b/test/validation/classification/odp_classification_common.c > > @@ -13,8 +13,8 @@ > > #include <odp/helper/tcp.h> > > > > typedef struct cls_test_packet { > > - uint32be_t magic; > > - uint32be_t seq; > > + odp_u32be_t magic; > > + odp_u32be_t seq; > > } cls_test_packet_t; > > > > int destroy_inq(odp_pktio_t pktio) > > @@ -254,7 +254,7 @@ odp_packet_t create_packet_len(odp_pool_t pool, bool > > vlan, > > vlan->tpid = odp_cpu_to_be_16(ODPH_ETHTYPE_VLAN); > > offset += sizeof(odph_vlanhdr_t); > > parseptr += sizeof(odph_vlanhdr_t); > > - uint16be_t *type = (uint16be_t *)(void *)parseptr; > > + odp_u16be_t *type = (odp_u16be_t *)(void *)parseptr; > > *type = odp_cpu_to_be_16(ODPH_ETHTYPE_IPV4); > > } else { > > ethhdr->type = odp_cpu_to_be_16(ODPH_ETHTYPE_IPV4); > > diff --git a/test/validation/pktio/pktio.c > b/test/validation/pktio/pktio.c > > index eda7993..6a5fb03 100644 > > --- a/test/validation/pktio/pktio.c > > +++ b/test/validation/pktio/pktio.c > > @@ -45,13 +45,13 @@ typedef struct { > > > > /** magic number and sequence at start of UDP payload */ > > typedef struct ODP_PACKED { > > - uint32be_t magic; > > - uint32be_t seq; > > + odp_u32be_t magic; > > + odp_u32be_t seq; > > } pkt_head_t; > > > > /** magic number at end of UDP payload */ > > typedef struct ODP_PACKED { > > - uint32be_t magic; > > + odp_u32be_t magic; > > } pkt_tail_t; > > > > /** Run mode */ > > -- > > 2.1.4 > >
Merged, Maxim. On 01/27/2016 18:43, Christophe Milard wrote: > maxim: do you want a v2 or are you fixing this in the merge? > > On 27 January 2016 at 16:14, Savolainen, Petri (Nokia - FI/Espoo) > <petri.savolainen@nokia.com <mailto:petri.savolainen@nokia.com>> wrote: > > For the git log readability better subject would be: > > "api: endian: rename endian types with odp_ prefix" > > > Maybe Maxim can edit that during merge. > > > > Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com > <mailto:petri.savolainen@nokia.com>> > > > > > -----Original Message----- > > From: EXT Christophe Milard [mailto:christophe.milard@linaro.org > <mailto:christophe.milard@linaro.org>] > > Sent: Wednesday, January 27, 2016 5:00 PM > > To: anders.roxell@linaro.org <mailto:anders.roxell@linaro.org>; > mike.holmes@linaro.org <mailto:mike.holmes@linaro.org>; > > bill.fischofer@linaro.org <mailto:bill.fischofer@linaro.org>; > petri.savolainen@linaro.org <mailto:petri.savolainen@linaro.org> > > Cc: lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>; > Christophe Milard > > Subject: [API-NEXT PATCH] api: prefixing some standard types. > > > > The following types: uint16le_t, uint16be_t, uint32le_t, uint32be_t, > > uint64le_t, uint64be_t, uint16sum_t, uint32sum_t defined in the > api and > > in platform/linux-generic/include/odp/plat/byteorder_types.h > were not > > odp_* prefixed and could create name clash with an applications. > > This patch prefixes those with odp_*, suppressing "int" for > shortness. > > e.g. uint16le_t becomes odp_u16le_t > > Also modifies files using these types, of course. > > > > Signed-off-by: Christophe Milard <christophe.milard@linaro.org > <mailto:christophe.milard@linaro.org>> > > --- > > example/classifier/odp_classifier.c | 2 +- > > example/ipsec/odp_ipsec_cache.h | 2 +- > > example/ipsec/odp_ipsec_stream.c | 2 +- > > example/packet/odp_pktio.c | 2 +- > > helper/include/odp/helper/chksum.h | 4 +- > > helper/include/odp/helper/eth.h | 6 +- > > helper/include/odp/helper/icmp.h | 12 ++-- > > helper/include/odp/helper/ip.h | 22 ++++---- > > helper/include/odp/helper/ipsec.h | 10 ++-- > > helper/include/odp/helper/tcp.h | 64 +++++++++++------ > > ----- > > helper/include/odp/helper/udp.h | 8 +-- > > include/odp/api/byteorder.h | 40 +++++++------- > > platform/linux-generic/include/odp/byteorder.h | 48 > ++++++++-------- > > .../include/odp/plat/byteorder_types.h | 16 +++--- > > test/performance/odp_pktio_perf.c | 2 +- > > .../classification/odp_classification_common.c | 6 +- > > test/validation/pktio/pktio.c | 6 +- > > 17 files changed, 126 insertions(+), 126 deletions(-) > > > > diff --git a/example/classifier/odp_classifier.c > > b/example/classifier/odp_classifier.c > > index 5a8cd37..9e5bec9 100644 > > --- a/example/classifier/odp_classifier.c > > +++ b/example/classifier/odp_classifier.c > > @@ -667,7 +667,7 @@ static void swap_pkt_addrs(odp_packet_t > pkt_tbl[], > > unsigned len) > > odph_ethhdr_t *eth; > > odph_ethaddr_t tmp_addr; > > odph_ipv4hdr_t *ip; > > - uint32be_t ip_tmp_addr; /* tmp ip addr */ > > + odp_u32be_t ip_tmp_addr; /* tmp ip addr */ > > unsigned i; > > > > for (i = 0; i < len; ++i) { > > diff --git a/example/ipsec/odp_ipsec_cache.h > > b/example/ipsec/odp_ipsec_cache.h > > index 91d9d7e..56be9d8 100644 > > --- a/example/ipsec/odp_ipsec_cache.h > > +++ b/example/ipsec/odp_ipsec_cache.h > > @@ -57,7 +57,7 @@ typedef struct ipsec_cache_entry_s { > > uint32_t esp_seq; /**< ESP TX sequence number */ > > uint32_t ah_seq; /**< AH TX sequence number */ > > uint8_t iv[MAX_IV_LEN]; /**< ESP IV storage */ > > - uint16be_t tun_hdr_id; /**< Tunnel header IP ID */ > > + odp_u16be_t tun_hdr_id; /**< Tunnel header IP > ID */ > > } state; > > } ipsec_cache_entry_t; > > > > diff --git a/example/ipsec/odp_ipsec_stream.c > > b/example/ipsec/odp_ipsec_stream.c > > index 9c2722e..ff2ca33 100644 > > --- a/example/ipsec/odp_ipsec_stream.c > > +++ b/example/ipsec/odp_ipsec_stream.c > > @@ -36,7 +36,7 @@ > > * Stream packet header > > */ > > typedef struct ODP_PACKED stream_pkt_hdr_s { > > - uint64be_t magic; /**< Stream magic value for > verification */ > > + odp_u64be_t magic; /**< Stream magic value for > verification */ > > uint8_t data[0]; /**< Incrementing data stream */ > > } stream_pkt_hdr_t; > > > > diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c > > index 75e92ac..13f044f 100644 > > --- a/example/packet/odp_pktio.c > > +++ b/example/packet/odp_pktio.c > > @@ -502,7 +502,7 @@ static void swap_pkt_addrs(odp_packet_t > pkt_tbl[], > > unsigned len) > > odph_ethhdr_t *eth; > > odph_ethaddr_t tmp_addr; > > odph_ipv4hdr_t *ip; > > - uint32be_t ip_tmp_addr; /* tmp ip addr */ > > + odp_u32be_t ip_tmp_addr; /* tmp ip addr */ > > unsigned i; > > > > for (i = 0; i < len; ++i) { > > diff --git a/helper/include/odp/helper/chksum.h > > b/helper/include/odp/helper/chksum.h > > index edb1c98..215917c 100644 > > --- a/helper/include/odp/helper/chksum.h > > +++ b/helper/include/odp/helper/chksum.h > > @@ -27,7 +27,7 @@ extern "C" { > > * > > * @return checksum value in host cpu order > > */ > > -static inline uint16sum_t odp_chksum(void *buffer, int len) > > +static inline odp_u16sum_t odp_chksum(void *buffer, int len) > > { > > uint16_t *buf = buffer; > > uint32_t sum = 0; > > @@ -43,7 +43,7 @@ static inline uint16sum_t odp_chksum(void > *buffer, int > > len) > > sum += (sum >> 16); > > result = ~sum; > > > > - return (__odp_force uint16sum_t) result; > > + return (__odp_force odp_u16sum_t) result; > > } > > > > #ifdef __cplusplus > > diff --git a/helper/include/odp/helper/eth.h > > b/helper/include/odp/helper/eth.h > > index f1c164d..7c9c728 100644 > > --- a/helper/include/odp/helper/eth.h > > +++ b/helper/include/odp/helper/eth.h > > @@ -51,7 +51,7 @@ _ODP_STATIC_ASSERT(sizeof(odph_ethaddr_t) == > > ODPH_ETHADDR_LEN, "ODPH_ETHADDR_T__ > > typedef struct ODP_PACKED { > > odph_ethaddr_t dst; /**< Destination address */ > > odph_ethaddr_t src; /**< Source address */ > > - uint16be_t type; /**< Type */ > > + odp_u16be_t type; /**< Type */ > > } odph_ethhdr_t; > > > > /** @internal Compile time assert */ > > @@ -63,8 +63,8 @@ _ODP_STATIC_ASSERT(sizeof(odph_ethhdr_t) == > > ODPH_ETHHDR_LEN, "ODPH_ETHHDR_T__SIZ > > * @todo Check usage of tpid vs ethertype. Check outer VLAN TPID. > > */ > > typedef struct ODP_PACKED { > > - uint16be_t tpid; /**< Tag protocol ID (located after > ethhdr.src) > > */ > > - uint16be_t tci; /**< Priority / CFI / VLAN ID */ > > + odp_u16be_t tpid; /**< Tag protocol ID (located after > ethhdr.src) > > */ > > + odp_u16be_t tci; /**< Priority / CFI / VLAN ID */ > > } odph_vlanhdr_t; > > > > /** @internal Compile time assert */ > > diff --git a/helper/include/odp/helper/icmp.h > > b/helper/include/odp/helper/icmp.h > > index abcf818..7f5097b 100644 > > --- a/helper/include/odp/helper/icmp.h > > +++ b/helper/include/odp/helper/icmp.h > > @@ -33,16 +33,16 @@ extern "C" { > > typedef struct ODP_PACKED { > > uint8_t type; /**< message type */ > > uint8_t code; /**< type sub-code */ > > - uint16sum_t chksum; /**< checksum of icmp header */ > > + odp_u16sum_t chksum; /**< checksum of icmp header */ > > union { > > struct { > > - uint16be_t id; > > - uint16be_t sequence; > > + odp_u16be_t id; > > + odp_u16be_t sequence; > > } echo; /**< echo datagram */ > > - uint32be_t gateway; /**< gateway address */ > > + odp_u32be_t gateway; /**< gateway address */ > > struct { > > - uint16be_t __unused; > > - uint16be_t mtu; > > + odp_u16be_t __unused; > > + odp_u16be_t mtu; > > } frag; /**< path mtu discovery */ > > } un; /**< icmp sub header */ > > } odph_icmphdr_t; > > diff --git a/helper/include/odp/helper/ip.h > > b/helper/include/odp/helper/ip.h > > index 41408a3..2fa4aae 100644 > > --- a/helper/include/odp/helper/ip.h > > +++ b/helper/include/odp/helper/ip.h > > @@ -61,14 +61,14 @@ extern "C" { > > typedef struct ODP_PACKED { > > uint8_t ver_ihl; /**< Version / Header length */ > > uint8_t tos; /**< Type of service */ > > - uint16be_t tot_len; /**< Total length */ > > - uint16be_t id; /**< ID */ > > - uint16be_t frag_offset; /**< Fragmentation offset */ > > + odp_u16be_t tot_len; /**< Total length */ > > + odp_u16be_t id; /**< ID */ > > + odp_u16be_t frag_offset;/**< Fragmentation offset */ > > uint8_t ttl; /**< Time to live */ > > uint8_t proto; /**< Protocol */ > > - uint16sum_t chksum; /**< Checksum */ > > - uint32be_t src_addr; /**< Source address */ > > - uint32be_t dst_addr; /**< Destination address */ > > + odp_u16sum_t chksum; /**< Checksum */ > > + odp_u32be_t src_addr; /**< Source address */ > > + odp_u32be_t dst_addr; /**< Destination address */ > > } odph_ipv4hdr_t; > > > > /** @internal Compile time assert */ > > @@ -83,11 +83,11 @@ _ODP_STATIC_ASSERT(sizeof(odph_ipv4hdr_t) == > > ODPH_IPV4HDR_LEN, "ODPH_IPV4HDR_T__ > > */ > > static inline int odph_ipv4_csum_valid(odp_packet_t pkt) > > { > > - uint16be_t res = 0; > > + odp_u16be_t res = 0; > > uint16_t *w; > > int nleft = sizeof(odph_ipv4hdr_t); > > odph_ipv4hdr_t ip; > > - uint16be_t chksum; > > + odp_u16be_t chksum; > > > > if (!odp_packet_l3_offset(pkt)) > > return 0; > > @@ -113,7 +113,7 @@ static inline int > odph_ipv4_csum_valid(odp_packet_t > > pkt) > > * > > * @return IPv4 checksum in host cpu order, or 0 on failure > > */ > > -static inline uint16sum_t odph_ipv4_csum_update(odp_packet_t pkt) > > +static inline odp_u16sum_t odph_ipv4_csum_update(odp_packet_t pkt) > > { > > uint16_t *w; > > odph_ipv4hdr_t *ip; > > @@ -138,8 +138,8 @@ static inline uint16sum_t > > odph_ipv4_csum_update(odp_packet_t pkt) > > * IPv6 header > > */ > > typedef struct ODP_PACKED { > > - uint32be_t ver_tc_flow; /**< Version / Traffic class / > Flow label > > */ > > - uint16be_t payload_len; /**< Payload length */ > > + odp_u32be_t ver_tc_flow; /**< Version / Traffic class / > Flow label > > */ > > + odp_u16be_t payload_len; /**< Payload length */ > > uint8_t next_hdr; /**< Next header */ > > uint8_t hop_limit; /**< Hop limit */ > > uint8_t src_addr[16]; /**< Source address */ > > diff --git a/helper/include/odp/helper/ipsec.h > > b/helper/include/odp/helper/ipsec.h > > index 2565f74..2cc2403 100644 > > --- a/helper/include/odp/helper/ipsec.h > > +++ b/helper/include/odp/helper/ipsec.h > > @@ -35,8 +35,8 @@ extern "C" { > > * IPSec ESP header > > */ > > typedef struct ODP_PACKED { > > - uint32be_t spi; /**< Security Parameter Index */ > > - uint32be_t seq_no; /**< Sequence Number */ > > + odp_u32be_t spi; /**< Security Parameter Index */ > > + odp_u32be_t seq_no; /**< Sequence Number */ > > uint8_t iv[0]; /**< Initialization vector */ > > } odph_esphdr_t; > > > > @@ -61,9 +61,9 @@ _ODP_STATIC_ASSERT(sizeof(odph_esptrl_t) == > > ODPH_ESPTRL_LEN, "ODPH_ESPTRL_T__SIZ > > typedef struct ODP_PACKED { > > uint8_t next_header; /**< Next header protocol */ > > uint8_t ah_len; /**< AH header length */ > > - uint16be_t pad; /**< Padding (must be 0) */ > > - uint32be_t spi; /**< Security Parameter Index */ > > - uint32be_t seq_no; /**< Sequence Number */ > > + odp_u16be_t pad; /**< Padding (must be 0) */ > > + odp_u32be_t spi; /**< Security Parameter Index */ > > + odp_u32be_t seq_no; /**< Sequence Number */ > > uint8_t icv[0]; /**< Integrity Check Value */ > > } odph_ahhdr_t; > > > > diff --git a/helper/include/odp/helper/tcp.h > > b/helper/include/odp/helper/tcp.h > > index 42f0cbe..eb0a268 100644 > > --- a/helper/include/odp/helper/tcp.h > > +++ b/helper/include/odp/helper/tcp.h > > @@ -30,54 +30,54 @@ extern "C" { > > > > /** TCP header */ > > typedef struct ODP_PACKED { > > - uint16be_t src_port; /**< Source port */ > > - uint16be_t dst_port; /**< Destination port */ > > - uint32be_t seq_no; /**< Sequence number */ > > - uint32be_t ack_no; /**< Acknowledgment number */ > > + odp_u16be_t src_port; /**< Source port */ > > + odp_u16be_t dst_port; /**< Destination port */ > > + odp_u32be_t seq_no; /**< Sequence number */ > > + odp_u32be_t ack_no; /**< Acknowledgment number */ > > union { > > - uint16be_t doffset_flags; > > + odp_u16be_t doffset_flags; > > #if defined(ODP_BIG_ENDIAN_BITFIELD) > > struct { > > - uint16be_t rsvd1:8; > > - uint16be_t flags:8; /**< TCP flags as a > byte */ > > + odp_u16be_t rsvd1:8; > > + odp_u16be_t flags:8; /**< TCP flags as a > byte */ > > }; > > struct { > > - uint16be_t hl:4; /**< Hdr len, in words */ > > - uint16be_t rsvd3:4; /**< Reserved */ > > - uint16be_t cwr:1; > > - uint16be_t ece:1; > > - uint16be_t urg:1; > > - uint16be_t ack:1; > > - uint16be_t psh:1; > > - uint16be_t rst:1; > > - uint16be_t syn:1; > > - uint16be_t fin:1; > > + odp_u16be_t hl:4; /**< Hdr len, in words */ > > + odp_u16be_t rsvd3:4; /**< Reserved */ > > + odp_u16be_t cwr:1; > > + odp_u16be_t ece:1; > > + odp_u16be_t urg:1; > > + odp_u16be_t ack:1; > > + odp_u16be_t psh:1; > > + odp_u16be_t rst:1; > > + odp_u16be_t syn:1; > > + odp_u16be_t fin:1; > > }; > > #elif defined(ODP_LITTLE_ENDIAN_BITFIELD) > > struct { > > - uint16be_t flags:8; > > - uint16be_t rsvd1:8; /**< TCP flags as a > byte */ > > + odp_u16be_t flags:8; > > + odp_u16be_t rsvd1:8; /**< TCP flags as a > byte */ > > }; > > struct { > > - uint16be_t rsvd3:4; /**< Reserved */ > > - uint16be_t hl:4; /**< Hdr len, in words */ > > - uint16be_t fin:1; > > - uint16be_t syn:1; > > - uint16be_t rst:1; > > - uint16be_t psh:1; > > - uint16be_t ack:1; > > - uint16be_t urg:1; > > - uint16be_t ece:1; > > - uint16be_t cwr:1; > > + odp_u16be_t rsvd3:4; /**< Reserved */ > > + odp_u16be_t hl:4; /**< Hdr len, in words */ > > + odp_u16be_t fin:1; > > + odp_u16be_t syn:1; > > + odp_u16be_t rst:1; > > + odp_u16be_t psh:1; > > + odp_u16be_t ack:1; > > + odp_u16be_t urg:1; > > + odp_u16be_t ece:1; > > + odp_u16be_t cwr:1; > > }; > > > > #else > > #error "Endian BitField order not defined!" > > #endif > > }; > > - uint16be_t window; /**< Window size */ > > - uint16be_t cksm; /**< Checksum */ > > - uint16be_t urgptr; /**< Urgent pointer */ > > + odp_u16be_t window; /**< Window size */ > > + odp_u16be_t cksm; /**< Checksum */ > > + odp_u16be_t urgptr; /**< Urgent pointer */ > > } odph_tcphdr_t; > > > > /** > > diff --git a/helper/include/odp/helper/udp.h > > b/helper/include/odp/helper/udp.h > > index 93b342d..88a77f8 100644 > > --- a/helper/include/odp/helper/udp.h > > +++ b/helper/include/odp/helper/udp.h > > @@ -30,10 +30,10 @@ extern "C" { > > > > /** UDP header */ > > typedef struct ODP_PACKED { > > - uint16be_t src_port; /**< Source port */ > > - uint16be_t dst_port; /**< Destination port */ > > - uint16be_t length; /**< UDP datagram length in bytes > (header+data) > > */ > > - uint16be_t chksum; /**< UDP header and data checksum (0 > if not > > used)*/ > > + odp_u16be_t src_port; /**< Source port */ > > + odp_u16be_t dst_port; /**< Destination port */ > > + odp_u16be_t length; /**< UDP datagram length in bytes > > (header+data) */ > > + odp_u16be_t chksum; /**< UDP header and data checksum (0 > if not > > used)*/ > > } odph_udphdr_t; > > > > /** > > diff --git a/include/odp/api/byteorder.h > b/include/odp/api/byteorder.h > > index a7b3647..a12a729 100644 > > --- a/include/odp/api/byteorder.h > > +++ b/include/odp/api/byteorder.h > > @@ -41,28 +41,28 @@ extern "C" { > > */ > > > > /** > > - * @typedef uint16le_t > > + * @typedef odp_u16le_t > > * unsigned 16bit little endian > > * > > - * @typedef uint16be_t > > + * @typedef odp_u16be_t > > * unsigned 16bit big endian > > * > > - * @typedef uint32le_t > > + * @typedef odp_u32le_t > > * unsigned 32bit little endian > > * > > - * @typedef uint32be_t > > + * @typedef odp_u32be_t > > * unsigned 32bit big endian > > * > > - * @typedef uint64le_t > > + * @typedef odp_u64le_t > > * unsigned 64bit little endian > > * > > - * @typedef uint64be_t > > + * @typedef odp_u64be_t > > * unsigned 64bit big endian > > * > > - * @typedef uint16sum_t > > + * @typedef odp_u16sum_t > > * unsigned 16bit bitwise > > * > > - * @typedef uint32sum_t > > + * @typedef odp_u32sum_t > > * unsigned 32bit bitwise > > */ > > > > @@ -75,21 +75,21 @@ extern "C" { > > * @param be16 big endian 16bit > > * @return cpu native uint16_t > > */ > > -uint16_t odp_be_to_cpu_16(uint16be_t be16); > > +uint16_t odp_be_to_cpu_16(odp_u16be_t be16); > > > > /** > > * Convert 32bit big endian to cpu native uint32_t > > * @param be32 big endian 32bit > > * @return cpu native uint32_t > > */ > > -uint32_t odp_be_to_cpu_32(uint32be_t be32); > > +uint32_t odp_be_to_cpu_32(odp_u32be_t be32); > > > > /** > > * Convert 64bit big endian to cpu native uint64_t > > * @param be64 big endian 64bit > > * @return cpu native uint64_t > > */ > > -uint64_t odp_be_to_cpu_64(uint64be_t be64); > > +uint64_t odp_be_to_cpu_64(odp_u64be_t be64); > > > > > > /* > > @@ -101,21 +101,21 @@ uint64_t odp_be_to_cpu_64(uint64be_t be64); > > * @param cpu16 uint16_t in cpu native format > > * @return big endian 16bit > > */ > > -uint16be_t odp_cpu_to_be_16(uint16_t cpu16); > > +odp_u16be_t odp_cpu_to_be_16(uint16_t cpu16); > > > > /** > > * Convert cpu native uint32_t to 32bit big endian > > * @param cpu32 uint32_t in cpu native format > > * @return big endian 32bit > > */ > > -uint32be_t odp_cpu_to_be_32(uint32_t cpu32); > > +odp_u32be_t odp_cpu_to_be_32(uint32_t cpu32); > > > > /** > > * Convert cpu native uint64_t to 64bit big endian > > * @param cpu64 uint64_t in cpu native format > > * @return big endian 64bit > > */ > > -uint64be_t odp_cpu_to_be_64(uint64_t cpu64); > > +odp_u64be_t odp_cpu_to_be_64(uint64_t cpu64); > > > > > > /* > > @@ -127,21 +127,21 @@ uint64be_t odp_cpu_to_be_64(uint64_t cpu64); > > * @param le16 little endian 16bit > > * @return cpu native uint16_t > > */ > > -uint16_t odp_le_to_cpu_16(uint16le_t le16); > > +uint16_t odp_le_to_cpu_16(odp_u16le_t le16); > > > > /** > > * Convert 32bit little endian to cpu native uint32_t > > * @param le32 little endian 32bit > > * @return cpu native uint32_t > > */ > > -uint32_t odp_le_to_cpu_32(uint32le_t le32); > > +uint32_t odp_le_to_cpu_32(odp_u32le_t le32); > > > > /** > > * Convert 64bit little endian to cpu native uint64_t > > * @param le64 little endian 64bit > > * @return cpu native uint64_t > > */ > > -uint64_t odp_le_to_cpu_64(uint64le_t le64); > > +uint64_t odp_le_to_cpu_64(odp_u64le_t le64); > > > > > > /* > > @@ -153,21 +153,21 @@ uint64_t odp_le_to_cpu_64(uint64le_t le64); > > * @param cpu16 uint16_t in cpu native format > > * @return little endian 16bit > > */ > > -uint16le_t odp_cpu_to_le_16(uint16_t cpu16); > > +odp_u16le_t odp_cpu_to_le_16(uint16_t cpu16); > > > > /** > > * Convert cpu native uint32_t to 32bit little endian > > * @param cpu32 uint32_t in cpu native format > > * @return little endian 32bit > > */ > > -uint32le_t odp_cpu_to_le_32(uint32_t cpu32); > > +odp_u32le_t odp_cpu_to_le_32(uint32_t cpu32); > > > > /** > > * Convert cpu native uint64_t to 64bit little endian > > * @param cpu64 uint64_t in cpu native format > > * @return little endian 64bit > > */ > > -uint64le_t odp_cpu_to_le_64(uint64_t cpu64); > > +odp_u64le_t odp_cpu_to_le_64(uint64_t cpu64); > > > > /** > > * @} > > diff --git a/platform/linux-generic/include/odp/byteorder.h > > b/platform/linux-generic/include/odp/byteorder.h > > index 173b047..ee60d02 100644 > > --- a/platform/linux-generic/include/odp/byteorder.h > > +++ b/platform/linux-generic/include/odp/byteorder.h > > @@ -27,7 +27,7 @@ extern "C" { > > * @{ > > */ > > > > -static inline uint16_t odp_be_to_cpu_16(uint16be_t be16) > > +static inline uint16_t odp_be_to_cpu_16(odp_u16be_t be16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return __odp_builtin_bswap16((__odp_force uint16_t)be16); > > @@ -36,7 +36,7 @@ static inline uint16_t > odp_be_to_cpu_16(uint16be_t be16) > > #endif > > } > > > > -static inline uint32_t odp_be_to_cpu_32(uint32be_t be32) > > +static inline uint32_t odp_be_to_cpu_32(odp_u32be_t be32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return __builtin_bswap32((__odp_force uint32_t)be32); > > @@ -45,7 +45,7 @@ static inline uint32_t > odp_be_to_cpu_32(uint32be_t be32) > > #endif > > } > > > > -static inline uint64_t odp_be_to_cpu_64(uint64be_t be64) > > +static inline uint64_t odp_be_to_cpu_64(odp_u64be_t be64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return __builtin_bswap64((__odp_force uint64_t)be64); > > @@ -55,35 +55,35 @@ static inline uint64_t > odp_be_to_cpu_64(uint64be_t > > be64) > > } > > > > > > -static inline uint16be_t odp_cpu_to_be_16(uint16_t cpu16) > > +static inline odp_u16be_t odp_cpu_to_be_16(uint16_t cpu16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint16be_t)__odp_builtin_bswap16(cpu16); > > + return (__odp_force odp_u16be_t)__odp_builtin_bswap16(cpu16); > > #else > > - return (__odp_force uint16be_t)cpu16; > > + return (__odp_force odp_u16be_t)cpu16; > > #endif > > } > > > > -static inline uint32be_t odp_cpu_to_be_32(uint32_t cpu32) > > +static inline odp_u32be_t odp_cpu_to_be_32(uint32_t cpu32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint32be_t)__builtin_bswap32(cpu32); > > + return (__odp_force odp_u32be_t)__builtin_bswap32(cpu32); > > #else > > - return (__odp_force uint32be_t)cpu32; > > + return (__odp_force odp_u32be_t)cpu32; > > #endif > > } > > > > -static inline uint64be_t odp_cpu_to_be_64(uint64_t cpu64) > > +static inline odp_u64be_t odp_cpu_to_be_64(uint64_t cpu64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint64be_t)__builtin_bswap64(cpu64); > > + return (__odp_force odp_u64be_t)__builtin_bswap64(cpu64); > > #else > > - return (__odp_force uint64be_t)cpu64; > > + return (__odp_force odp_u64be_t)cpu64; > > #endif > > } > > > > > > -static inline uint16_t odp_le_to_cpu_16(uint16le_t le16) > > +static inline uint16_t odp_le_to_cpu_16(odp_u16le_t le16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return (__odp_force uint16_t)le16; > > @@ -92,7 +92,7 @@ static inline uint16_t > odp_le_to_cpu_16(uint16le_t le16) > > #endif > > } > > > > -static inline uint32_t odp_le_to_cpu_32(uint32le_t le32) > > +static inline uint32_t odp_le_to_cpu_32(odp_u32le_t le32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return (__odp_force uint32_t)le32; > > @@ -101,7 +101,7 @@ static inline uint32_t > odp_le_to_cpu_32(uint32le_t > > le32) > > #endif > > } > > > > -static inline uint64_t odp_le_to_cpu_64(uint64le_t le64) > > +static inline uint64_t odp_le_to_cpu_64(odp_u64le_t le64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > return (__odp_force uint64_t)le64; > > @@ -111,30 +111,30 @@ static inline uint64_t > odp_le_to_cpu_64(uint64le_t > > le64) > > } > > > > > > -static inline uint16le_t odp_cpu_to_le_16(uint16_t cpu16) > > +static inline odp_u16le_t odp_cpu_to_le_16(uint16_t cpu16) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint16le_t)cpu16; > > + return (__odp_force odp_u16le_t)cpu16; > > #else > > - return (__odp_force uint16le_t)__odp_builtin_bswap16(cpu16); > > + return (__odp_force odp_u16le_t)__odp_builtin_bswap16(cpu16); > > #endif > > } > > > > -static inline uint32le_t odp_cpu_to_le_32(uint32_t cpu32) > > +static inline odp_u32le_t odp_cpu_to_le_32(uint32_t cpu32) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint32le_t)cpu32; > > + return (__odp_force odp_u32le_t)cpu32; > > #else > > - return (__odp_force uint32le_t)__builtin_bswap32(cpu32); > > + return (__odp_force odp_u32le_t)__builtin_bswap32(cpu32); > > #endif > > } > > > > -static inline uint64le_t odp_cpu_to_le_64(uint64_t cpu64) > > +static inline odp_u64le_t odp_cpu_to_le_64(uint64_t cpu64) > > { > > #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN > > - return (__odp_force uint64le_t)cpu64; > > + return (__odp_force odp_u64le_t)cpu64; > > #else > > - return (__odp_force uint64le_t)__builtin_bswap64(cpu64); > > + return (__odp_force odp_u64le_t)__builtin_bswap64(cpu64); > > #endif > > } > > > > diff --git > a/platform/linux-generic/include/odp/plat/byteorder_types.h > > b/platform/linux-generic/include/odp/plat/byteorder_types.h > > index cf917b1..0a8e409 100644 > > --- a/platform/linux-generic/include/odp/plat/byteorder_types.h > > +++ b/platform/linux-generic/include/odp/plat/byteorder_types.h > > @@ -67,17 +67,17 @@ extern "C" { > > #define ODP_BYTE_ORDER ODP_BIG_ENDIAN > > #endif > > > > -typedef uint16_t __odp_bitwise uint16le_t; > > -typedef uint16_t __odp_bitwise uint16be_t; > > +typedef uint16_t __odp_bitwise odp_u16le_t; > > +typedef uint16_t __odp_bitwise odp_u16be_t; > > > > -typedef uint32_t __odp_bitwise uint32le_t; > > -typedef uint32_t __odp_bitwise uint32be_t; > > +typedef uint32_t __odp_bitwise odp_u32le_t; > > +typedef uint32_t __odp_bitwise odp_u32be_t; > > > > -typedef uint64_t __odp_bitwise uint64le_t; > > -typedef uint64_t __odp_bitwise uint64be_t; > > +typedef uint64_t __odp_bitwise odp_u64le_t; > > +typedef uint64_t __odp_bitwise odp_u64be_t; > > > > -typedef uint16_t __odp_bitwise uint16sum_t; > > -typedef uint32_t __odp_bitwise uint32sum_t; > > +typedef uint16_t __odp_bitwise odp_u16sum_t; > > +typedef uint32_t __odp_bitwise odp_u32sum_t; > > > > /** > > * @} > > diff --git a/test/performance/odp_pktio_perf.c > > b/test/performance/odp_pktio_perf.c > > index 9fdc4cb..f85f882 100644 > > --- a/test/performance/odp_pktio_perf.c > > +++ b/test/performance/odp_pktio_perf.c > > @@ -145,7 +145,7 @@ typedef struct { > > } thread_args_t; > > > > typedef struct { > > - uint32be_t magic; /* Packet header magic number */ > > + odp_u32be_t magic; /* Packet header magic number */ > > } pkt_head_t; > > > > /* Pool from which transmitted packets are allocated */ > > diff --git > a/test/validation/classification/odp_classification_common.c > > b/test/validation/classification/odp_classification_common.c > > index bb937dc..70d21a2 100644 > > --- a/test/validation/classification/odp_classification_common.c > > +++ b/test/validation/classification/odp_classification_common.c > > @@ -13,8 +13,8 @@ > > #include <odp/helper/tcp.h> > > > > typedef struct cls_test_packet { > > - uint32be_t magic; > > - uint32be_t seq; > > + odp_u32be_t magic; > > + odp_u32be_t seq; > > } cls_test_packet_t; > > > > int destroy_inq(odp_pktio_t pktio) > > @@ -254,7 +254,7 @@ odp_packet_t create_packet_len(odp_pool_t > pool, bool > > vlan, > > vlan->tpid = odp_cpu_to_be_16(ODPH_ETHTYPE_VLAN); > > offset += sizeof(odph_vlanhdr_t); > > parseptr += sizeof(odph_vlanhdr_t); > > - uint16be_t *type = (uint16be_t *)(void *)parseptr; > > + odp_u16be_t *type = (odp_u16be_t *)(void *)parseptr; > > *type = odp_cpu_to_be_16(ODPH_ETHTYPE_IPV4); > > } else { > > ethhdr->type = odp_cpu_to_be_16(ODPH_ETHTYPE_IPV4); > > diff --git a/test/validation/pktio/pktio.c > b/test/validation/pktio/pktio.c > > index eda7993..6a5fb03 100644 > > --- a/test/validation/pktio/pktio.c > > +++ b/test/validation/pktio/pktio.c > > @@ -45,13 +45,13 @@ typedef struct { > > > > /** magic number and sequence at start of UDP payload */ > > typedef struct ODP_PACKED { > > - uint32be_t magic; > > - uint32be_t seq; > > + odp_u32be_t magic; > > + odp_u32be_t seq; > > } pkt_head_t; > > > > /** magic number at end of UDP payload */ > > typedef struct ODP_PACKED { > > - uint32be_t magic; > > + odp_u32be_t magic; > > } pkt_tail_t; > > > > /** Run mode */ > > -- > > 2.1.4 > > > > > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/lng-odp
diff --git a/example/classifier/odp_classifier.c b/example/classifier/odp_classifier.c index 5a8cd37..9e5bec9 100644 --- a/example/classifier/odp_classifier.c +++ b/example/classifier/odp_classifier.c @@ -667,7 +667,7 @@ static void swap_pkt_addrs(odp_packet_t pkt_tbl[], unsigned len) odph_ethhdr_t *eth; odph_ethaddr_t tmp_addr; odph_ipv4hdr_t *ip; - uint32be_t ip_tmp_addr; /* tmp ip addr */ + odp_u32be_t ip_tmp_addr; /* tmp ip addr */ unsigned i; for (i = 0; i < len; ++i) { diff --git a/example/ipsec/odp_ipsec_cache.h b/example/ipsec/odp_ipsec_cache.h index 91d9d7e..56be9d8 100644 --- a/example/ipsec/odp_ipsec_cache.h +++ b/example/ipsec/odp_ipsec_cache.h @@ -57,7 +57,7 @@ typedef struct ipsec_cache_entry_s { uint32_t esp_seq; /**< ESP TX sequence number */ uint32_t ah_seq; /**< AH TX sequence number */ uint8_t iv[MAX_IV_LEN]; /**< ESP IV storage */ - uint16be_t tun_hdr_id; /**< Tunnel header IP ID */ + odp_u16be_t tun_hdr_id; /**< Tunnel header IP ID */ } state; } ipsec_cache_entry_t; diff --git a/example/ipsec/odp_ipsec_stream.c b/example/ipsec/odp_ipsec_stream.c index 9c2722e..ff2ca33 100644 --- a/example/ipsec/odp_ipsec_stream.c +++ b/example/ipsec/odp_ipsec_stream.c @@ -36,7 +36,7 @@ * Stream packet header */ typedef struct ODP_PACKED stream_pkt_hdr_s { - uint64be_t magic; /**< Stream magic value for verification */ + odp_u64be_t magic; /**< Stream magic value for verification */ uint8_t data[0]; /**< Incrementing data stream */ } stream_pkt_hdr_t; diff --git a/example/packet/odp_pktio.c b/example/packet/odp_pktio.c index 75e92ac..13f044f 100644 --- a/example/packet/odp_pktio.c +++ b/example/packet/odp_pktio.c @@ -502,7 +502,7 @@ static void swap_pkt_addrs(odp_packet_t pkt_tbl[], unsigned len) odph_ethhdr_t *eth; odph_ethaddr_t tmp_addr; odph_ipv4hdr_t *ip; - uint32be_t ip_tmp_addr; /* tmp ip addr */ + odp_u32be_t ip_tmp_addr; /* tmp ip addr */ unsigned i; for (i = 0; i < len; ++i) { diff --git a/helper/include/odp/helper/chksum.h b/helper/include/odp/helper/chksum.h index edb1c98..215917c 100644 --- a/helper/include/odp/helper/chksum.h +++ b/helper/include/odp/helper/chksum.h @@ -27,7 +27,7 @@ extern "C" { * * @return checksum value in host cpu order */ -static inline uint16sum_t odp_chksum(void *buffer, int len) +static inline odp_u16sum_t odp_chksum(void *buffer, int len) { uint16_t *buf = buffer; uint32_t sum = 0; @@ -43,7 +43,7 @@ static inline uint16sum_t odp_chksum(void *buffer, int len) sum += (sum >> 16); result = ~sum; - return (__odp_force uint16sum_t) result; + return (__odp_force odp_u16sum_t) result; } #ifdef __cplusplus diff --git a/helper/include/odp/helper/eth.h b/helper/include/odp/helper/eth.h index f1c164d..7c9c728 100644 --- a/helper/include/odp/helper/eth.h +++ b/helper/include/odp/helper/eth.h @@ -51,7 +51,7 @@ _ODP_STATIC_ASSERT(sizeof(odph_ethaddr_t) == ODPH_ETHADDR_LEN, "ODPH_ETHADDR_T__ typedef struct ODP_PACKED { odph_ethaddr_t dst; /**< Destination address */ odph_ethaddr_t src; /**< Source address */ - uint16be_t type; /**< Type */ + odp_u16be_t type; /**< Type */ } odph_ethhdr_t; /** @internal Compile time assert */ @@ -63,8 +63,8 @@ _ODP_STATIC_ASSERT(sizeof(odph_ethhdr_t) == ODPH_ETHHDR_LEN, "ODPH_ETHHDR_T__SIZ * @todo Check usage of tpid vs ethertype. Check outer VLAN TPID. */ typedef struct ODP_PACKED { - uint16be_t tpid; /**< Tag protocol ID (located after ethhdr.src) */ - uint16be_t tci; /**< Priority / CFI / VLAN ID */ + odp_u16be_t tpid; /**< Tag protocol ID (located after ethhdr.src) */ + odp_u16be_t tci; /**< Priority / CFI / VLAN ID */ } odph_vlanhdr_t; /** @internal Compile time assert */ diff --git a/helper/include/odp/helper/icmp.h b/helper/include/odp/helper/icmp.h index abcf818..7f5097b 100644 --- a/helper/include/odp/helper/icmp.h +++ b/helper/include/odp/helper/icmp.h @@ -33,16 +33,16 @@ extern "C" { typedef struct ODP_PACKED { uint8_t type; /**< message type */ uint8_t code; /**< type sub-code */ - uint16sum_t chksum; /**< checksum of icmp header */ + odp_u16sum_t chksum; /**< checksum of icmp header */ union { struct { - uint16be_t id; - uint16be_t sequence; + odp_u16be_t id; + odp_u16be_t sequence; } echo; /**< echo datagram */ - uint32be_t gateway; /**< gateway address */ + odp_u32be_t gateway; /**< gateway address */ struct { - uint16be_t __unused; - uint16be_t mtu; + odp_u16be_t __unused; + odp_u16be_t mtu; } frag; /**< path mtu discovery */ } un; /**< icmp sub header */ } odph_icmphdr_t; diff --git a/helper/include/odp/helper/ip.h b/helper/include/odp/helper/ip.h index 41408a3..2fa4aae 100644 --- a/helper/include/odp/helper/ip.h +++ b/helper/include/odp/helper/ip.h @@ -61,14 +61,14 @@ extern "C" { typedef struct ODP_PACKED { uint8_t ver_ihl; /**< Version / Header length */ uint8_t tos; /**< Type of service */ - uint16be_t tot_len; /**< Total length */ - uint16be_t id; /**< ID */ - uint16be_t frag_offset; /**< Fragmentation offset */ + odp_u16be_t tot_len; /**< Total length */ + odp_u16be_t id; /**< ID */ + odp_u16be_t frag_offset;/**< Fragmentation offset */ uint8_t ttl; /**< Time to live */ uint8_t proto; /**< Protocol */ - uint16sum_t chksum; /**< Checksum */ - uint32be_t src_addr; /**< Source address */ - uint32be_t dst_addr; /**< Destination address */ + odp_u16sum_t chksum; /**< Checksum */ + odp_u32be_t src_addr; /**< Source address */ + odp_u32be_t dst_addr; /**< Destination address */ } odph_ipv4hdr_t; /** @internal Compile time assert */ @@ -83,11 +83,11 @@ _ODP_STATIC_ASSERT(sizeof(odph_ipv4hdr_t) == ODPH_IPV4HDR_LEN, "ODPH_IPV4HDR_T__ */ static inline int odph_ipv4_csum_valid(odp_packet_t pkt) { - uint16be_t res = 0; + odp_u16be_t res = 0; uint16_t *w; int nleft = sizeof(odph_ipv4hdr_t); odph_ipv4hdr_t ip; - uint16be_t chksum; + odp_u16be_t chksum; if (!odp_packet_l3_offset(pkt)) return 0; @@ -113,7 +113,7 @@ static inline int odph_ipv4_csum_valid(odp_packet_t pkt) * * @return IPv4 checksum in host cpu order, or 0 on failure */ -static inline uint16sum_t odph_ipv4_csum_update(odp_packet_t pkt) +static inline odp_u16sum_t odph_ipv4_csum_update(odp_packet_t pkt) { uint16_t *w; odph_ipv4hdr_t *ip; @@ -138,8 +138,8 @@ static inline uint16sum_t odph_ipv4_csum_update(odp_packet_t pkt) * IPv6 header */ typedef struct ODP_PACKED { - uint32be_t ver_tc_flow; /**< Version / Traffic class / Flow label */ - uint16be_t payload_len; /**< Payload length */ + odp_u32be_t ver_tc_flow; /**< Version / Traffic class / Flow label */ + odp_u16be_t payload_len; /**< Payload length */ uint8_t next_hdr; /**< Next header */ uint8_t hop_limit; /**< Hop limit */ uint8_t src_addr[16]; /**< Source address */ diff --git a/helper/include/odp/helper/ipsec.h b/helper/include/odp/helper/ipsec.h index 2565f74..2cc2403 100644 --- a/helper/include/odp/helper/ipsec.h +++ b/helper/include/odp/helper/ipsec.h @@ -35,8 +35,8 @@ extern "C" { * IPSec ESP header */ typedef struct ODP_PACKED { - uint32be_t spi; /**< Security Parameter Index */ - uint32be_t seq_no; /**< Sequence Number */ + odp_u32be_t spi; /**< Security Parameter Index */ + odp_u32be_t seq_no; /**< Sequence Number */ uint8_t iv[0]; /**< Initialization vector */ } odph_esphdr_t; @@ -61,9 +61,9 @@ _ODP_STATIC_ASSERT(sizeof(odph_esptrl_t) == ODPH_ESPTRL_LEN, "ODPH_ESPTRL_T__SIZ typedef struct ODP_PACKED { uint8_t next_header; /**< Next header protocol */ uint8_t ah_len; /**< AH header length */ - uint16be_t pad; /**< Padding (must be 0) */ - uint32be_t spi; /**< Security Parameter Index */ - uint32be_t seq_no; /**< Sequence Number */ + odp_u16be_t pad; /**< Padding (must be 0) */ + odp_u32be_t spi; /**< Security Parameter Index */ + odp_u32be_t seq_no; /**< Sequence Number */ uint8_t icv[0]; /**< Integrity Check Value */ } odph_ahhdr_t; diff --git a/helper/include/odp/helper/tcp.h b/helper/include/odp/helper/tcp.h index 42f0cbe..eb0a268 100644 --- a/helper/include/odp/helper/tcp.h +++ b/helper/include/odp/helper/tcp.h @@ -30,54 +30,54 @@ extern "C" { /** TCP header */ typedef struct ODP_PACKED { - uint16be_t src_port; /**< Source port */ - uint16be_t dst_port; /**< Destination port */ - uint32be_t seq_no; /**< Sequence number */ - uint32be_t ack_no; /**< Acknowledgment number */ + odp_u16be_t src_port; /**< Source port */ + odp_u16be_t dst_port; /**< Destination port */ + odp_u32be_t seq_no; /**< Sequence number */ + odp_u32be_t ack_no; /**< Acknowledgment number */ union { - uint16be_t doffset_flags; + odp_u16be_t doffset_flags; #if defined(ODP_BIG_ENDIAN_BITFIELD) struct { - uint16be_t rsvd1:8; - uint16be_t flags:8; /**< TCP flags as a byte */ + odp_u16be_t rsvd1:8; + odp_u16be_t flags:8; /**< TCP flags as a byte */ }; struct { - uint16be_t hl:4; /**< Hdr len, in words */ - uint16be_t rsvd3:4; /**< Reserved */ - uint16be_t cwr:1; - uint16be_t ece:1; - uint16be_t urg:1; - uint16be_t ack:1; - uint16be_t psh:1; - uint16be_t rst:1; - uint16be_t syn:1; - uint16be_t fin:1; + odp_u16be_t hl:4; /**< Hdr len, in words */ + odp_u16be_t rsvd3:4; /**< Reserved */ + odp_u16be_t cwr:1; + odp_u16be_t ece:1; + odp_u16be_t urg:1; + odp_u16be_t ack:1; + odp_u16be_t psh:1; + odp_u16be_t rst:1; + odp_u16be_t syn:1; + odp_u16be_t fin:1; }; #elif defined(ODP_LITTLE_ENDIAN_BITFIELD) struct { - uint16be_t flags:8; - uint16be_t rsvd1:8; /**< TCP flags as a byte */ + odp_u16be_t flags:8; + odp_u16be_t rsvd1:8; /**< TCP flags as a byte */ }; struct { - uint16be_t rsvd3:4; /**< Reserved */ - uint16be_t hl:4; /**< Hdr len, in words */ - uint16be_t fin:1; - uint16be_t syn:1; - uint16be_t rst:1; - uint16be_t psh:1; - uint16be_t ack:1; - uint16be_t urg:1; - uint16be_t ece:1; - uint16be_t cwr:1; + odp_u16be_t rsvd3:4; /**< Reserved */ + odp_u16be_t hl:4; /**< Hdr len, in words */ + odp_u16be_t fin:1; + odp_u16be_t syn:1; + odp_u16be_t rst:1; + odp_u16be_t psh:1; + odp_u16be_t ack:1; + odp_u16be_t urg:1; + odp_u16be_t ece:1; + odp_u16be_t cwr:1; }; #else #error "Endian BitField order not defined!" #endif }; - uint16be_t window; /**< Window size */ - uint16be_t cksm; /**< Checksum */ - uint16be_t urgptr; /**< Urgent pointer */ + odp_u16be_t window; /**< Window size */ + odp_u16be_t cksm; /**< Checksum */ + odp_u16be_t urgptr; /**< Urgent pointer */ } odph_tcphdr_t; /** diff --git a/helper/include/odp/helper/udp.h b/helper/include/odp/helper/udp.h index 93b342d..88a77f8 100644 --- a/helper/include/odp/helper/udp.h +++ b/helper/include/odp/helper/udp.h @@ -30,10 +30,10 @@ extern "C" { /** UDP header */ typedef struct ODP_PACKED { - uint16be_t src_port; /**< Source port */ - uint16be_t dst_port; /**< Destination port */ - uint16be_t length; /**< UDP datagram length in bytes (header+data) */ - uint16be_t chksum; /**< UDP header and data checksum (0 if not used)*/ + odp_u16be_t src_port; /**< Source port */ + odp_u16be_t dst_port; /**< Destination port */ + odp_u16be_t length; /**< UDP datagram length in bytes (header+data) */ + odp_u16be_t chksum; /**< UDP header and data checksum (0 if not used)*/ } odph_udphdr_t; /** diff --git a/include/odp/api/byteorder.h b/include/odp/api/byteorder.h index a7b3647..a12a729 100644 --- a/include/odp/api/byteorder.h +++ b/include/odp/api/byteorder.h @@ -41,28 +41,28 @@ extern "C" { */ /** - * @typedef uint16le_t + * @typedef odp_u16le_t * unsigned 16bit little endian * - * @typedef uint16be_t + * @typedef odp_u16be_t * unsigned 16bit big endian * - * @typedef uint32le_t + * @typedef odp_u32le_t * unsigned 32bit little endian * - * @typedef uint32be_t + * @typedef odp_u32be_t * unsigned 32bit big endian * - * @typedef uint64le_t + * @typedef odp_u64le_t * unsigned 64bit little endian * - * @typedef uint64be_t + * @typedef odp_u64be_t * unsigned 64bit big endian * - * @typedef uint16sum_t + * @typedef odp_u16sum_t * unsigned 16bit bitwise * - * @typedef uint32sum_t + * @typedef odp_u32sum_t * unsigned 32bit bitwise */ @@ -75,21 +75,21 @@ extern "C" { * @param be16 big endian 16bit * @return cpu native uint16_t */ -uint16_t odp_be_to_cpu_16(uint16be_t be16); +uint16_t odp_be_to_cpu_16(odp_u16be_t be16); /** * Convert 32bit big endian to cpu native uint32_t * @param be32 big endian 32bit * @return cpu native uint32_t */ -uint32_t odp_be_to_cpu_32(uint32be_t be32); +uint32_t odp_be_to_cpu_32(odp_u32be_t be32); /** * Convert 64bit big endian to cpu native uint64_t * @param be64 big endian 64bit * @return cpu native uint64_t */ -uint64_t odp_be_to_cpu_64(uint64be_t be64); +uint64_t odp_be_to_cpu_64(odp_u64be_t be64); /* @@ -101,21 +101,21 @@ uint64_t odp_be_to_cpu_64(uint64be_t be64); * @param cpu16 uint16_t in cpu native format * @return big endian 16bit */ -uint16be_t odp_cpu_to_be_16(uint16_t cpu16); +odp_u16be_t odp_cpu_to_be_16(uint16_t cpu16); /** * Convert cpu native uint32_t to 32bit big endian * @param cpu32 uint32_t in cpu native format * @return big endian 32bit */ -uint32be_t odp_cpu_to_be_32(uint32_t cpu32); +odp_u32be_t odp_cpu_to_be_32(uint32_t cpu32); /** * Convert cpu native uint64_t to 64bit big endian * @param cpu64 uint64_t in cpu native format * @return big endian 64bit */ -uint64be_t odp_cpu_to_be_64(uint64_t cpu64); +odp_u64be_t odp_cpu_to_be_64(uint64_t cpu64); /* @@ -127,21 +127,21 @@ uint64be_t odp_cpu_to_be_64(uint64_t cpu64); * @param le16 little endian 16bit * @return cpu native uint16_t */ -uint16_t odp_le_to_cpu_16(uint16le_t le16); +uint16_t odp_le_to_cpu_16(odp_u16le_t le16); /** * Convert 32bit little endian to cpu native uint32_t * @param le32 little endian 32bit * @return cpu native uint32_t */ -uint32_t odp_le_to_cpu_32(uint32le_t le32); +uint32_t odp_le_to_cpu_32(odp_u32le_t le32); /** * Convert 64bit little endian to cpu native uint64_t * @param le64 little endian 64bit * @return cpu native uint64_t */ -uint64_t odp_le_to_cpu_64(uint64le_t le64); +uint64_t odp_le_to_cpu_64(odp_u64le_t le64); /* @@ -153,21 +153,21 @@ uint64_t odp_le_to_cpu_64(uint64le_t le64); * @param cpu16 uint16_t in cpu native format * @return little endian 16bit */ -uint16le_t odp_cpu_to_le_16(uint16_t cpu16); +odp_u16le_t odp_cpu_to_le_16(uint16_t cpu16); /** * Convert cpu native uint32_t to 32bit little endian * @param cpu32 uint32_t in cpu native format * @return little endian 32bit */ -uint32le_t odp_cpu_to_le_32(uint32_t cpu32); +odp_u32le_t odp_cpu_to_le_32(uint32_t cpu32); /** * Convert cpu native uint64_t to 64bit little endian * @param cpu64 uint64_t in cpu native format * @return little endian 64bit */ -uint64le_t odp_cpu_to_le_64(uint64_t cpu64); +odp_u64le_t odp_cpu_to_le_64(uint64_t cpu64); /** * @} diff --git a/platform/linux-generic/include/odp/byteorder.h b/platform/linux-generic/include/odp/byteorder.h index 173b047..ee60d02 100644 --- a/platform/linux-generic/include/odp/byteorder.h +++ b/platform/linux-generic/include/odp/byteorder.h @@ -27,7 +27,7 @@ extern "C" { * @{ */ -static inline uint16_t odp_be_to_cpu_16(uint16be_t be16) +static inline uint16_t odp_be_to_cpu_16(odp_u16be_t be16) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN return __odp_builtin_bswap16((__odp_force uint16_t)be16); @@ -36,7 +36,7 @@ static inline uint16_t odp_be_to_cpu_16(uint16be_t be16) #endif } -static inline uint32_t odp_be_to_cpu_32(uint32be_t be32) +static inline uint32_t odp_be_to_cpu_32(odp_u32be_t be32) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN return __builtin_bswap32((__odp_force uint32_t)be32); @@ -45,7 +45,7 @@ static inline uint32_t odp_be_to_cpu_32(uint32be_t be32) #endif } -static inline uint64_t odp_be_to_cpu_64(uint64be_t be64) +static inline uint64_t odp_be_to_cpu_64(odp_u64be_t be64) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN return __builtin_bswap64((__odp_force uint64_t)be64); @@ -55,35 +55,35 @@ static inline uint64_t odp_be_to_cpu_64(uint64be_t be64) } -static inline uint16be_t odp_cpu_to_be_16(uint16_t cpu16) +static inline odp_u16be_t odp_cpu_to_be_16(uint16_t cpu16) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN - return (__odp_force uint16be_t)__odp_builtin_bswap16(cpu16); + return (__odp_force odp_u16be_t)__odp_builtin_bswap16(cpu16); #else - return (__odp_force uint16be_t)cpu16; + return (__odp_force odp_u16be_t)cpu16; #endif } -static inline uint32be_t odp_cpu_to_be_32(uint32_t cpu32) +static inline odp_u32be_t odp_cpu_to_be_32(uint32_t cpu32) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN - return (__odp_force uint32be_t)__builtin_bswap32(cpu32); + return (__odp_force odp_u32be_t)__builtin_bswap32(cpu32); #else - return (__odp_force uint32be_t)cpu32; + return (__odp_force odp_u32be_t)cpu32; #endif } -static inline uint64be_t odp_cpu_to_be_64(uint64_t cpu64) +static inline odp_u64be_t odp_cpu_to_be_64(uint64_t cpu64) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN - return (__odp_force uint64be_t)__builtin_bswap64(cpu64); + return (__odp_force odp_u64be_t)__builtin_bswap64(cpu64); #else - return (__odp_force uint64be_t)cpu64; + return (__odp_force odp_u64be_t)cpu64; #endif } -static inline uint16_t odp_le_to_cpu_16(uint16le_t le16) +static inline uint16_t odp_le_to_cpu_16(odp_u16le_t le16) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN return (__odp_force uint16_t)le16; @@ -92,7 +92,7 @@ static inline uint16_t odp_le_to_cpu_16(uint16le_t le16) #endif } -static inline uint32_t odp_le_to_cpu_32(uint32le_t le32) +static inline uint32_t odp_le_to_cpu_32(odp_u32le_t le32) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN return (__odp_force uint32_t)le32; @@ -101,7 +101,7 @@ static inline uint32_t odp_le_to_cpu_32(uint32le_t le32) #endif } -static inline uint64_t odp_le_to_cpu_64(uint64le_t le64) +static inline uint64_t odp_le_to_cpu_64(odp_u64le_t le64) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN return (__odp_force uint64_t)le64; @@ -111,30 +111,30 @@ static inline uint64_t odp_le_to_cpu_64(uint64le_t le64) } -static inline uint16le_t odp_cpu_to_le_16(uint16_t cpu16) +static inline odp_u16le_t odp_cpu_to_le_16(uint16_t cpu16) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN - return (__odp_force uint16le_t)cpu16; + return (__odp_force odp_u16le_t)cpu16; #else - return (__odp_force uint16le_t)__odp_builtin_bswap16(cpu16); + return (__odp_force odp_u16le_t)__odp_builtin_bswap16(cpu16); #endif } -static inline uint32le_t odp_cpu_to_le_32(uint32_t cpu32) +static inline odp_u32le_t odp_cpu_to_le_32(uint32_t cpu32) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN - return (__odp_force uint32le_t)cpu32; + return (__odp_force odp_u32le_t)cpu32; #else - return (__odp_force uint32le_t)__builtin_bswap32(cpu32); + return (__odp_force odp_u32le_t)__builtin_bswap32(cpu32); #endif } -static inline uint64le_t odp_cpu_to_le_64(uint64_t cpu64) +static inline odp_u64le_t odp_cpu_to_le_64(uint64_t cpu64) { #if ODP_BYTE_ORDER == ODP_LITTLE_ENDIAN - return (__odp_force uint64le_t)cpu64; + return (__odp_force odp_u64le_t)cpu64; #else - return (__odp_force uint64le_t)__builtin_bswap64(cpu64); + return (__odp_force odp_u64le_t)__builtin_bswap64(cpu64); #endif } diff --git a/platform/linux-generic/include/odp/plat/byteorder_types.h b/platform/linux-generic/include/odp/plat/byteorder_types.h index cf917b1..0a8e409 100644 --- a/platform/linux-generic/include/odp/plat/byteorder_types.h +++ b/platform/linux-generic/include/odp/plat/byteorder_types.h @@ -67,17 +67,17 @@ extern "C" { #define ODP_BYTE_ORDER ODP_BIG_ENDIAN #endif -typedef uint16_t __odp_bitwise uint16le_t; -typedef uint16_t __odp_bitwise uint16be_t; +typedef uint16_t __odp_bitwise odp_u16le_t; +typedef uint16_t __odp_bitwise odp_u16be_t; -typedef uint32_t __odp_bitwise uint32le_t; -typedef uint32_t __odp_bitwise uint32be_t; +typedef uint32_t __odp_bitwise odp_u32le_t; +typedef uint32_t __odp_bitwise odp_u32be_t; -typedef uint64_t __odp_bitwise uint64le_t; -typedef uint64_t __odp_bitwise uint64be_t; +typedef uint64_t __odp_bitwise odp_u64le_t; +typedef uint64_t __odp_bitwise odp_u64be_t; -typedef uint16_t __odp_bitwise uint16sum_t; -typedef uint32_t __odp_bitwise uint32sum_t; +typedef uint16_t __odp_bitwise odp_u16sum_t; +typedef uint32_t __odp_bitwise odp_u32sum_t; /** * @} diff --git a/test/performance/odp_pktio_perf.c b/test/performance/odp_pktio_perf.c index 9fdc4cb..f85f882 100644 --- a/test/performance/odp_pktio_perf.c +++ b/test/performance/odp_pktio_perf.c @@ -145,7 +145,7 @@ typedef struct { } thread_args_t; typedef struct { - uint32be_t magic; /* Packet header magic number */ + odp_u32be_t magic; /* Packet header magic number */ } pkt_head_t; /* Pool from which transmitted packets are allocated */ diff --git a/test/validation/classification/odp_classification_common.c b/test/validation/classification/odp_classification_common.c index bb937dc..70d21a2 100644 --- a/test/validation/classification/odp_classification_common.c +++ b/test/validation/classification/odp_classification_common.c @@ -13,8 +13,8 @@ #include <odp/helper/tcp.h> typedef struct cls_test_packet { - uint32be_t magic; - uint32be_t seq; + odp_u32be_t magic; + odp_u32be_t seq; } cls_test_packet_t; int destroy_inq(odp_pktio_t pktio) @@ -254,7 +254,7 @@ odp_packet_t create_packet_len(odp_pool_t pool, bool vlan, vlan->tpid = odp_cpu_to_be_16(ODPH_ETHTYPE_VLAN); offset += sizeof(odph_vlanhdr_t); parseptr += sizeof(odph_vlanhdr_t); - uint16be_t *type = (uint16be_t *)(void *)parseptr; + odp_u16be_t *type = (odp_u16be_t *)(void *)parseptr; *type = odp_cpu_to_be_16(ODPH_ETHTYPE_IPV4); } else { ethhdr->type = odp_cpu_to_be_16(ODPH_ETHTYPE_IPV4); diff --git a/test/validation/pktio/pktio.c b/test/validation/pktio/pktio.c index eda7993..6a5fb03 100644 --- a/test/validation/pktio/pktio.c +++ b/test/validation/pktio/pktio.c @@ -45,13 +45,13 @@ typedef struct { /** magic number and sequence at start of UDP payload */ typedef struct ODP_PACKED { - uint32be_t magic; - uint32be_t seq; + odp_u32be_t magic; + odp_u32be_t seq; } pkt_head_t; /** magic number at end of UDP payload */ typedef struct ODP_PACKED { - uint32be_t magic; + odp_u32be_t magic; } pkt_tail_t; /** Run mode */
The following types: uint16le_t, uint16be_t, uint32le_t, uint32be_t, uint64le_t, uint64be_t, uint16sum_t, uint32sum_t defined in the api and in platform/linux-generic/include/odp/plat/byteorder_types.h were not odp_* prefixed and could create name clash with an applications. This patch prefixes those with odp_*, suppressing "int" for shortness. e.g. uint16le_t becomes odp_u16le_t Also modifies files using these types, of course. Signed-off-by: Christophe Milard <christophe.milard@linaro.org> --- example/classifier/odp_classifier.c | 2 +- example/ipsec/odp_ipsec_cache.h | 2 +- example/ipsec/odp_ipsec_stream.c | 2 +- example/packet/odp_pktio.c | 2 +- helper/include/odp/helper/chksum.h | 4 +- helper/include/odp/helper/eth.h | 6 +- helper/include/odp/helper/icmp.h | 12 ++-- helper/include/odp/helper/ip.h | 22 ++++---- helper/include/odp/helper/ipsec.h | 10 ++-- helper/include/odp/helper/tcp.h | 64 +++++++++++----------- helper/include/odp/helper/udp.h | 8 +-- include/odp/api/byteorder.h | 40 +++++++------- platform/linux-generic/include/odp/byteorder.h | 48 ++++++++-------- .../include/odp/plat/byteorder_types.h | 16 +++--- test/performance/odp_pktio_perf.c | 2 +- .../classification/odp_classification_common.c | 6 +- test/validation/pktio/pktio.c | 6 +- 17 files changed, 126 insertions(+), 126 deletions(-)