DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
@ 2021-11-02 12:50 Andrew Rybchenko
  2021-11-02 13:47 ` Ferruh Yigit
  2021-11-05 16:03 ` Ferruh Yigit
  0 siblings, 2 replies; 8+ messages in thread
From: Andrew Rybchenko @ 2021-11-02 12:50 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, Jerin Jacob; +Cc: dev, David Marchand

The problem is solved using Rx metadata delivery negotiation API [1].

[1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")

Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
---
 doc/guides/rel_notes/deprecation.rst | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 4366015b01..ec5073908f 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -69,14 +69,6 @@ Deprecation Notices
   and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
   will be removed in DPDK 20.11.
 
-* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
-  This will allow application to enable or disable PMDs from updating
-  ``rte_mbuf::hash::fdir``.
-  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
-  thereby improve Rx performance if application wishes do so.
-  In 19.11 PMDs will still update the field even when the offload is not
-  enabled.
-
 * ethdev: Announce moving from dedicated modify function for each field,
   to using the general ``rte_flow_modify_field`` action.
 
-- 
2.30.2


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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-02 12:50 [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice Andrew Rybchenko
@ 2021-11-02 13:47 ` Ferruh Yigit
  2021-11-02 14:09   ` Andrew Rybchenko
  2021-11-05 16:03 ` Ferruh Yigit
  1 sibling, 1 reply; 8+ messages in thread
From: Ferruh Yigit @ 2021-11-02 13:47 UTC (permalink / raw)
  To: Andrew Rybchenko, Thomas Monjalon, Jerin Jacob; +Cc: dev, David Marchand

On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
> The problem is solved using Rx metadata delivery negotiation API [1].
> 

Can this API replace RX_OFFLOAD_RSS_HASH too?

> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")
> 
> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
> ---
>   doc/guides/rel_notes/deprecation.rst | 8 --------
>   1 file changed, 8 deletions(-)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 4366015b01..ec5073908f 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -69,14 +69,6 @@ Deprecation Notices
>     and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
>     will be removed in DPDK 20.11.
>   
> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
> -  This will allow application to enable or disable PMDs from updating
> -  ``rte_mbuf::hash::fdir``.
> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
> -  thereby improve Rx performance if application wishes do so.
> -  In 19.11 PMDs will still update the field even when the offload is not
> -  enabled.
> -
>   * ethdev: Announce moving from dedicated modify function for each field,
>     to using the general ``rte_flow_modify_field`` action.
>   
> 


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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-02 13:47 ` Ferruh Yigit
@ 2021-11-02 14:09   ` Andrew Rybchenko
  2021-11-02 14:34     ` Ferruh Yigit
  0 siblings, 1 reply; 8+ messages in thread
From: Andrew Rybchenko @ 2021-11-02 14:09 UTC (permalink / raw)
  To: Ferruh Yigit, Thomas Monjalon, Jerin Jacob
  Cc: dev, David Marchand, Ivan Malov

On 11/2/21 4:47 PM, Ferruh Yigit wrote:
> On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
>> The problem is solved using Rx metadata delivery negotiation API [1].
>>
> 
> Can this API replace RX_OFFLOAD_RSS_HASH too?

It can, but it should not. RSS hash is an offload since it is a
calculation of the hash in HW and delivery of the result to SW.
Like checksums: check in HW and delivery of the result to SW.

> 
>> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet 
>> metadata from HW to PMD")
>>
>> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
>> ---
>>   doc/guides/rel_notes/deprecation.rst | 8 --------
>>   1 file changed, 8 deletions(-)
>>
>> diff --git a/doc/guides/rel_notes/deprecation.rst 
>> b/doc/guides/rel_notes/deprecation.rst
>> index 4366015b01..ec5073908f 100644
>> --- a/doc/guides/rel_notes/deprecation.rst
>> +++ b/doc/guides/rel_notes/deprecation.rst
>> @@ -69,14 +69,6 @@ Deprecation Notices
>>     and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
>>     will be removed in DPDK 20.11.
>> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be 
>> added in 19.11.
>> -  This will allow application to enable or disable PMDs from updating
>> -  ``rte_mbuf::hash::fdir``.
>> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields 
>> on Rx and
>> -  thereby improve Rx performance if application wishes do so.
>> -  In 19.11 PMDs will still update the field even when the offload is not
>> -  enabled.
>> -
>>   * ethdev: Announce moving from dedicated modify function for each 
>> field,
>>     to using the general ``rte_flow_modify_field`` action.
>>


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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-02 14:09   ` Andrew Rybchenko
@ 2021-11-02 14:34     ` Ferruh Yigit
  0 siblings, 0 replies; 8+ messages in thread
From: Ferruh Yigit @ 2021-11-02 14:34 UTC (permalink / raw)
  To: Andrew Rybchenko, Thomas Monjalon, Jerin Jacob
  Cc: dev, David Marchand, Ivan Malov

On 11/2/2021 2:09 PM, Andrew Rybchenko wrote:
> On 11/2/21 4:47 PM, Ferruh Yigit wrote:
>> On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
>>> The problem is solved using Rx metadata delivery negotiation API [1].
>>>
>>
>> Can this API replace RX_OFFLOAD_RSS_HASH too?
> 
> It can, but it should not. RSS hash is an offload since it is a
> calculation of the hash in HW and delivery of the result to SW.
> Like checksums: check in HW and delivery of the result to SW.
> 

There is a slight difference, RSS hash is always calculated if RSS is enabled,
independent from RSS offload enabled or not.

That is why some PMDs always provides RSS hash and force enables this offload,
I would be happy to get rid of this quirk if possible.

>>
>>> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")
>>>
>>> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
>>> ---
>>>   doc/guides/rel_notes/deprecation.rst | 8 --------
>>>   1 file changed, 8 deletions(-)
>>>
>>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
>>> index 4366015b01..ec5073908f 100644
>>> --- a/doc/guides/rel_notes/deprecation.rst
>>> +++ b/doc/guides/rel_notes/deprecation.rst
>>> @@ -69,14 +69,6 @@ Deprecation Notices
>>>     and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
>>>     will be removed in DPDK 20.11.
>>> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
>>> -  This will allow application to enable or disable PMDs from updating
>>> -  ``rte_mbuf::hash::fdir``.
>>> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
>>> -  thereby improve Rx performance if application wishes do so.
>>> -  In 19.11 PMDs will still update the field even when the offload is not
>>> -  enabled.
>>> -
>>>   * ethdev: Announce moving from dedicated modify function for each field,
>>>     to using the general ``rte_flow_modify_field`` action.
>>>
> 


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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-02 12:50 [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice Andrew Rybchenko
  2021-11-02 13:47 ` Ferruh Yigit
@ 2021-11-05 16:03 ` Ferruh Yigit
  2021-11-19 17:13   ` Ferruh Yigit
  1 sibling, 1 reply; 8+ messages in thread
From: Ferruh Yigit @ 2021-11-05 16:03 UTC (permalink / raw)
  To: Andrew Rybchenko, Jerin Jacob, Pavan Nikhilesh, Hemant Agrawal,
	Konstantin Ananyev
  Cc: dev, David Marchand, Thomas Monjalon

On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
> The problem is solved using Rx metadata delivery negotiation API [1].
> 
> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")
> 
> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
> ---
>   doc/guides/rel_notes/deprecation.rst | 8 --------
>   1 file changed, 8 deletions(-)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> index 4366015b01..ec5073908f 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -69,14 +69,6 @@ Deprecation Notices
>     and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
>     will be removed in DPDK 20.11.
>   
> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
> -  This will allow application to enable or disable PMDs from updating
> -  ``rte_mbuf::hash::fdir``.
> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
> -  thereby improve Rx performance if application wishes do so.
> -  In 19.11 PMDs will still update the field even when the offload is not
> -  enabled.
> -
>   * ethdev: Announce moving from dedicated modify function for each field,
>     to using the general ``rte_flow_modify_field`` action.
>   
> 

+ Pavan, who added the deprecation notice at first place, plus people who ack'ed
the patch are cc'ed.

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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-05 16:03 ` Ferruh Yigit
@ 2021-11-19 17:13   ` Ferruh Yigit
  2021-11-22  4:10     ` Jerin Jacob
  0 siblings, 1 reply; 8+ messages in thread
From: Ferruh Yigit @ 2021-11-19 17:13 UTC (permalink / raw)
  To: Andrew Rybchenko, Jerin Jacob, Pavan Nikhilesh, Hemant Agrawal,
	Konstantin Ananyev
  Cc: dev, David Marchand, Thomas Monjalon

On 11/5/2021 4:03 PM, Ferruh Yigit wrote:
> On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
>> The problem is solved using Rx metadata delivery negotiation API [1].
>>
>> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")
>>
>> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
>> ---
>>   doc/guides/rel_notes/deprecation.rst | 8 --------
>>   1 file changed, 8 deletions(-)
>>
>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
>> index 4366015b01..ec5073908f 100644
>> --- a/doc/guides/rel_notes/deprecation.rst
>> +++ b/doc/guides/rel_notes/deprecation.rst
>> @@ -69,14 +69,6 @@ Deprecation Notices
>>     and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
>>     will be removed in DPDK 20.11.
>> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
>> -  This will allow application to enable or disable PMDs from updating
>> -  ``rte_mbuf::hash::fdir``.
>> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
>> -  thereby improve Rx performance if application wishes do so.
>> -  In 19.11 PMDs will still update the field even when the offload is not
>> -  enabled.
>> -
>>   * ethdev: Announce moving from dedicated modify function for each field,
>>     to using the general ``rte_flow_modify_field`` action.
>>
> 
> + Pavan, who added the deprecation notice at first place, plus people who ack'ed
> the patch are cc'ed.

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Pavan, Jerin, Hemant, Konstantin, any objection to remove the deprecation notice?

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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-19 17:13   ` Ferruh Yigit
@ 2021-11-22  4:10     ` Jerin Jacob
  2021-11-24 12:17       ` Ferruh Yigit
  0 siblings, 1 reply; 8+ messages in thread
From: Jerin Jacob @ 2021-11-22  4:10 UTC (permalink / raw)
  To: Ferruh Yigit
  Cc: Andrew Rybchenko, Jerin Jacob, Pavan Nikhilesh, Hemant Agrawal,
	Konstantin Ananyev, dpdk-dev, David Marchand, Thomas Monjalon

On Fri, Nov 19, 2021 at 11:23 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>
> On 11/5/2021 4:03 PM, Ferruh Yigit wrote:
> > On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
> >> The problem is solved using Rx metadata delivery negotiation API [1].
> >>
> >> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")
> >>
> >> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
> >> ---
> >>   doc/guides/rel_notes/deprecation.rst | 8 --------
> >>   1 file changed, 8 deletions(-)
> >>
> >> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
> >> index 4366015b01..ec5073908f 100644
> >> --- a/doc/guides/rel_notes/deprecation.rst
> >> +++ b/doc/guides/rel_notes/deprecation.rst
> >> @@ -69,14 +69,6 @@ Deprecation Notices
> >>     and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
> >>     will be removed in DPDK 20.11.
> >> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
> >> -  This will allow application to enable or disable PMDs from updating
> >> -  ``rte_mbuf::hash::fdir``.
> >> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
> >> -  thereby improve Rx performance if application wishes do so.
> >> -  In 19.11 PMDs will still update the field even when the offload is not
> >> -  enabled.
> >> -
> >>   * ethdev: Announce moving from dedicated modify function for each field,
> >>     to using the general ``rte_flow_modify_field`` action.
> >>
> >
> > + Pavan, who added the deprecation notice at first place, plus people who ack'ed
> > the patch are cc'ed.
>
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
>
> Pavan, Jerin, Hemant, Konstantin, any objection to remove the deprecation notice?

Acked-by: Jerin Jacob <jerinj@marvell.com>

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

* Re: [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice
  2021-11-22  4:10     ` Jerin Jacob
@ 2021-11-24 12:17       ` Ferruh Yigit
  0 siblings, 0 replies; 8+ messages in thread
From: Ferruh Yigit @ 2021-11-24 12:17 UTC (permalink / raw)
  To: Andrew Rybchenko
  Cc: Jerin Jacob, Pavan Nikhilesh, Hemant Agrawal, Konstantin Ananyev,
	dpdk-dev, David Marchand, Thomas Monjalon, Jerin Jacob

On 11/22/2021 4:10 AM, Jerin Jacob wrote:
> On Fri, Nov 19, 2021 at 11:23 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
>>
>> On 11/5/2021 4:03 PM, Ferruh Yigit wrote:
>>> On 11/2/2021 12:50 PM, Andrew Rybchenko wrote:
>>>> The problem is solved using Rx metadata delivery negotiation API [1].
>>>>
>>>> [1] commit f6d8a6d3fad7 ("ethdev: negotiate delivery of packet metadata from HW to PMD")
>>>>
>>>> Signed-off-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>
>>>> ---
>>>>    doc/guides/rel_notes/deprecation.rst | 8 --------
>>>>    1 file changed, 8 deletions(-)
>>>>
>>>> diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
>>>> index 4366015b01..ec5073908f 100644
>>>> --- a/doc/guides/rel_notes/deprecation.rst
>>>> +++ b/doc/guides/rel_notes/deprecation.rst
>>>> @@ -69,14 +69,6 @@ Deprecation Notices
>>>>      and the related structures (``rte_fdir_*`` and ``rte_eth_fdir_*``),
>>>>      will be removed in DPDK 20.11.
>>>> -* ethdev: New offload flags ``RTE_ETH_RX_OFFLOAD_FLOW_MARK`` will be added in 19.11.
>>>> -  This will allow application to enable or disable PMDs from updating
>>>> -  ``rte_mbuf::hash::fdir``.
>>>> -  This scheme will allow PMDs to avoid writes to ``rte_mbuf`` fields on Rx and
>>>> -  thereby improve Rx performance if application wishes do so.
>>>> -  In 19.11 PMDs will still update the field even when the offload is not
>>>> -  enabled.
>>>> -
>>>>    * ethdev: Announce moving from dedicated modify function for each field,
>>>>      to using the general ``rte_flow_modify_field`` action.
>>>>
>>>
>>> + Pavan, who added the deprecation notice at first place, plus people who ack'ed
>>> the patch are cc'ed.
>>
>> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>
>>
>> Pavan, Jerin, Hemant, Konstantin, any objection to remove the deprecation notice?
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>
> 

There is no objection, so proceeding with the patch, thanks for cleanup.

Applied to dpdk-next-net/main, thanks.

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

end of thread, other threads:[~2021-11-24 12:17 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-02 12:50 [dpdk-dev] [PATCH] doc: cleanup flow mark Rx offload deprecation notice Andrew Rybchenko
2021-11-02 13:47 ` Ferruh Yigit
2021-11-02 14:09   ` Andrew Rybchenko
2021-11-02 14:34     ` Ferruh Yigit
2021-11-05 16:03 ` Ferruh Yigit
2021-11-19 17:13   ` Ferruh Yigit
2021-11-22  4:10     ` Jerin Jacob
2021-11-24 12:17       ` 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).