[dpdk-dev] [PATCH] Made i40 header CPP compatible using extern "C". Library headers work directly in cpp code. Linking errors thrown due to the absence of this change in i40e pmd header does not help in resolving the problem. I needed this header d...

Zhang, Qi Z qi.z.zhang at intel.com
Mon Nov 9 10:49:49 CET 2020



> -----Original Message-----
> From: dev <dev-bounces at dpdk.org> On Behalf Of Guo, Jia
> Sent: Monday, November 9, 2020 2:24 PM
> To: Prateek Agarwal <pratekag at gmail.com>
> Cc: dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] Made i40 header CPP compatible using extern
> "C". Library headers work directly in cpp code. Linking errors thrown due to the
> absence of this change in i40e pmd header does not help in resolving the
> problem. I needed this header d...
> 
> Hi, Agarwal
> 
> > -----Original Message-----
> > From: Prateek Agarwal <pratekag at gmail.com>
> > Sent: Saturday, November 7, 2020 5:01 PM
> > To: Guo, Jia <jia.guo at intel.com>
> > Cc: dev at dpdk.org; Prateek Agarwal <pratekag at gmail.com>
> > Subject: [PATCH] Made i40 header CPP compatible using extern "C".
> > Library headers work directly in cpp code. Linking errors thrown due
> > to the absence of this change in i40e pmd header does not help in
> > resolving the problem. I needed this header directly to use...
> >
> 
> Obviously that is not you want to merge the title and the commit log. Please
> separate it and update one new version.


Fixed the format during merge as below

net/i40e: add extern "C" for cpp compile

Add extern "C" in rte_pmd_i40e.h when be compiled with cpp.

Applied to dpdk-next-net-intel.

Thanks
Qi


> 
> > Signed-off-by: Prateek Agarwal <pratekag at gmail.com>
> > ---
> >  drivers/net/i40e/rte_pmd_i40e.h | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> >
> > diff --git a/drivers/net/i40e/rte_pmd_i40e.h
> > b/drivers/net/i40e/rte_pmd_i40e.h index fc3560c28..4cb21c371 100644
> > --- a/drivers/net/i40e/rte_pmd_i40e.h
> > +++ b/drivers/net/i40e/rte_pmd_i40e.h
> > @@ -14,6 +14,10 @@
> >   *
> >   */
> >
> > +#ifdef __cplusplus
> > +extern "C" {
> > +#endif
> > +
> >  #include <rte_compat.h>
> >  #include <rte_ethdev.h>
> >  #include <rte_ether.h>
> > @@ -1130,4 +1134,8 @@ __rte_experimental  int
> > rte_pmd_i40e_set_switch_dev(uint16_t port_id, struct rte_eth_dev
> > *switch_dev);
> >
> > +#ifdef __cplusplus
> > +}
> > +#endif
> > +
> >  #endif /* _PMD_I40E_H_ */
> > --
> > 2.25.1



More information about the dev mailing list