d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

eventdev: fix headers for strict compilation flags

Browse Source 
Exported headers must allow compilation with the strictest flags. This
commit addresses the following errors:

 In file included from build/include/rte_eventdev_pmd.h:55:0,
                  from /tmp/check-includes.sh.25816.c:1:
 build/include/rte_eventdev.h:908:8: error: struct has no named members
    [-Werror=pedantic]
 [...]
 In file included from /tmp/check-includes.sh.25816.c:1:0:
 build/include/rte_eventdev_pmd.h:65:35: error: ISO C does not permit named
    variadic macros [-Werror=variadic-macros]
 [...]

Fixes: 71f238432865 ("eventdev: introduce event driven programming model")
Fixes: 4f0804bbdfb9 ("eventdev: implement the northbound APIs")

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
This commit is contained in:
Adrien Mazarguil 2017-04-26 14:07:10 +02:00 committed by Thomas Monjalon
parent f48a55f78e
commit 507fb14c2a
2 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/librte_eventdev/rte_eventdev.h
View File

@ -905,9 +905,9 @@ rte_event_dev_close(uint8_t dev_id);
* The generic *rte_event* structure to hold the event attributes
* for dequeue and enqueue operation
*/
RTE_STD_C11
struct rte_event {
/** WORD0 */
RTE_STD_C11
union {
uint64_t event;
/** Event attributes for dequeue or enqueue operation */
@ -967,7 +967,6 @@ struct rte_event {
};
};
/** WORD1 */
RTE_STD_C11
union {
uint64_t u64;
/**< Opaque 64-bit value */

16
lib/librte_eventdev/rte_eventdev_pmd.h
View File

@ -55,16 +55,18 @@ extern "C" {
#include "rte_eventdev.h"
/* Logging Macros */
#define RTE_EDEV_LOG_ERR(fmt, args...) \
RTE_LOG(ERR, EVENTDEV, "%s() line %u: " fmt "\n", \
__func__, __LINE__, ## args)
#define RTE_EDEV_LOG_ERR(...) \
RTE_LOG(ERR, EVENTDEV, \
RTE_FMT("%s() line %u: " RTE_FMT_HEAD(__VA_ARGS__,) "\n", \
__func__, __LINE__, RTE_FMT_TAIL(__VA_ARGS__,)))
#ifdef RTE_LIBRTE_EVENTDEV_DEBUG
#define RTE_EDEV_LOG_DEBUG(fmt, args...) \
RTE_LOG(DEBUG, EVENTDEV, "%s() line %u: " fmt "\n", \
__func__, __LINE__, ## args)
#define RTE_EDEV_LOG_DEBUG(...) \
RTE_LOG(DEBUG, EVENTDEV, \
RTE_FMT("%s() line %u: " RTE_FMT_HEAD(__VA_ARGS__,) "\n", \
__func__, __LINE__, RTE_FMT_TAIL(__VA_ARGS__,)))
#else
#define RTE_EDEV_LOG_DEBUG(fmt, args...) (void)0
#define RTE_EDEV_LOG_DEBUG(...) (void)0
#endif
/* Macros to check for valid device */