doc: announce removal of L2 tunnel filtering API

Message ID 20200803180627.3292763-1-thomas@monjalon.net (mailing list archive)
State Accepted, archived
Headers
Series doc: announce removal of L2 tunnel filtering API |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/travis-robot success Travis build: passed

Commit Message

Thomas Monjalon Aug. 3, 2020, 6:06 p.m. UTC
  The functions for L2 tunnel were missed when marking the legacy
filtering API as deprecated. That's why a separate notice is done
to make clear that it will be removed as well in DPDK 20.11.

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)
  

Comments

Ajit Khaparde Aug. 4, 2020, 3:54 a.m. UTC | #1
On Mon, Aug 3, 2020 at 11:06 AM Thomas Monjalon <thomas@monjalon.net> wrote:

> The functions for L2 tunnel were missed when marking the legacy
> filtering API as deprecated. That's why a separate notice is done
> to make clear that it will be removed as well in DPDK 20.11.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>
Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>


> ---
>  doc/guides/rel_notes/deprecation.rst | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index ea4cfa7a48..cad2f777f2 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -84,6 +84,12 @@ Deprecation Notices
>    Target release for removal of the legacy API will be defined once most
>    PMDs have switched to rte_flow.
>
> +* ethdev: the legacy L2 tunnel filtering API is deprecated as the rest of
> +  the legacy filtering API.
> +  The functions ``rte_eth_dev_l2_tunnel_eth_type_conf`` and
> +  ``rte_eth_dev_l2_tunnel_offload_set`` which were not marked as
> deprecated,
> +  will be removed in DPDK 20.11.
> +
>  * ethdev: Update API functions returning ``void`` to return ``int`` with
>    negative errno values to indicate various error conditions (e.g.
>    invalid port ID, unsupported operation, failed operation):
> --
> 2.27.0
>
>
  
David Marchand Aug. 4, 2020, 7:51 a.m. UTC | #2
On Mon, Aug 3, 2020 at 8:06 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> The functions for L2 tunnel were missed when marking the legacy
> filtering API as deprecated. That's why a separate notice is done
> to make clear that it will be removed as well in DPDK 20.11.
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>

Acked-by: David Marchand <david.marchand@redhat.com>
  
Jerin Jacob Aug. 4, 2020, 8:54 a.m. UTC | #3
On Tue, Aug 4, 2020 at 1:21 PM David Marchand <david.marchand@redhat.com> wrote:
>
> On Mon, Aug 3, 2020 at 8:06 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> >
> > The functions for L2 tunnel were missed when marking the legacy
> > filtering API as deprecated. That's why a separate notice is done
> > to make clear that it will be removed as well in DPDK 20.11.
> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>
> Acked-by: David Marchand <david.marchand@redhat.com>

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



>
>
> --
> David Marchand
>
  
Andrew Rybchenko Aug. 5, 2020, 8:45 a.m. UTC | #4
On 8/4/20 11:54 AM, Jerin Jacob wrote:
> On Tue, Aug 4, 2020 at 1:21 PM David Marchand <david.marchand@redhat.com> wrote:
>>
>> On Mon, Aug 3, 2020 at 8:06 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>>>
>>> The functions for L2 tunnel were missed when marking the legacy
>>> filtering API as deprecated. That's why a separate notice is done
>>> to make clear that it will be removed as well in DPDK 20.11.
>>>
>>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
>>
>> Acked-by: David Marchand <david.marchand@redhat.com>
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>

Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>
  
Thomas Monjalon Aug. 6, 2020, 12:37 a.m. UTC | #5
> >>> The functions for L2 tunnel were missed when marking the legacy
> >>> filtering API as deprecated. That's why a separate notice is done
> >>> to make clear that it will be removed as well in DPDK 20.11.
> >>>
> >>> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> >> Acked-by: Ajit Khaparde <ajit.khaparde@broadcom.com>
> >> Acked-by: David Marchand <david.marchand@redhat.com>
> > Acked-by: Jerin Jacob <jerinj@marvell.com>
> Acked-by: Andrew Rybchenko <arybchenko@solarflare.com>

Applied
  

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index ea4cfa7a48..cad2f777f2 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -84,6 +84,12 @@  Deprecation Notices
   Target release for removal of the legacy API will be defined once most
   PMDs have switched to rte_flow.
 
+* ethdev: the legacy L2 tunnel filtering API is deprecated as the rest of
+  the legacy filtering API.
+  The functions ``rte_eth_dev_l2_tunnel_eth_type_conf`` and
+  ``rte_eth_dev_l2_tunnel_offload_set`` which were not marked as deprecated,
+  will be removed in DPDK 20.11.
+
 * ethdev: Update API functions returning ``void`` to return ``int`` with
   negative errno values to indicate various error conditions (e.g.
   invalid port ID, unsupported operation, failed operation):