[v5,12/12] raw/ioat: deprecate ioat rawdev driver

Message ID 20210924143335.1092300-13-conor.walsh@intel.com (mailing list archive)
State Superseded, archived
Headers
Series dma: add dmadev driver for ioat devices |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation warning apply issues

Commit Message

Conor Walsh Sept. 24, 2021, 2:33 p.m. UTC
  Deprecate the rawdev IOAT driver as both IOAT and IDXD drivers have
moved to dmadev.

Signed-off-by: Conor Walsh <conor.walsh@intel.com>
---
 MAINTAINERS                 | 2 +-
 doc/guides/rawdevs/ioat.rst | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)
  

Comments

Kevin Laatz Sept. 24, 2021, 4:21 p.m. UTC | #1
On 24/09/2021 15:33, Conor Walsh wrote:
> Deprecate the rawdev IOAT driver as both IOAT and IDXD drivers have
> moved to dmadev.
>
> Signed-off-by: Conor Walsh <conor.walsh@intel.com>
> ---
>   MAINTAINERS                 | 2 +-
>   doc/guides/rawdevs/ioat.rst | 4 ++++
>   2 files changed, 5 insertions(+), 1 deletion(-)

Acked-by: Kevin Laatz <kevin.laatz@intel.com>
  
Bruce Richardson Sept. 24, 2021, 4:56 p.m. UTC | #2
On Fri, Sep 24, 2021 at 02:33:35PM +0000, Conor Walsh wrote:
> Deprecate the rawdev IOAT driver as both IOAT and IDXD drivers have
> moved to dmadev.
> 
> Signed-off-by: Conor Walsh <conor.walsh@intel.com>
> ---

This probably needs a release note entry for it too.
With such a RN addition, feel free to add my acked-by.

>  MAINTAINERS                 | 2 +-
>  doc/guides/rawdevs/ioat.rst | 4 ++++
>  2 files changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ccabba9169..a4bcd2d024 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1322,7 +1322,7 @@ T: git://dpdk.org/next/dpdk-next-net-intel
>  F: drivers/raw/ifpga/
>  F: doc/guides/rawdevs/ifpga.rst
>  
> -IOAT Rawdev
> +IOAT Rawdev - DEPRECATED
>  M: Bruce Richardson <bruce.richardson@intel.com>
>  F: drivers/raw/ioat/
>  F: doc/guides/rawdevs/ioat.rst
> diff --git a/doc/guides/rawdevs/ioat.rst b/doc/guides/rawdevs/ioat.rst
> index a65530bd30..98d15dd032 100644
> --- a/doc/guides/rawdevs/ioat.rst
> +++ b/doc/guides/rawdevs/ioat.rst
> @@ -6,6 +6,10 @@
>  IOAT Rawdev Driver
>  ===================
>  
> +.. warning::
> +        As of DPDK 21.11 the rawdev implementation of the IOAT driver has been deprecated.
> +        Please use the dmadev library instead.
> +
>  The ``ioat`` rawdev driver provides a poll-mode driver (PMD) for Intel\ |reg|
>  Data Streaming Accelerator `(Intel DSA)
>  <https://01.org/blogs/2019/introducing-intel-data-streaming-accelerator>`_ and for Intel\ |reg|
> -- 
> 2.25.1
>
  
Conor Walsh Sept. 27, 2021, 9:36 a.m. UTC | #3
> From: Richardson, Bruce <bruce.richardson@intel.com>
> Sent: Friday 24 September 2021 17:57
> To: Walsh, Conor <conor.walsh@intel.com>
> Cc: fengchengwen@huawei.com; jerinj@marvell.com; Laatz, Kevin
> <kevin.laatz@intel.com>; dev@dpdk.org
> Subject: Re: [PATCH v5 12/12] raw/ioat: deprecate ioat rawdev driver
> 
> On Fri, Sep 24, 2021 at 02:33:35PM +0000, Conor Walsh wrote:
> > Deprecate the rawdev IOAT driver as both IOAT and IDXD drivers have
> > moved to dmadev.
> >
> > Signed-off-by: Conor Walsh <conor.walsh@intel.com>
> > ---
> 
> This probably needs a release note entry for it too.
> With such a RN addition, feel free to add my acked-by.

Thanks Bruce, I will add a deprecation notice in v6.
/Conor.

> 
> >  MAINTAINERS                 | 2 +-
> >  doc/guides/rawdevs/ioat.rst | 4 ++++
> >  2 files changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index ccabba9169..a4bcd2d024 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1322,7 +1322,7 @@ T: git://dpdk.org/next/dpdk-next-net-intel
> >  F: drivers/raw/ifpga/
> >  F: doc/guides/rawdevs/ifpga.rst
> >
> > -IOAT Rawdev
> > +IOAT Rawdev - DEPRECATED
> >  M: Bruce Richardson <bruce.richardson@intel.com>
> >  F: drivers/raw/ioat/
> >  F: doc/guides/rawdevs/ioat.rst
> > diff --git a/doc/guides/rawdevs/ioat.rst b/doc/guides/rawdevs/ioat.rst
> > index a65530bd30..98d15dd032 100644
> > --- a/doc/guides/rawdevs/ioat.rst
> > +++ b/doc/guides/rawdevs/ioat.rst
> > @@ -6,6 +6,10 @@
> >  IOAT Rawdev Driver
> >  ===================
> >
> > +.. warning::
> > +        As of DPDK 21.11 the rawdev implementation of the IOAT driver has
> been deprecated.
> > +        Please use the dmadev library instead.
> > +
> >  The ``ioat`` rawdev driver provides a poll-mode driver (PMD) for Intel\
> |reg|
> >  Data Streaming Accelerator `(Intel DSA)
> >  <https://01.org/blogs/2019/introducing-intel-data-streaming-
> accelerator>`_ and for Intel\ |reg|
> > --
> > 2.25.1
> >
  

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index ccabba9169..a4bcd2d024 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1322,7 +1322,7 @@  T: git://dpdk.org/next/dpdk-next-net-intel
 F: drivers/raw/ifpga/
 F: doc/guides/rawdevs/ifpga.rst
 
-IOAT Rawdev
+IOAT Rawdev - DEPRECATED
 M: Bruce Richardson <bruce.richardson@intel.com>
 F: drivers/raw/ioat/
 F: doc/guides/rawdevs/ioat.rst
diff --git a/doc/guides/rawdevs/ioat.rst b/doc/guides/rawdevs/ioat.rst
index a65530bd30..98d15dd032 100644
--- a/doc/guides/rawdevs/ioat.rst
+++ b/doc/guides/rawdevs/ioat.rst
@@ -6,6 +6,10 @@ 
 IOAT Rawdev Driver
 ===================
 
+.. warning::
+        As of DPDK 21.11 the rawdev implementation of the IOAT driver has been deprecated.
+        Please use the dmadev library instead.
+
 The ``ioat`` rawdev driver provides a poll-mode driver (PMD) for Intel\ |reg|
 Data Streaming Accelerator `(Intel DSA)
 <https://01.org/blogs/2019/introducing-intel-data-streaming-accelerator>`_ and for Intel\ |reg|