doc: announce deprecation of dpaa2 exposed mem structure

Message ID 20181115132841.4444-1-shreyansh.jain@nxp.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series doc: announce deprecation of dpaa2 exposed mem structure |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/Intel-compilation success Compilation OK
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/intel-Performance-Testing success Performance Testing PASS

Commit Message

Shreyansh Jain Nov. 15, 2018, 1:29 p.m. UTC
  rte_dpaa2_memsegs is no more required once the dpaax (pa-va) translation
library has been introduced. This can be made internal (for fallback
operations) in subsequent release.

Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
---
 This diff was generated over a previous patch on same file (a para
 before) [1] - so, it might not apply in case [1] is missed.
 It is possible to merge this and [1] as well, if that makes better sense.

 [1] http://patches.dpdk.org/patch/48118/

 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)
  

Comments

Hemant Agrawal Nov. 21, 2018, 8:50 a.m. UTC | #1
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
  
Burakov, Anatoly Nov. 23, 2018, 12:46 p.m. UTC | #2
On 15-Nov-18 1:29 PM, Shreyansh Jain wrote:
> rte_dpaa2_memsegs is no more required once the dpaax (pa-va) translation
> library has been introduced. This can be made internal (for fallback
> operations) in subsequent release.
> 
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> ---
>   This diff was generated over a previous patch on same file (a para
>   before) [1] - so, it might not apply in case [1] is missed.
>   It is possible to merge this and [1] as well, if that makes better sense.
> 
>   [1] http://patches.dpdk.org/patch/48118/
> 
>   doc/guides/rel_notes/deprecation.rst | 6 ++++++
>   1 file changed, 6 insertions(+)
> 

FWIW

Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
  
Maxime Coquelin Nov. 23, 2018, 1:51 p.m. UTC | #3
On 11/15/18 2:29 PM, Shreyansh Jain wrote:
> rte_dpaa2_memsegs is no more required once the dpaax (pa-va) translation
> library has been introduced. This can be made internal (for fallback
> operations) in subsequent release.
> 
> Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> ---
>   This diff was generated over a previous patch on same file (a para
>   before) [1] - so, it might not apply in case [1] is missed.
>   It is possible to merge this and [1] as well, if that makes better sense.
> 
>   [1] http://patches.dpdk.org/patch/48118/
> 
>   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 776b9b245..a2c0d21bc 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -60,3 +60,9 @@ Deprecation Notices
>     Currently this is exposed from the library. Ideally, no application should
>     be directly using it. This is expected to be deprecated and made internal
>     in v19.02 or later.
> +
> +* dpaa2: removal of ``rte_dpaa2_memsegs`` structure which has been replaced
> +  by a pa-va search library. This structure was earlier being used for holding
> +  memory segments used by dpaa2 driver for faster pa->va translation. This
> +  structure would be made internal (or removed if all dependencies are cleared)
> +  in future releases.
> 

Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
  
Jerin Jacob Nov. 23, 2018, 4:34 p.m. UTC | #4
-----Original Message-----
> Date: Fri, 23 Nov 2018 12:46:42 +0000
> From: "Burakov, Anatoly" <anatoly.burakov@intel.com>
> To: Shreyansh Jain <shreyansh.jain@nxp.com>, "dev@dpdk.org" <dev@dpdk.org>
> Subject: Re: [dpdk-dev] [PATCH] doc: announce deprecation of dpaa2 exposed
>  mem structure
> User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101
>  Thunderbird/52.9.1
> 
> External Email
> 
> On 15-Nov-18 1:29 PM, Shreyansh Jain wrote:
> > rte_dpaa2_memsegs is no more required once the dpaax (pa-va) translation
> > library has been introduced. This can be made internal (for fallback
> > operations) in subsequent release.
> > 
> > Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> > ---
> >   This diff was generated over a previous patch on same file (a para
> >   before) [1] - so, it might not apply in case [1] is missed.
> >   It is possible to merge this and [1] as well, if that makes better sense.
> > 
> >   [1] http://patches.dpdk.org/patch/48118/
> > 
> >   doc/guides/rel_notes/deprecation.rst | 6 ++++++
> >   1 file changed, 6 insertions(+)
> > 
> 
> FWIW
> 
> Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>

Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>
  
Thomas Monjalon Nov. 24, 2018, 5:25 p.m. UTC | #5
23/11/2018 17:34, Jerin Jacob:
> From: "Burakov, Anatoly" <anatoly.burakov@intel.com>
> > On 15-Nov-18 1:29 PM, Shreyansh Jain wrote:
> > > rte_dpaa2_memsegs is no more required once the dpaax (pa-va) translation
> > > library has been introduced. This can be made internal (for fallback
> > > operations) in subsequent release.
> > > 
> > > Signed-off-by: Shreyansh Jain <shreyansh.jain@nxp.com>
> > > ---
> > >   This diff was generated over a previous patch on same file (a para
> > >   before) [1] - so, it might not apply in case [1] is missed.
> > >   It is possible to merge this and [1] as well, if that makes better sense.
> > > 
> > >   [1] http://patches.dpdk.org/patch/48118/
> > > 
> > >   doc/guides/rel_notes/deprecation.rst | 6 ++++++
> > >   1 file changed, 6 insertions(+)
> > > 
> > 
> > FWIW
> > 
> > Acked-by: Anatoly Burakov <anatoly.burakov@intel.com>
> 
> Acked-by: Jerin Jacob <jerin.jacob@caviumnetworks.com>

Applied
  

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 776b9b245..a2c0d21bc 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -60,3 +60,9 @@  Deprecation Notices
   Currently this is exposed from the library. Ideally, no application should
   be directly using it. This is expected to be deprecated and made internal
   in v19.02 or later.
+
+* dpaa2: removal of ``rte_dpaa2_memsegs`` structure which has been replaced
+  by a pa-va search library. This structure was earlier being used for holding
+  memory segments used by dpaa2 driver for faster pa->va translation. This
+  structure would be made internal (or removed if all dependencies are cleared)
+  in future releases.