diff mbox series

doc: announce SA config option struct changes

Message ID 20210731174428.626705-1-gakhil@marvell.com (mailing list archive)
State Accepted, archived
Headers
Series doc: announce SA config option struct changes |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/iol-testing warning apply patch failure
ci/github-robot success github build: passed
ci/Intel-compilation success Compilation OK
ci/intel-Testing success Testing PASS

Commit Message

Akhil Goyal July 31, 2021, 5:44 p.m. UTC 
  From: Archana Muniganti <marchana@marvell.com>

Propose new fields to support offloads like
- IPsec inner checksum(L3/L4)
- IPsec tunnel header verification
- TSO
- etc
in the structure ``rte_security_ipsec_sa_options``.

Signed-off-by: Archana Muniganti <marchana@marvell.com>
Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
Acked-by: Akhil Goyal <gakhil@marvell.com>
---
Proposal for changes are floated as per below links.
https://mails.dpdk.org/archives/dev/2021-June/212977.html
https://mails.dpdk.org/archives/dev/2021-July/213484.html
https://mails.dpdk.org/archives/dev/2021-July/214521.html

Cc: Radu Nicolau <radu.nicolau@intel.com>

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

Comments

Anoob Joseph Aug. 2, 2021, 9:43 a.m. UTC | #1 
Hi Akhil,

There are 2 more items that I would like to add.
- IV gen (in ``rte_security_ipsec_sa_options``)
- SA lifetime configuration (in ``rte_security_ipsec_xform``)

http://patches.dpdk.org/project/dpdk/patch/1626759974-334-2-git-send-email-anoobj@marvell.com/
http://patches.dpdk.org/project/dpdk/patch/1626759974-334-3-git-send-email-anoobj@marvell.com/

Do you recommend sending separate deprecation notice or can this patch be updated to cover those as well?

Thanks,
Anoob

> -----Original Message-----
> From: Akhil Goyal <gakhil@marvell.com>
> Sent: Saturday, July 31, 2021 11:14 PM
> To: dev@dpdk.org
> Cc: Anoob Joseph <anoobj@marvell.com>; radu.nicolau@intel.com;
> declan.doherty@intel.com; abhijits.sinha@intel.com;
> daniel.m.buckley@intel.com; Archana Muniganti <marchana@marvell.com>;
> Tejasree Kondoj <ktejasree@marvell.com>; hemant.agrawal@nxp.com;
> matan@nvidia.com; konstantin.ananyev@intel.com; Akhil Goyal
> <gakhil@marvell.com>
> Subject: [PATCH] doc: announce SA config option struct changes
> 
> From: Archana Muniganti <marchana@marvell.com>
> 
> Propose new fields to support offloads like
> - IPsec inner checksum(L3/L4)
> - IPsec tunnel header verification
> - TSO
> - etc
> in the structure ``rte_security_ipsec_sa_options``.
> 
> Signed-off-by: Archana Muniganti <marchana@marvell.com>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> Acked-by: Akhil Goyal <gakhil@marvell.com>
> ---
> Proposal for changes are floated as per below links.
> https://mails.dpdk.org/archives/dev/2021-June/212977.html
> https://mails.dpdk.org/archives/dev/2021-July/213484.html
> https://mails.dpdk.org/archives/dev/2021-July/214521.html
> 
> Cc: Radu Nicolau <radu.nicolau@intel.com>
> 
>  doc/guides/rel_notes/deprecation.rst | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/deprecation.rst
> b/doc/guides/rel_notes/deprecation.rst
> index 6a35c7649a..96192b4a41 100644
> --- a/doc/guides/rel_notes/deprecation.rst
> +++ b/doc/guides/rel_notes/deprecation.rst
> @@ -151,3 +151,7 @@ Deprecation Notices
>  * security: The functions ``rte_security_set_pkt_metadata`` and
>    ``rte_security_get_userdata`` will be made inline functions and additional
>    flags will be added in structure ``rte_security_ctx`` in DPDK 21.11.
> +
> +* security: The IPsec SA config options ``struct
> rte_security_ipsec_sa_options``
> +  will be updated with new fields to support new features like IPsec inner
> +  checksum, tunnel header verification, TSO in case of protocol offload.
> --
> 2.25.1
Akhil Goyal Aug. 2, 2021, 12:41 p.m. UTC | #2 
> Hi Akhil,
> 
> There are 2 more items that I would like to add.
> - IV gen (in ``rte_security_ipsec_sa_options``)
> - SA lifetime configuration (in ``rte_security_ipsec_xform``)
> 
> http://patches.dpdk.org/project/dpdk/patch/1626759974-334-2-git-send-
> email-anoobj@marvell.com/
> http://patches.dpdk.org/project/dpdk/patch/1626759974-334-3-git-send-
> email-anoobj@marvell.com/
> 
> Do you recommend sending separate deprecation notice or can this patch be
> updated to cover those as well?
> 
> Thanks,
> Anoob
> 
This patch is to cover changes in ``rte_security_ipsec_sa_options``, hence for iv_gen no need.
For ipsec_xform a new one would be good.
Akhil Goyal Aug. 4, 2021, 5:03 p.m. UTC | #3 
> Subject: [PATCH] doc: announce SA config option struct changes
> 
> From: Archana Muniganti <marchana@marvell.com>
> 
> Propose new fields to support offloads like
> - IPsec inner checksum(L3/L4)
> - IPsec tunnel header verification
> - TSO
> - etc
> in the structure ``rte_security_ipsec_sa_options``.
> 
> Signed-off-by: Archana Muniganti <marchana@marvell.com>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> Acked-by: Akhil Goyal <gakhil@marvell.com>
> ---

Ping for Acks.
Radu Nicolau Aug. 5, 2021, 10 a.m. UTC | #4 
On 7/31/2021 6:44 PM, Akhil Goyal wrote:
> From: Archana Muniganti <marchana@marvell.com>
>
> Propose new fields to support offloads like
> - IPsec inner checksum(L3/L4)
> - IPsec tunnel header verification
> - TSO
> - etc
> in the structure ``rte_security_ipsec_sa_options``.
>
> Signed-off-by: Archana Muniganti <marchana@marvell.com>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> Acked-by: Akhil Goyal <gakhil@marvell.com>
> ---
> Proposal for changes are floated as per below links.
> https://mails.dpdk.org/archives/dev/2021-June/212977.html
> https://mails.dpdk.org/archives/dev/2021-July/213484.html
> https://mails.dpdk.org/archives/dev/2021-July/214521.html
>
> Cc: Radu Nicolau <radu.nicolau@intel.com>
Acked-by: Radu Nicolau <radu.nicolau@intel.com>
Hemant Agrawal Aug. 5, 2021, 10:14 a.m. UTC | #5 
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Jerin Jacob Aug. 7, 2021, 1:31 p.m. UTC | #6 
On Thu, Aug 5, 2021 at 3:44 PM Hemant Agrawal <hemant.agrawal@nxp.com> wrote:
>
> Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>


Acked-by: Jerin Jacob <jerinj@marvell.com>
Thomas Monjalon Aug. 7, 2021, 5:17 p.m. UTC | #7 
> > Propose new fields to support offloads like
> > - IPsec inner checksum(L3/L4)
> > - IPsec tunnel header verification
> > - TSO
> > - etc
> > in the structure ``rte_security_ipsec_sa_options``.
> >
> > Signed-off-by: Archana Muniganti <marchana@marvell.com>
> > Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> > Acked-by: Akhil Goyal <gakhil@marvell.com>
> > ---
> > Proposal for changes are floated as per below links.
> > https://mails.dpdk.org/archives/dev/2021-June/212977.html
> > https://mails.dpdk.org/archives/dev/2021-July/213484.html
> > https://mails.dpdk.org/archives/dev/2021-July/214521.html
> >
> Acked-by: Radu Nicolau <radu.nicolau@intel.com>
Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>
Acked-by: Jerin Jacob <jerinj@marvell.com>

Applied, thanks.
diff mbox series

Patch

diff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst
index 6a35c7649a..96192b4a41 100644
--- a/doc/guides/rel_notes/deprecation.rst
+++ b/doc/guides/rel_notes/deprecation.rst
@@ -151,3 +151,7 @@  Deprecation Notices
 * security: The functions ``rte_security_set_pkt_metadata`` and
   ``rte_security_get_userdata`` will be made inline functions and additional
   flags will be added in structure ``rte_security_ctx`` in DPDK 21.11.
+
+* security: The IPsec SA config options ``struct rte_security_ipsec_sa_options``
+  will be updated with new fields to support new features like IPsec inner
+  checksum, tunnel header verification, TSO in case of protocol offload.