[dpdk-dev] [PATCH v5 0/3] security: support for pdcp

Hemant Agrawal hemant.agrawal at nxp.com
Mon Oct 22 09:10:23 CEST 2018



On 10/18/2018 8:10 PM, Thomas Monjalon wrote:
> 16/10/2018 16:35, Akhil Goyal:
>>> Akhil Goyal (3):
>>>     security: support pdcp protocol
>>>     crypto/dpaa2_sec: add sample pdcp descriptor apis
>>>     crypto/dpaa2_sec: support pdcp offload
>>>
>>>    doc/guides/prog_guide/rte_security.rst      |  107 +-
>>>    drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c |  257 ++
>>>    drivers/crypto/dpaa2_sec/dpaa2_sec_priv.h   |  208 +-
>>>    drivers/crypto/dpaa2_sec/hw/desc.h          |    2 +-
>>>    drivers/crypto/dpaa2_sec/hw/desc/pdcp.h     | 2796 +++++++++++++++++++
>>>    lib/librte_security/rte_security.c          |    4 +
>>>    lib/librte_security/rte_security.h          |   92 +
>>>    7 files changed, 3450 insertions(+), 16 deletions(-)
>>>    create mode 100644 drivers/crypto/dpaa2_sec/hw/desc/pdcp.h
>>>
>> Patchset applied to dpdk-next-crypto
>>
>> minor modification as suggested by Anoob is done.
> This patchset won't be part of the pull in master because of 2 issues:
>
> - clang error:
> drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c:2759:18: fatal error:
> implicit conversion from enumeration type 'enum rte_security_pdcp_sn_size'
> to different enumeration type 'enum pdcp_sn_size'
Got it.  I am sending the fix (Akhil is on leave).
> - doxygen error:
> rte_security.h:225: warning: Found unknown command `\PDCP_SN_SIZE_5'
> rte_security.h:226: warning: Found unknown command `\PDCP_SN_SIZE_7'
> rte_security.h:227: warning: Found unknown command `\PDCP_SN_SIZE_12'
> rte_security.h:228: warning: Found unknown command `\PDCP_SN_SIZE_15'
> rte_security.h:229: warning: Found unknown command `\PDCP_SN_SIZE_18'
>
> I think these doxygen comments can be removed.
>
>
I am fixing the comment and placing them at right place. The whole file 
has comment for structures, so it is better that we define these as well.


More information about the dev mailing list