DPDK patches and discussions
 help / color / mirror / Atom feed
* 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 directly to us
@ 2020-11-09  6:24 Guo, Jia
  2020-11-09  9:49 ` [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
  0 siblings, 1 reply; 4+ messages in thread
From: Guo, Jia @ 2020-11-09  6:24 UTC (permalink / raw)
  To: Prateek Agarwal; +Cc: dev

Hi, Agarwal

> -----Original Message-----
> From: Prateek Agarwal <pratekag@gmail.com>
> Sent: Saturday, November 7, 2020 5:01 PM
> To: Guo, Jia <jia.guo@intel.com>
> Cc: dev@dpdk.org; Prateek Agarwal <pratekag@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. 

> Signed-off-by: Prateek Agarwal <pratekag@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


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

* 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...
  2020-11-09  6:24 [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 directly to us Guo, Jia
@ 2020-11-09  9:49 ` Zhang, Qi Z
  2020-11-09 17:44   ` Ferruh Yigit
  0 siblings, 1 reply; 4+ messages in thread
From: Zhang, Qi Z @ 2020-11-09  9:49 UTC (permalink / raw)
  To: Guo, Jia, Prateek Agarwal; +Cc: dev



> -----Original Message-----
> From: dev <dev-bounces@dpdk.org> On Behalf Of Guo, Jia
> Sent: Monday, November 9, 2020 2:24 PM
> To: Prateek Agarwal <pratekag@gmail.com>
> Cc: dev@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@gmail.com>
> > Sent: Saturday, November 7, 2020 5:01 PM
> > To: Guo, Jia <jia.guo@intel.com>
> > Cc: dev@dpdk.org; Prateek Agarwal <pratekag@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@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


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

* 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...
  2020-11-09  9:49 ` [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
@ 2020-11-09 17:44   ` Ferruh Yigit
  2020-11-10 12:44     ` Ferruh Yigit
  0 siblings, 1 reply; 4+ messages in thread
From: Ferruh Yigit @ 2020-11-09 17:44 UTC (permalink / raw)
  To: Zhang, Qi Z, Guo, Jia, Prateek Agarwal; +Cc: dev

On 11/9/2020 9:49 AM, Zhang, Qi Z wrote:
> 
> 
>> -----Original Message-----
>> From: dev <dev-bounces@dpdk.org> On Behalf Of Guo, Jia
>> Sent: Monday, November 9, 2020 2:24 PM
>> To: Prateek Agarwal <pratekag@gmail.com>
>> Cc: dev@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@gmail.com>
>>> Sent: Saturday, November 7, 2020 5:01 PM
>>> To: Guo, Jia <jia.guo@intel.com>
>>> Cc: dev@dpdk.org; Prateek Agarwal <pratekag@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.
> 

Hi Prateek,

In the git history, we have your name as:
"Prateek Agarwal <prateekag@cse.iitb.ac.in>"

We are trying to keep authors unique in the git history, as much as possible, 
will be OK to continue with the one in the git history?

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

* 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...
  2020-11-09 17:44   ` Ferruh Yigit
@ 2020-11-10 12:44     ` Ferruh Yigit
  0 siblings, 0 replies; 4+ messages in thread
From: Ferruh Yigit @ 2020-11-10 12:44 UTC (permalink / raw)
  To: Zhang, Qi Z, Guo, Jia, Prateek Agarwal; +Cc: dev

On 11/9/2020 5:44 PM, Ferruh Yigit wrote:
> On 11/9/2020 9:49 AM, Zhang, Qi Z wrote:
>>
>>
>>> -----Original Message-----
>>> From: dev <dev-bounces@dpdk.org> On Behalf Of Guo, Jia
>>> Sent: Monday, November 9, 2020 2:24 PM
>>> To: Prateek Agarwal <pratekag@gmail.com>
>>> Cc: dev@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@gmail.com>
>>>> Sent: Saturday, November 7, 2020 5:01 PM
>>>> To: Guo, Jia <jia.guo@intel.com>
>>>> Cc: dev@dpdk.org; Prateek Agarwal <pratekag@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.
>>
> 
> Hi Prateek,
> 
> In the git history, we have your name as:
> "Prateek Agarwal <prateekag@cse.iitb.ac.in>"
> 
> We are trying to keep authors unique in the git history, as much as possible, 
> will be OK to continue with the one in the git history?

Using the one in the git history, thanks.

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

end of thread, other threads:[~2020-11-10 12:45 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-09  6:24 [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 directly to us Guo, Jia
2020-11-09  9:49 ` [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
2020-11-09 17:44   ` Ferruh Yigit
2020-11-10 12:44     ` Ferruh Yigit

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).