doc: announce change in crypto adapter queue add

Message ID 20220628120927.1645242-1-vfialko@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series doc: announce change in crypto adapter queue add |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-intel-Performance success Performance Testing PASS
ci/iol-intel-Functional success Functional Testing PASS
ci/iol-aarch64-unit-testing success Testing PASS
ci/github-robot: build success github build: passed
ci/iol-aarch64-compile-testing success Testing PASS
ci/iol-x86_64-compile-testing success Testing PASS
ci/iol-x86_64-unit-testing success Testing PASS
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS
ci/iol-abi-testing success Testing PASS

Commit Message

Volodymyr Fialko June 28, 2022, 12:09 p.m. UTC
  The function `rte_event_crypto_adapter_queue_pair_add` will accept
`rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.

Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
---
 doc/guides/rel_notes/deprecation.rst | 6 ++++++
 1 file changed, 6 insertions(+)
  

Comments

Akhil Goyal June 28, 2022, 12:40 p.m. UTC | #1
> Subject: [PATCH] doc: announce change in crypto adapter queue add
> 
> The function `rte_event_crypto_adapter_queue_pair_add` will accept
> `rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.
> 
> Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
Acked-by: Akhil Goyal <gakhil@marvell.com>
  
Jerin Jacob July 11, 2022, 2:56 p.m. UTC | #2
On Tue, Jun 28, 2022 at 6:12 PM Akhil Goyal <gakhil@marvell.com> wrote:
>
> > Subject: [PATCH] doc: announce change in crypto adapter queue add
> >
> > The function `rte_event_crypto_adapter_queue_pair_add` will accept
> > `rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.
> >
> > Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
> Acked-by: Akhil Goyal <gakhil@marvell.com>

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


>
  
Akhil Goyal July 12, 2022, 5:31 a.m. UTC | #3
> On Tue, Jun 28, 2022 at 6:12 PM Akhil Goyal <gakhil@marvell.com> wrote:
> >
> > > Subject: [PATCH] doc: announce change in crypto adapter queue add
> > >
> > > The function `rte_event_crypto_adapter_queue_pair_add` will accept
> > > `rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.
> > >
> > > Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
> > Acked-by: Akhil Goyal <gakhil@marvell.com>
> 
> Acked-by: Jerin Jacob <jerinj@marvell.com>
Hi Abhinandan,

Could you please Ack this deprecation notice.

Regards,
Akhil
  
Gujjar, Abhinandan S July 13, 2022, 6:49 a.m. UTC | #4
> -----Original Message-----
> From: Akhil Goyal <gakhil@marvell.com>
> Sent: Tuesday, July 12, 2022 11:02 AM
> To: Jerin Jacob <jerinjacobk@gmail.com>; Gujjar, Abhinandan S
> <abhinandan.gujjar@intel.com>
> Cc: Volodymyr Fialko <vfialko@marvell.com>; dev@dpdk.org; Ray Kinsella
> <mdr@ashroe.eu>; Jerin Jacob Kollanukkaran <jerinj@marvell.com>; Anoob
> Joseph <anoobj@marvell.com>
> Subject: RE: [EXT] Re: [PATCH] doc: announce change in crypto adapter queue
> add
> 
> > On Tue, Jun 28, 2022 at 6:12 PM Akhil Goyal <gakhil@marvell.com> wrote:
> > >
> > > > Subject: [PATCH] doc: announce change in crypto adapter queue add
> > > >
> > > > The function `rte_event_crypto_adapter_queue_pair_add` will accept
> > > > `rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.
> > > >
> > > > Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
> > > Acked-by: Akhil Goyal <gakhil@marvell.com>
> >
> > Acked-by: Jerin Jacob <jerinj@marvell.com>
Acked-by: Abhinandan Gujjar <abhinandan.gujjar@intel.com>
> Hi Abhinandan,
> 
> Could you please Ack this deprecation notice.
> 
> Regards,
> Akhil
  
Hemant Agrawal July 14, 2022, 9:04 a.m. UTC | #5
On 6/28/2022 5:39 PM, Volodymyr Fialko wrote:
> The function `rte_event_crypto_adapter_queue_pair_add` will accept
> `rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.
>
> Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.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 4e5b23c53d..63a6459f17 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -125,3 +125,9 @@ Deprecation Notices
>     applications should be updated to use the ``dmadev`` library instead,
>     with the underlying HW-functionality being provided by the ``ioat`` or
>     ``idxd`` dma drivers
> +
> +* eventdev: The function ``rte_event_crypto_adapter_queue_pair_add`` will
> +  accept configuration of type ``rte_event_crypto_adapter_queue_conf`` instead
> +  of ``rte_event``, similar to ``rte_event_eth_rx_adapter_queue_add`` signature.
> +  Event will be one of the configuration fields, together with additional
> +  vector parameters.
  
Thomas Monjalon July 17, 2022, 11:32 a.m. UTC | #6
28/06/2022 14:09, Volodymyr Fialko:
> The function `rte_event_crypto_adapter_queue_pair_add` will accept
> `rte_event_crypto_adapter_queue_conf` argument instead of `rte_event`.
> 
> Signed-off-by: Volodymyr Fialko <vfialko@marvell.com>

    Acked-by: Akhil Goyal <gakhil@marvell.com>
    Acked-by: Jerin Jacob <jerinj@marvell.com>
    Acked-by: Abhinandan Gujjar <abhinandan.gujjar@intel.com>
    Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

Applied, thanks.
  

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 4e5b23c53d..63a6459f17 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -125,3 +125,9 @@  Deprecation Notices
   applications should be updated to use the ``dmadev`` library instead,
   with the underlying HW-functionality being provided by the ``ioat`` or
   ``idxd`` dma drivers
+
+* eventdev: The function ``rte_event_crypto_adapter_queue_pair_add`` will
+  accept configuration of type ``rte_event_crypto_adapter_queue_conf`` instead
+  of ``rte_event``, similar to ``rte_event_eth_rx_adapter_queue_add`` signature.
+  Event will be one of the configuration fields, together with additional
+  vector parameters.