DPDK-dev Archive on lore.kernel.org
 help / color / Atom feed
* [dpdk-dev] [PATCH] net/octeontx2: fix ptp functionality
@ 2020-02-15  8:35 Harman Kalra
  2020-02-15 14:12 ` Jerin Jacob
  2020-02-17 13:21 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
  0 siblings, 2 replies; 10+ messages in thread
From: Harman Kalra @ 2020-02-15  8:35 UTC (permalink / raw)
  To: Jerin Jacob, Nithin Dabilpuram, Vamsi Attunuru; +Cc: dev, Harman Kalra, stable

PTP functionality has been broken after a change in kernel
where enum npc_kpu_lc_ltype is change to allow adjustment of
LTYPE_MASK to detect all types of IP headers.
Syncing the required changes in DPDK to fix the issue.

Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
Cc: stable@dpdk.org

Signed-off-by: Harman Kalra <hkalra@marvell.com>
---
 drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
index a0536e0ae..3dfc137a3 100644
--- a/drivers/common/octeontx2/hw/otx2_npc.h
+++ b/drivers/common/octeontx2/hw/otx2_npc.h
@@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
 };
 
 enum npc_kpu_lc_ltype {
-	NPC_LT_LC_IP = 1,
+	NPC_LT_LC_PTP = 1,
+	NPC_LT_LC_IP,
 	NPC_LT_LC_IP_OPT,
 	NPC_LT_LC_IP6,
 	NPC_LT_LC_IP6_EXT,
@@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
 	NPC_LT_LC_RARP,
 	NPC_LT_LC_MPLS,
 	NPC_LT_LC_NSH,
-	NPC_LT_LC_PTP,
 	NPC_LT_LC_FCOE,
 };
 
-- 
2.18.0


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

* Re: [dpdk-dev] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-15  8:35 [dpdk-dev] [PATCH] net/octeontx2: fix ptp functionality Harman Kalra
@ 2020-02-15 14:12 ` Jerin Jacob
  2020-02-17 13:21 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
  1 sibling, 0 replies; 10+ messages in thread
From: Jerin Jacob @ 2020-02-15 14:12 UTC (permalink / raw)
  To: Harman Kalra, Ferruh Yigit
  Cc: Jerin Jacob, Nithin Dabilpuram, Vamsi Attunuru, dpdk-dev, dpdk stable

On Sat, Feb 15, 2020 at 2:05 PM Harman Kalra <hkalra@marvell.com> wrote:
>
> PTP functionality has been broken after a change in kernel
> where enum npc_kpu_lc_ltype is change to allow adjustment of
> LTYPE_MASK to detect all types of IP headers.
> Syncing the required changes in DPDK to fix the issue.
>
> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
> Cc: stable@dpdk.org
>
> Signed-off-by: Harman Kalra <hkalra@marvell.com>

Acked-by: Jerin Jacob <jerinj@marvell.com>
Applied to dpdk-next-net-mrvl/master. Thanks


> ---
>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
> index a0536e0ae..3dfc137a3 100644
> --- a/drivers/common/octeontx2/hw/otx2_npc.h
> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
>  };
>
>  enum npc_kpu_lc_ltype {
> -       NPC_LT_LC_IP = 1,
> +       NPC_LT_LC_PTP = 1,
> +       NPC_LT_LC_IP,
>         NPC_LT_LC_IP_OPT,
>         NPC_LT_LC_IP6,
>         NPC_LT_LC_IP6_EXT,
> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
>         NPC_LT_LC_RARP,
>         NPC_LT_LC_MPLS,
>         NPC_LT_LC_NSH,
> -       NPC_LT_LC_PTP,
>         NPC_LT_LC_FCOE,
>  };
>
> --
> 2.18.0
>

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

* Re: [dpdk-dev] [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-15  8:35 [dpdk-dev] [PATCH] net/octeontx2: fix ptp functionality Harman Kalra
  2020-02-15 14:12 ` Jerin Jacob
@ 2020-02-17 13:21 ` " Ferruh Yigit
  2020-02-17 14:35   ` [dpdk-dev] [EXT] " Harman Kalra
  1 sibling, 1 reply; 10+ messages in thread
From: Ferruh Yigit @ 2020-02-17 13:21 UTC (permalink / raw)
  To: Harman Kalra, Jerin Jacob, Nithin Dabilpuram, Vamsi Attunuru; +Cc: dev, stable

On 2/15/2020 8:35 AM, Harman Kalra wrote:
> PTP functionality has been broken after a change in kernel
> where enum npc_kpu_lc_ltype is change to allow adjustment of
> LTYPE_MASK to detect all types of IP headers.
> Syncing the required changes in DPDK to fix the issue.

Hi Harman,

Can you please provide references for,
- What is the kernel commit causing the mentioned change?
- After this change which version of the kernel supported by the PMD?

> 
> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Harman Kalra <hkalra@marvell.com>
> ---
>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
> index a0536e0ae..3dfc137a3 100644
> --- a/drivers/common/octeontx2/hw/otx2_npc.h
> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
>  };
>  
>  enum npc_kpu_lc_ltype {
> -	NPC_LT_LC_IP = 1,
> +	NPC_LT_LC_PTP = 1,
> +	NPC_LT_LC_IP,
>  	NPC_LT_LC_IP_OPT,
>  	NPC_LT_LC_IP6,
>  	NPC_LT_LC_IP6_EXT,
> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
>  	NPC_LT_LC_RARP,
>  	NPC_LT_LC_MPLS,
>  	NPC_LT_LC_NSH,
> -	NPC_LT_LC_PTP,
>  	NPC_LT_LC_FCOE,
>  };
>  
> 


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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 13:21 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
@ 2020-02-17 14:35   ` " Harman Kalra
  2020-02-17 14:54     ` Ferruh Yigit
  0 siblings, 1 reply; 10+ messages in thread
From: Harman Kalra @ 2020-02-17 14:35 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
> External Email
> 
> ----------------------------------------------------------------------
> On 2/15/2020 8:35 AM, Harman Kalra wrote:
> > PTP functionality has been broken after a change in kernel
> > where enum npc_kpu_lc_ltype is change to allow adjustment of
> > LTYPE_MASK to detect all types of IP headers.
> > Syncing the required changes in DPDK to fix the issue.
> 
> Hi Harman,
> 
> Can you please provide references for,
> - What is the kernel commit causing the mentioned change?
> - After this change which version of the kernel supported by the PMD?

Hi Ferruh
The kernel change I was refering to was in our internel kernel repo
and those kernel changes have not been upstreamed yet. Currently its part
of our marvell SDK version SDK-10.3.1.x.

Thanks
Harman

> 
> > 
> > Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
> > Cc: stable@dpdk.org
> > 
> > Signed-off-by: Harman Kalra <hkalra@marvell.com>
> > ---
> >  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
> > index a0536e0ae..3dfc137a3 100644
> > --- a/drivers/common/octeontx2/hw/otx2_npc.h
> > +++ b/drivers/common/octeontx2/hw/otx2_npc.h
> > @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
> >  };
> >  
> >  enum npc_kpu_lc_ltype {
> > -	NPC_LT_LC_IP = 1,
> > +	NPC_LT_LC_PTP = 1,
> > +	NPC_LT_LC_IP,
> >  	NPC_LT_LC_IP_OPT,
> >  	NPC_LT_LC_IP6,
> >  	NPC_LT_LC_IP6_EXT,
> > @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
> >  	NPC_LT_LC_RARP,
> >  	NPC_LT_LC_MPLS,
> >  	NPC_LT_LC_NSH,
> > -	NPC_LT_LC_PTP,
> >  	NPC_LT_LC_FCOE,
> >  };
> >  
> > 
> 

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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 14:35   ` [dpdk-dev] [EXT] " Harman Kalra
@ 2020-02-17 14:54     ` Ferruh Yigit
  2020-02-17 15:06       ` Harman Kalra
  0 siblings, 1 reply; 10+ messages in thread
From: Ferruh Yigit @ 2020-02-17 14:54 UTC (permalink / raw)
  To: Harman Kalra; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On 2/17/2020 2:35 PM, Harman Kalra wrote:
> On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
>> External Email
>>
>> ----------------------------------------------------------------------
>> On 2/15/2020 8:35 AM, Harman Kalra wrote:
>>> PTP functionality has been broken after a change in kernel
>>> where enum npc_kpu_lc_ltype is change to allow adjustment of
>>> LTYPE_MASK to detect all types of IP headers.
>>> Syncing the required changes in DPDK to fix the issue.
>>
>> Hi Harman,
>>
>> Can you please provide references for,
>> - What is the kernel commit causing the mentioned change?
>> - After this change which version of the kernel supported by the PMD?
> 
> Hi Ferruh
> The kernel change I was refering to was in our internel kernel repo
> and those kernel changes have not been upstreamed yet. Currently its part
> of our marvell SDK version SDK-10.3.1.x.

Got it, what is the dependency of the PMD to the Linux kernel? And does the PMD
works with upstream kernels?

> 
> Thanks
> Harman
> 
>>
>>>
>>> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
>>> Cc: stable@dpdk.org
>>>
>>> Signed-off-by: Harman Kalra <hkalra@marvell.com>
>>> ---
>>>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
>>> index a0536e0ae..3dfc137a3 100644
>>> --- a/drivers/common/octeontx2/hw/otx2_npc.h
>>> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
>>> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
>>>  };
>>>  
>>>  enum npc_kpu_lc_ltype {
>>> -	NPC_LT_LC_IP = 1,
>>> +	NPC_LT_LC_PTP = 1,
>>> +	NPC_LT_LC_IP,
>>>  	NPC_LT_LC_IP_OPT,
>>>  	NPC_LT_LC_IP6,
>>>  	NPC_LT_LC_IP6_EXT,
>>> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
>>>  	NPC_LT_LC_RARP,
>>>  	NPC_LT_LC_MPLS,
>>>  	NPC_LT_LC_NSH,
>>> -	NPC_LT_LC_PTP,
>>>  	NPC_LT_LC_FCOE,
>>>  };
>>>  
>>>
>>


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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 14:54     ` Ferruh Yigit
@ 2020-02-17 15:06       ` Harman Kalra
  2020-02-17 15:19         ` Ferruh Yigit
  0 siblings, 1 reply; 10+ messages in thread
From: Harman Kalra @ 2020-02-17 15:06 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On Mon, Feb 17, 2020 at 02:54:13PM +0000, Ferruh Yigit wrote:
> On 2/17/2020 2:35 PM, Harman Kalra wrote:
> > On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
> >> External Email
> >>
> >> ----------------------------------------------------------------------
> >> On 2/15/2020 8:35 AM, Harman Kalra wrote:
> >>> PTP functionality has been broken after a change in kernel
> >>> where enum npc_kpu_lc_ltype is change to allow adjustment of
> >>> LTYPE_MASK to detect all types of IP headers.
> >>> Syncing the required changes in DPDK to fix the issue.
> >>
> >> Hi Harman,
> >>
> >> Can you please provide references for,
> >> - What is the kernel commit causing the mentioned change?
> >> - After this change which version of the kernel supported by the PMD?
> > 
> > Hi Ferruh
> > The kernel change I was refering to was in our internel kernel repo
> > and those kernel changes have not been upstreamed yet. Currently its part
> > of our marvell SDK version SDK-10.3.1.x.
> 
> Got it, what is the dependency of the PMD to the Linux kernel? And does the PMD
> works with upstream kernels?

Yes, for basic functionality PMD works fine with upstreamed kernel but for
supporting some advanced features like PTP, kernel changes are yet to be
upstreamed. Our kernel driver for octeontx2 PMD:
https://github.com/torvalds/linux/tree/master/drivers/net/ethernet/marvell/octeontx2/af
Kernel upstreaming activity is not in same pace as DPDK.

Thanks
Harman
> 
> > 
> > Thanks
> > Harman
> > 
> >>
> >>>
> >>> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
> >>> Cc: stable@dpdk.org
> >>>
> >>> Signed-off-by: Harman Kalra <hkalra@marvell.com>
> >>> ---
> >>>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
> >>>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>>
> >>> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
> >>> index a0536e0ae..3dfc137a3 100644
> >>> --- a/drivers/common/octeontx2/hw/otx2_npc.h
> >>> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
> >>> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
> >>>  };
> >>>  
> >>>  enum npc_kpu_lc_ltype {
> >>> -	NPC_LT_LC_IP = 1,
> >>> +	NPC_LT_LC_PTP = 1,
> >>> +	NPC_LT_LC_IP,
> >>>  	NPC_LT_LC_IP_OPT,
> >>>  	NPC_LT_LC_IP6,
> >>>  	NPC_LT_LC_IP6_EXT,
> >>> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
> >>>  	NPC_LT_LC_RARP,
> >>>  	NPC_LT_LC_MPLS,
> >>>  	NPC_LT_LC_NSH,
> >>> -	NPC_LT_LC_PTP,
> >>>  	NPC_LT_LC_FCOE,
> >>>  };
> >>>  
> >>>
> >>
> 

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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 15:06       ` Harman Kalra
@ 2020-02-17 15:19         ` Ferruh Yigit
  2020-02-17 15:30           ` Harman Kalra
  0 siblings, 1 reply; 10+ messages in thread
From: Ferruh Yigit @ 2020-02-17 15:19 UTC (permalink / raw)
  To: Harman Kalra; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On 2/17/2020 3:06 PM, Harman Kalra wrote:
> On Mon, Feb 17, 2020 at 02:54:13PM +0000, Ferruh Yigit wrote:
>> On 2/17/2020 2:35 PM, Harman Kalra wrote:
>>> On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
>>>> External Email
>>>>
>>>> ----------------------------------------------------------------------
>>>> On 2/15/2020 8:35 AM, Harman Kalra wrote:
>>>>> PTP functionality has been broken after a change in kernel
>>>>> where enum npc_kpu_lc_ltype is change to allow adjustment of
>>>>> LTYPE_MASK to detect all types of IP headers.
>>>>> Syncing the required changes in DPDK to fix the issue.
>>>>
>>>> Hi Harman,
>>>>
>>>> Can you please provide references for,
>>>> - What is the kernel commit causing the mentioned change?
>>>> - After this change which version of the kernel supported by the PMD?
>>>
>>> Hi Ferruh
>>> The kernel change I was refering to was in our internel kernel repo
>>> and those kernel changes have not been upstreamed yet. Currently its part
>>> of our marvell SDK version SDK-10.3.1.x.
>>
>> Got it, what is the dependency of the PMD to the Linux kernel? And does the PMD
>> works with upstream kernels?
> 
> Yes, for basic functionality PMD works fine with upstreamed kernel but for
> supporting some advanced features like PTP, kernel changes are yet to be
> upstreamed. Our kernel driver for octeontx2 PMD:
> https://github.com/torvalds/linux/tree/master/drivers/net/ethernet/marvell/octeontx2/af
> Kernel upstreaming activity is not in same pace as DPDK.

Is the marvell SDK code in a public repo? If so can you provide the details of
the kernel changes in that repo? It is good to have some reference on the matter.

> 
> Thanks
> Harman
>>
>>>
>>> Thanks
>>> Harman
>>>
>>>>
>>>>>
>>>>> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
>>>>> Cc: stable@dpdk.org
>>>>>
>>>>> Signed-off-by: Harman Kalra <hkalra@marvell.com>
>>>>> ---
>>>>>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
>>>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>>>
>>>>> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
>>>>> index a0536e0ae..3dfc137a3 100644
>>>>> --- a/drivers/common/octeontx2/hw/otx2_npc.h
>>>>> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
>>>>> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
>>>>>  };
>>>>>  
>>>>>  enum npc_kpu_lc_ltype {
>>>>> -	NPC_LT_LC_IP = 1,
>>>>> +	NPC_LT_LC_PTP = 1,
>>>>> +	NPC_LT_LC_IP,
>>>>>  	NPC_LT_LC_IP_OPT,
>>>>>  	NPC_LT_LC_IP6,
>>>>>  	NPC_LT_LC_IP6_EXT,
>>>>> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
>>>>>  	NPC_LT_LC_RARP,
>>>>>  	NPC_LT_LC_MPLS,
>>>>>  	NPC_LT_LC_NSH,
>>>>> -	NPC_LT_LC_PTP,
>>>>>  	NPC_LT_LC_FCOE,
>>>>>  };
>>>>>  
>>>>>
>>>>
>>


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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 15:19         ` Ferruh Yigit
@ 2020-02-17 15:30           ` Harman Kalra
  2020-02-17 16:17             ` Ferruh Yigit
  0 siblings, 1 reply; 10+ messages in thread
From: Harman Kalra @ 2020-02-17 15:30 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On Mon, Feb 17, 2020 at 03:19:53PM +0000, Ferruh Yigit wrote:
> On 2/17/2020 3:06 PM, Harman Kalra wrote:
> > On Mon, Feb 17, 2020 at 02:54:13PM +0000, Ferruh Yigit wrote:
> >> On 2/17/2020 2:35 PM, Harman Kalra wrote:
> >>> On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
> >>>> External Email
> >>>>
> >>>> ----------------------------------------------------------------------
> >>>> On 2/15/2020 8:35 AM, Harman Kalra wrote:
> >>>>> PTP functionality has been broken after a change in kernel
> >>>>> where enum npc_kpu_lc_ltype is change to allow adjustment of
> >>>>> LTYPE_MASK to detect all types of IP headers.
> >>>>> Syncing the required changes in DPDK to fix the issue.
> >>>>
> >>>> Hi Harman,
> >>>>
> >>>> Can you please provide references for,
> >>>> - What is the kernel commit causing the mentioned change?
> >>>> - After this change which version of the kernel supported by the PMD?
> >>>
> >>> Hi Ferruh
> >>> The kernel change I was refering to was in our internel kernel repo
> >>> and those kernel changes have not been upstreamed yet. Currently its part
> >>> of our marvell SDK version SDK-10.3.1.x.
> >>
> >> Got it, what is the dependency of the PMD to the Linux kernel? And does the PMD
> >> works with upstream kernels?
> > 
> > Yes, for basic functionality PMD works fine with upstreamed kernel but for
> > supporting some advanced features like PTP, kernel changes are yet to be
> > upstreamed. Our kernel driver for octeontx2 PMD:
> > https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_torvalds_linux_tree_master_drivers_net_ethernet_marvell_octeontx2_af&d=DwIDaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=5ESHPj7V-7JdkxT_Z_SU6RrS37ys4UXudBQ_rrS5LRo&m=pXArrQOu5hyDcQfkwc34h1CkBsAQuEnZDcanbaM-GCQ&s=c7WM8a2aLkuXH5P5HpQofM1nmCAXufFYva5YVyEiYcM&e= 
> > Kernel upstreaming activity is not in same pace as DPDK.
> 
> Is the marvell SDK code in a public repo? If so can you provide the details of
> the kernel changes in that repo? It is good to have some reference on the matter.

Marvel SDK is not in any public repo. But for reference internal kernel viz part of
SDK version SDK-10.3.1.x with commit id "08705a860e2d3aebce44d6c357ed1c0ae71d3fd9"
onwards will have the required changes.
> 
> > 
> > Thanks
> > Harman
> >>
> >>>
> >>> Thanks
> >>> Harman
> >>>
> >>>>
> >>>>>
> >>>>> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
> >>>>> Cc: stable@dpdk.org
> >>>>>
> >>>>> Signed-off-by: Harman Kalra <hkalra@marvell.com>
> >>>>> ---
> >>>>>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
> >>>>>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>>>>
> >>>>> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
> >>>>> index a0536e0ae..3dfc137a3 100644
> >>>>> --- a/drivers/common/octeontx2/hw/otx2_npc.h
> >>>>> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
> >>>>> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
> >>>>>  };
> >>>>>  
> >>>>>  enum npc_kpu_lc_ltype {
> >>>>> -	NPC_LT_LC_IP = 1,
> >>>>> +	NPC_LT_LC_PTP = 1,
> >>>>> +	NPC_LT_LC_IP,
> >>>>>  	NPC_LT_LC_IP_OPT,
> >>>>>  	NPC_LT_LC_IP6,
> >>>>>  	NPC_LT_LC_IP6_EXT,
> >>>>> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
> >>>>>  	NPC_LT_LC_RARP,
> >>>>>  	NPC_LT_LC_MPLS,
> >>>>>  	NPC_LT_LC_NSH,
> >>>>> -	NPC_LT_LC_PTP,
> >>>>>  	NPC_LT_LC_FCOE,
> >>>>>  };
> >>>>>  
> >>>>>
> >>>>
> >>
> 

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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 15:30           ` Harman Kalra
@ 2020-02-17 16:17             ` Ferruh Yigit
  2020-02-17 16:37               ` Harman Kalra
  0 siblings, 1 reply; 10+ messages in thread
From: Ferruh Yigit @ 2020-02-17 16:17 UTC (permalink / raw)
  To: Harman Kalra; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On 2/17/2020 3:30 PM, Harman Kalra wrote:
> On Mon, Feb 17, 2020 at 03:19:53PM +0000, Ferruh Yigit wrote:
>> On 2/17/2020 3:06 PM, Harman Kalra wrote:
>>> On Mon, Feb 17, 2020 at 02:54:13PM +0000, Ferruh Yigit wrote:
>>>> On 2/17/2020 2:35 PM, Harman Kalra wrote:
>>>>> On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
>>>>>> External Email
>>>>>>
>>>>>> ----------------------------------------------------------------------
>>>>>> On 2/15/2020 8:35 AM, Harman Kalra wrote:
>>>>>>> PTP functionality has been broken after a change in kernel
>>>>>>> where enum npc_kpu_lc_ltype is change to allow adjustment of
>>>>>>> LTYPE_MASK to detect all types of IP headers.
>>>>>>> Syncing the required changes in DPDK to fix the issue.
>>>>>>
>>>>>> Hi Harman,
>>>>>>
>>>>>> Can you please provide references for,
>>>>>> - What is the kernel commit causing the mentioned change?
>>>>>> - After this change which version of the kernel supported by the PMD?
>>>>>
>>>>> Hi Ferruh
>>>>> The kernel change I was refering to was in our internel kernel repo
>>>>> and those kernel changes have not been upstreamed yet. Currently its part
>>>>> of our marvell SDK version SDK-10.3.1.x.
>>>>
>>>> Got it, what is the dependency of the PMD to the Linux kernel? And does the PMD
>>>> works with upstream kernels?
>>>
>>> Yes, for basic functionality PMD works fine with upstreamed kernel but for
>>> supporting some advanced features like PTP, kernel changes are yet to be
>>> upstreamed. Our kernel driver for octeontx2 PMD:
>>> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_torvalds_linux_tree_master_drivers_net_ethernet_marvell_octeontx2_af&d=DwIDaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=5ESHPj7V-7JdkxT_Z_SU6RrS37ys4UXudBQ_rrS5LRo&m=pXArrQOu5hyDcQfkwc34h1CkBsAQuEnZDcanbaM-GCQ&s=c7WM8a2aLkuXH5P5HpQofM1nmCAXufFYva5YVyEiYcM&e= 
>>> Kernel upstreaming activity is not in same pace as DPDK.
>>
>> Is the marvell SDK code in a public repo? If so can you provide the details of
>> the kernel changes in that repo? It is good to have some reference on the matter.
> 
> Marvel SDK is not in any public repo. But for reference internal kernel viz part of
> SDK version SDK-10.3.1.x with commit id "08705a860e2d3aebce44d6c357ed1c0ae71d3fd9"
> onwards will have the required changes.

If it is not public, the commit id won't help much. Would you be OK if I add
something like following while merging:

The kernel side changes are exists in marvell SDK version SDK-10.3.1.x.

>>
>>>
>>> Thanks
>>> Harman
>>>>
>>>>>
>>>>> Thanks
>>>>> Harman
>>>>>
>>>>>>
>>>>>>>
>>>>>>> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
>>>>>>> Cc: stable@dpdk.org
>>>>>>>
>>>>>>> Signed-off-by: Harman Kalra <hkalra@marvell.com>
>>>>>>> ---
>>>>>>>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
>>>>>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>>>>>
>>>>>>> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
>>>>>>> index a0536e0ae..3dfc137a3 100644
>>>>>>> --- a/drivers/common/octeontx2/hw/otx2_npc.h
>>>>>>> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
>>>>>>> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
>>>>>>>  };
>>>>>>>  
>>>>>>>  enum npc_kpu_lc_ltype {
>>>>>>> -	NPC_LT_LC_IP = 1,
>>>>>>> +	NPC_LT_LC_PTP = 1,
>>>>>>> +	NPC_LT_LC_IP,
>>>>>>>  	NPC_LT_LC_IP_OPT,
>>>>>>>  	NPC_LT_LC_IP6,
>>>>>>>  	NPC_LT_LC_IP6_EXT,
>>>>>>> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
>>>>>>>  	NPC_LT_LC_RARP,
>>>>>>>  	NPC_LT_LC_MPLS,
>>>>>>>  	NPC_LT_LC_NSH,
>>>>>>> -	NPC_LT_LC_PTP,
>>>>>>>  	NPC_LT_LC_FCOE,
>>>>>>>  };
>>>>>>>  
>>>>>>>
>>>>>>
>>>>
>>


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

* Re: [dpdk-dev] [EXT] Re: [dpdk-stable] [PATCH] net/octeontx2: fix ptp functionality
  2020-02-17 16:17             ` Ferruh Yigit
@ 2020-02-17 16:37               ` Harman Kalra
  0 siblings, 0 replies; 10+ messages in thread
From: Harman Kalra @ 2020-02-17 16:37 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: Jerin Jacob, Vamsi Attunuru, dev, stable

On Mon, Feb 17, 2020 at 04:17:53PM +0000, Ferruh Yigit wrote:
> On 2/17/2020 3:30 PM, Harman Kalra wrote:
> > On Mon, Feb 17, 2020 at 03:19:53PM +0000, Ferruh Yigit wrote:
> >> On 2/17/2020 3:06 PM, Harman Kalra wrote:
> >>> On Mon, Feb 17, 2020 at 02:54:13PM +0000, Ferruh Yigit wrote:
> >>>> On 2/17/2020 2:35 PM, Harman Kalra wrote:
> >>>>> On Mon, Feb 17, 2020 at 01:21:13PM +0000, Ferruh Yigit wrote:
> >>>>>> External Email
> >>>>>>
> >>>>>> ----------------------------------------------------------------------
> >>>>>> On 2/15/2020 8:35 AM, Harman Kalra wrote:
> >>>>>>> PTP functionality has been broken after a change in kernel
> >>>>>>> where enum npc_kpu_lc_ltype is change to allow adjustment of
> >>>>>>> LTYPE_MASK to detect all types of IP headers.
> >>>>>>> Syncing the required changes in DPDK to fix the issue.
> >>>>>>
> >>>>>> Hi Harman,
> >>>>>>
> >>>>>> Can you please provide references for,
> >>>>>> - What is the kernel commit causing the mentioned change?
> >>>>>> - After this change which version of the kernel supported by the PMD?
> >>>>>
> >>>>> Hi Ferruh
> >>>>> The kernel change I was refering to was in our internel kernel repo
> >>>>> and those kernel changes have not been upstreamed yet. Currently its part
> >>>>> of our marvell SDK version SDK-10.3.1.x.
> >>>>
> >>>> Got it, what is the dependency of the PMD to the Linux kernel? And does the PMD
> >>>> works with upstream kernels?
> >>>
> >>> Yes, for basic functionality PMD works fine with upstreamed kernel but for
> >>> supporting some advanced features like PTP, kernel changes are yet to be
> >>> upstreamed. Our kernel driver for octeontx2 PMD:
> >>> https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_torvalds_linux_tree_master_drivers_net_ethernet_marvell_octeontx2_af&d=DwIDaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=5ESHPj7V-7JdkxT_Z_SU6RrS37ys4UXudBQ_rrS5LRo&m=pXArrQOu5hyDcQfkwc34h1CkBsAQuEnZDcanbaM-GCQ&s=c7WM8a2aLkuXH5P5HpQofM1nmCAXufFYva5YVyEiYcM&e= 
> >>> Kernel upstreaming activity is not in same pace as DPDK.
> >>
> >> Is the marvell SDK code in a public repo? If so can you provide the details of
> >> the kernel changes in that repo? It is good to have some reference on the matter.
> > 
> > Marvel SDK is not in any public repo. But for reference internal kernel viz part of
> > SDK version SDK-10.3.1.x with commit id "08705a860e2d3aebce44d6c357ed1c0ae71d3fd9"
> > onwards will have the required changes.
> 
> If it is not public, the commit id won't help much. Would you be OK if I add
> something like following while merging:
> 
> The kernel side changes are exists in marvell SDK version SDK-10.3.1.x.

I am sorry, one correction kernel side changes exists in SDK version SDK-10.3.2.x
and not in SDK-10.3.1.x
Please feel free to mention SDK version SDK-10.3.2.x

> 
> >>
> >>>
> >>> Thanks
> >>> Harman
> >>>>
> >>>>>
> >>>>> Thanks
> >>>>> Harman
> >>>>>
> >>>>>>
> >>>>>>>
> >>>>>>> Fixes: b5dc3140448e ("net/octeontx2: support base PTP")
> >>>>>>> Cc: stable@dpdk.org
> >>>>>>>
> >>>>>>> Signed-off-by: Harman Kalra <hkalra@marvell.com>
> >>>>>>> ---
> >>>>>>>  drivers/common/octeontx2/hw/otx2_npc.h | 4 ++--
> >>>>>>>  1 file changed, 2 insertions(+), 2 deletions(-)
> >>>>>>>
> >>>>>>> diff --git a/drivers/common/octeontx2/hw/otx2_npc.h b/drivers/common/octeontx2/hw/otx2_npc.h
> >>>>>>> index a0536e0ae..3dfc137a3 100644
> >>>>>>> --- a/drivers/common/octeontx2/hw/otx2_npc.h
> >>>>>>> +++ b/drivers/common/octeontx2/hw/otx2_npc.h
> >>>>>>> @@ -201,7 +201,8 @@ enum npc_kpu_lb_ltype {
> >>>>>>>  };
> >>>>>>>  
> >>>>>>>  enum npc_kpu_lc_ltype {
> >>>>>>> -	NPC_LT_LC_IP = 1,
> >>>>>>> +	NPC_LT_LC_PTP = 1,
> >>>>>>> +	NPC_LT_LC_IP,
> >>>>>>>  	NPC_LT_LC_IP_OPT,
> >>>>>>>  	NPC_LT_LC_IP6,
> >>>>>>>  	NPC_LT_LC_IP6_EXT,
> >>>>>>> @@ -209,7 +210,6 @@ enum npc_kpu_lc_ltype {
> >>>>>>>  	NPC_LT_LC_RARP,
> >>>>>>>  	NPC_LT_LC_MPLS,
> >>>>>>>  	NPC_LT_LC_NSH,
> >>>>>>> -	NPC_LT_LC_PTP,
> >>>>>>>  	NPC_LT_LC_FCOE,
> >>>>>>>  };
> >>>>>>>  
> >>>>>>>
> >>>>>>
> >>>>
> >>
> 

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

end of thread, back to index

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-02-15  8:35 [dpdk-dev] [PATCH] net/octeontx2: fix ptp functionality Harman Kalra
2020-02-15 14:12 ` Jerin Jacob
2020-02-17 13:21 ` [dpdk-dev] [dpdk-stable] " Ferruh Yigit
2020-02-17 14:35   ` [dpdk-dev] [EXT] " Harman Kalra
2020-02-17 14:54     ` Ferruh Yigit
2020-02-17 15:06       ` Harman Kalra
2020-02-17 15:19         ` Ferruh Yigit
2020-02-17 15:30           ` Harman Kalra
2020-02-17 16:17             ` Ferruh Yigit
2020-02-17 16:37               ` Harman Kalra

DPDK-dev Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/dpdk-dev/0 dpdk-dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dpdk-dev dpdk-dev/ https://lore.kernel.org/dpdk-dev \
		dev@dpdk.org
	public-inbox-index dpdk-dev

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git