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