Skip to content

Commit

Permalink
mbuf: add C++ include guard for dynamic fields header
Browse files Browse the repository at this point in the history
[ upstream commit 397fb6a ]

The header was missing the extern "C" directive which causes name
mangling of functions by C++ compilers, leading to linker errors
complaining of undefined references to these functions.

Fixes: 4958ca3 ("mbuf: support dynamic fields and flags")

Signed-off-by: Ashish Sadanandan <ashish.sadanandan@gmail.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
  • Loading branch information
praetorian20 authored and bluca committed Feb 2, 2021
1 parent a02a695 commit 81a66f3
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions lib/librte_mbuf/rte_mbuf_dyn.h
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,11 @@
*/

#include <sys/types.h>

#ifdef __cplusplus
extern "C" {
#endif

/**
* Maximum length of the dynamic field or flag string.
*/
Expand Down Expand Up @@ -326,4 +331,8 @@ int rte_mbuf_dyn_rx_timestamp_register(int *field_offset, uint64_t *rx_flag);
__rte_experimental
int rte_mbuf_dyn_tx_timestamp_register(int *field_offset, uint64_t *tx_flag);

#ifdef __cplusplus
}
#endif

#endif /* _RTE_MBUF_DYN_H_ */

0 comments on commit 81a66f3

Please sign in to comment.