* [dpdk-dev] [PATCH v3] crypto/aesni_mb: support DOCSIS AES-256 @ 2020-04-08 14:04 Mairtin o Loingsigh 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 1/2] " Mairtin o Loingsigh 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors Mairtin o Loingsigh 0 siblings, 2 replies; 12+ messages in thread From: Mairtin o Loingsigh @ 2020-04-08 14:04 UTC (permalink / raw) To: pablo.de.lara.guarch; +Cc: dev, Mairtin o Loingsigh This patch adds support for DOCSIS AES-256 when using AESNI-MB Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> --- doc/guides/rel_notes/release_20_05.rst | 5 +++++ drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 6 ++++++ 2 files changed, 11 insertions(+) diff --git a/doc/guides/rel_notes/release_20_05.rst b/doc/guides/rel_notes/release_20_05.rst index 6b1a7c58c..72d2f6c27 100644 --- a/doc/guides/rel_notes/release_20_05.rst +++ b/doc/guides/rel_notes/release_20_05.rst @@ -81,6 +81,11 @@ New Features by making use of the event device capabilities. The event mode currently supports only inline IPsec protocol offload. +* **Updated the AESNI MB PMD.** + + Updated the AESNI PMD with additional DOCSIS algorithm + + * Added support for AES-256 DOCSIS Removed Items ------------- diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c index 845661174..0d555ec29 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c @@ -319,8 +319,14 @@ static const struct rte_cryptodev_capabilities aesni_mb_pmd_capabilities[] = { }, .iv_size = { .min = 16, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .max = 32, + .increment = 16 +#else .max = 16, .increment = 0 +#endif + } }, } }, } -- 2.17.1 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [dpdk-dev] [PATCH v4 1/2] crypto/aesni_mb: support DOCSIS AES-256 2020-04-08 14:04 [dpdk-dev] [PATCH v3] crypto/aesni_mb: support DOCSIS AES-256 Mairtin o Loingsigh @ 2020-04-10 15:03 ` Mairtin o Loingsigh 2020-04-14 8:45 ` De Lara Guarch, Pablo 2020-04-15 9:04 ` [dpdk-dev] [PATCH v5] " Mairtin o Loingsigh 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors Mairtin o Loingsigh 1 sibling, 2 replies; 12+ messages in thread From: Mairtin o Loingsigh @ 2020-04-10 15:03 UTC (permalink / raw) To: pablo.de.lara.guarch; +Cc: dev, Mairtin o Loingsigh This patch adds support for DOCSIS AES-256 when using AESNI-MB Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> --- v2: Added IPSec MB version check v3: Added doc update v4: Added missing patch --- doc/guides/rel_notes/release_20_05.rst | 5 +++++ drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 6 ++++++ 2 files changed, 11 insertions(+) diff --git a/doc/guides/rel_notes/release_20_05.rst b/doc/guides/rel_notes/release_20_05.rst index 6b1a7c58c..72d2f6c27 100644 --- a/doc/guides/rel_notes/release_20_05.rst +++ b/doc/guides/rel_notes/release_20_05.rst @@ -81,6 +81,11 @@ New Features by making use of the event device capabilities. The event mode currently supports only inline IPsec protocol offload. +* **Updated the AESNI MB PMD.** + + Updated the AESNI PMD with additional DOCSIS algorithm + + * Added support for AES-256 DOCSIS Removed Items ------------- diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c index 845661174..0d555ec29 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c @@ -319,8 +319,14 @@ static const struct rte_cryptodev_capabilities aesni_mb_pmd_capabilities[] = { }, .iv_size = { .min = 16, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .max = 32, + .increment = 16 +#else .max = 16, .increment = 0 +#endif + } }, } }, } -- 2.12.3 ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v4 1/2] crypto/aesni_mb: support DOCSIS AES-256 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 1/2] " Mairtin o Loingsigh @ 2020-04-14 8:45 ` De Lara Guarch, Pablo 2020-04-15 9:04 ` [dpdk-dev] [PATCH v5] " Mairtin o Loingsigh 1 sibling, 0 replies; 12+ messages in thread From: De Lara Guarch, Pablo @ 2020-04-14 8:45 UTC (permalink / raw) To: O'loingsigh, Mairtin; +Cc: dev Hi Mairtin, > -----Original Message----- > From: O'loingsigh, Mairtin <mairtin.oloingsigh@intel.com> > Sent: Friday, April 10, 2020 4:03 PM > To: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com> > Cc: dev@dpdk.org; O'loingsigh, Mairtin <mairtin.oloingsigh@intel.com> > Subject: [PATCH v4 1/2] crypto/aesni_mb: support DOCSIS AES-256 > > This patch adds support for DOCSIS AES-256 when using AESNI-MB > > Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> > Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> > --- > v2: Added IPSec MB version check > v3: Added doc update > v4: Added missing patch > --- > doc/guides/rel_notes/release_20_05.rst | 5 +++++ > drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 6 ++++++ > 2 files changed, 11 insertions(+) > > diff --git a/doc/guides/rel_notes/release_20_05.rst > b/doc/guides/rel_notes/release_20_05.rst > index 6b1a7c58c..72d2f6c27 100644 > --- a/doc/guides/rel_notes/release_20_05.rst > +++ b/doc/guides/rel_notes/release_20_05.rst > @@ -81,6 +81,11 @@ New Features > by making use of the event device capabilities. The event mode currently > supports > only inline IPsec protocol offload. > > +* **Updated the AESNI MB PMD.** > + > + Updated the AESNI PMD with additional DOCSIS algorithm > + > + * Added support for AES-256 DOCSIS > > Removed Items > ------------- > diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > index 845661174..0d555ec29 100644 > --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > @@ -319,8 +319,14 @@ static const struct rte_cryptodev_capabilities > aesni_mb_pmd_capabilities[] = { > }, > .iv_size = { > .min = 16, > +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) > + .max = 32, > + .increment = 16 > +#else > .max = 16, > .increment = 0 > +#endif Apologies, I missed this earlier and I just realized this is wrong. This should be done in key_size, not iv_size. Thanks, Pablo > + > } > }, } > }, } > -- > 2.12.3 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [dpdk-dev] [PATCH v5] crypto/aesni_mb: support DOCSIS AES-256 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 1/2] " Mairtin o Loingsigh 2020-04-14 8:45 ` De Lara Guarch, Pablo @ 2020-04-15 9:04 ` Mairtin o Loingsigh 2020-04-15 11:07 ` De Lara Guarch, Pablo 2020-04-15 11:51 ` [dpdk-dev] [PATCH v6] " Mairtin o Loingsigh 1 sibling, 2 replies; 12+ messages in thread From: Mairtin o Loingsigh @ 2020-04-15 9:04 UTC (permalink / raw) To: pablo.de.lara.guarch; +Cc: dev, Mairtin o Loingsigh This patch adds support for DOCSIS AES-256 when using AESNI-MB Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> --- v2: Added IPSec MB version check v3: Added doc update v4: Added missing patch v5: Dropping test vector due to refactoring --- doc/guides/rel_notes/release_20_05.rst | 5 +++++ drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 6 ++++++ 2 files changed, 11 insertions(+) diff --git a/doc/guides/rel_notes/release_20_05.rst b/doc/guides/rel_notes/release_20_05.rst index 6b1a7c58c..72d2f6c27 100644 --- a/doc/guides/rel_notes/release_20_05.rst +++ b/doc/guides/rel_notes/release_20_05.rst @@ -81,6 +81,11 @@ New Features by making use of the event device capabilities. The event mode currently supports only inline IPsec protocol offload. +* **Updated the AESNI MB PMD.** + + Updated the AESNI PMD with additional DOCSIS algorithm + + * Added support for AES-256 DOCSIS Removed Items ------------- diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c index 845661174..0d555ec29 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c @@ -319,8 +319,14 @@ static const struct rte_cryptodev_capabilities aesni_mb_pmd_capabilities[] = { }, .iv_size = { .min = 16, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .max = 32, + .increment = 16 +#else .max = 16, .increment = 0 +#endif + } }, } }, } -- 2.12.3 ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v5] crypto/aesni_mb: support DOCSIS AES-256 2020-04-15 9:04 ` [dpdk-dev] [PATCH v5] " Mairtin o Loingsigh @ 2020-04-15 11:07 ` De Lara Guarch, Pablo 2020-04-15 11:51 ` [dpdk-dev] [PATCH v6] " Mairtin o Loingsigh 1 sibling, 0 replies; 12+ messages in thread From: De Lara Guarch, Pablo @ 2020-04-15 11:07 UTC (permalink / raw) To: O'loingsigh, Mairtin; +Cc: dev Hi Mairtin, > -----Original Message----- > From: O'loingsigh, Mairtin <mairtin.oloingsigh@intel.com> > Sent: Wednesday, April 15, 2020 10:04 AM > To: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com> > Cc: dev@dpdk.org; O'loingsigh, Mairtin <mairtin.oloingsigh@intel.com> > Subject: [PATCH v5] crypto/aesni_mb: support DOCSIS AES-256 > > This patch adds support for DOCSIS AES-256 when using AESNI-MB > > Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> > Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> > --- > v2: Added IPSec MB version check > v3: Added doc update > v4: Added missing patch > v5: Dropping test vector due to refactoring > --- > doc/guides/rel_notes/release_20_05.rst | 5 +++++ > drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 6 ++++++ > 2 files changed, 11 insertions(+) > > diff --git a/doc/guides/rel_notes/release_20_05.rst > b/doc/guides/rel_notes/release_20_05.rst > index 6b1a7c58c..72d2f6c27 100644 > --- a/doc/guides/rel_notes/release_20_05.rst > +++ b/doc/guides/rel_notes/release_20_05.rst > @@ -81,6 +81,11 @@ New Features > by making use of the event device capabilities. The event mode currently > supports > only inline IPsec protocol offload. > > +* **Updated the AESNI MB PMD.** > + > + Updated the AESNI PMD with additional DOCSIS algorithm > + > + * Added support for AES-256 DOCSIS > > Removed Items > ------------- > diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > index 845661174..0d555ec29 100644 > --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c > @@ -319,8 +319,14 @@ static const struct rte_cryptodev_capabilities > aesni_mb_pmd_capabilities[] = { > }, > .iv_size = { > .min = 16, > +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) > + .max = 32, > + .increment = 16 > +#else This should be done in "key_size", not iv_size. Thanks, Pablo ^ permalink raw reply [flat|nested] 12+ messages in thread
* [dpdk-dev] [PATCH v6] crypto/aesni_mb: support DOCSIS AES-256 2020-04-15 9:04 ` [dpdk-dev] [PATCH v5] " Mairtin o Loingsigh 2020-04-15 11:07 ` De Lara Guarch, Pablo @ 2020-04-15 11:51 ` Mairtin o Loingsigh 2020-04-17 16:31 ` Akhil Goyal 2020-04-17 20:59 ` Akhil Goyal 1 sibling, 2 replies; 12+ messages in thread From: Mairtin o Loingsigh @ 2020-04-15 11:51 UTC (permalink / raw) To: pablo.de.lara.guarch; +Cc: dev, Mairtin o Loingsigh This patch adds support for DOCSIS AES-256 when using AESNI-MB Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> --- This patch depends on these two patches: v2: Added IPSec MB version check v3: Added doc update v4: Added missing patch v5: Dropping test vector due to refactoring v6: Fix key size --- doc/guides/rel_notes/release_20_05.rst | 5 +++++ drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c | 5 +++++ 2 files changed, 10 insertions(+) diff --git a/doc/guides/rel_notes/release_20_05.rst b/doc/guides/rel_notes/release_20_05.rst index 6b1a7c58c..72d2f6c27 100644 --- a/doc/guides/rel_notes/release_20_05.rst +++ b/doc/guides/rel_notes/release_20_05.rst @@ -81,6 +81,11 @@ New Features by making use of the event device capabilities. The event mode currently supports only inline IPsec protocol offload. +* **Updated the AESNI MB PMD.** + + Updated the AESNI PMD with additional DOCSIS algorithm + + * Added support for AES-256 DOCSIS Removed Items ------------- diff --git a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c index 845661174..434e2652b 100644 --- a/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c +++ b/drivers/crypto/aesni_mb/rte_aesni_mb_pmd_ops.c @@ -314,8 +314,13 @@ static const struct rte_cryptodev_capabilities aesni_mb_pmd_capabilities[] = { .block_size = 16, .key_size = { .min = 16, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .max = 32, + .increment = 16 +#else .max = 16, .increment = 0 +#endif }, .iv_size = { .min = 16, -- 2.12.3 ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v6] crypto/aesni_mb: support DOCSIS AES-256 2020-04-15 11:51 ` [dpdk-dev] [PATCH v6] " Mairtin o Loingsigh @ 2020-04-17 16:31 ` Akhil Goyal 2020-04-17 20:59 ` Akhil Goyal 1 sibling, 0 replies; 12+ messages in thread From: Akhil Goyal @ 2020-04-17 16:31 UTC (permalink / raw) To: Mairtin o Loingsigh, pablo.de.lara.guarch; +Cc: dev > > This patch adds support for DOCSIS AES-256 when using AESNI-MB > > Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> > Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> > --- > This patch depends on these two patches: > What are the dependent patches? ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v6] crypto/aesni_mb: support DOCSIS AES-256 2020-04-15 11:51 ` [dpdk-dev] [PATCH v6] " Mairtin o Loingsigh 2020-04-17 16:31 ` Akhil Goyal @ 2020-04-17 20:59 ` Akhil Goyal 1 sibling, 0 replies; 12+ messages in thread From: Akhil Goyal @ 2020-04-17 20:59 UTC (permalink / raw) To: Mairtin o Loingsigh, pablo.de.lara.guarch; +Cc: dev > > This patch adds support for DOCSIS AES-256 when using AESNI-MB > > Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> > Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> > --- > This patch depends on these two patches: > Applied to dpdk-next-crypto Thanks. ^ permalink raw reply [flat|nested] 12+ messages in thread
* [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors 2020-04-08 14:04 [dpdk-dev] [PATCH v3] crypto/aesni_mb: support DOCSIS AES-256 Mairtin o Loingsigh 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 1/2] " Mairtin o Loingsigh @ 2020-04-10 15:03 ` Mairtin o Loingsigh 2020-04-12 16:43 ` Thomas Monjalon 1 sibling, 1 reply; 12+ messages in thread From: Mairtin o Loingsigh @ 2020-04-10 15:03 UTC (permalink / raw) To: pablo.de.lara.guarch; +Cc: dev, Mairtin o Loingsigh This patch adds test vectors for AES-256 and sets AESNI-MB as the target PMD Signed-off-by: Mairtin o Loingsigh <mairtin.oloingsigh@intel.com> Acked-by: Pablo de Lara <Pablo.de.lara.guarch@intel.com> --- v2: Remove duplicate code --- app/test/test_cryptodev_aes_test_vectors.h | 81 ++++++++++++++++++++++++++++++ 1 file changed, 81 insertions(+) diff --git a/app/test/test_cryptodev_aes_test_vectors.h b/app/test/test_cryptodev_aes_test_vectors.h index 08ae09390..df338756f 100644 --- a/app/test/test_cryptodev_aes_test_vectors.h +++ b/app/test/test_cryptodev_aes_test_vectors.h @@ -5,6 +5,15 @@ #ifndef TEST_CRYPTODEV_AES_TEST_VECTORS_H_ #define TEST_CRYPTODEV_AES_TEST_VECTORS_H_ +#ifdef RTE_LIBRTE_PMD_AESNI_MB +#include <intel-ipsec-mb.h> +#endif + +#if !defined(IMB_VERSION_NUM) +#define IMB_VERSION(a, b, c) (((a) << 16) + ((b) << 8) + (c)) +#define IMB_VERSION_NUM IMB_VERSION(0, 49, 0) +#endif + /* test vectors */ static const uint8_t plaintext_aes128ctr[] = { 0x6B, 0xC1, 0xBE, 0xE2, 0x2E, 0x40, 0x9F, 0x96, @@ -2739,79 +2748,151 @@ static const struct blockcipher_test_case aes_docsis_test_cases[] = { .test_descr = "AES-256-DOCSIS-BPI Full Block Encryption", .test_data = &aes_test_data_docsis_4, .op_mask = BLOCKCIPHER_TEST_OP_ENCRYPT, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI Runt Block Encryption", .test_data = &aes_test_data_docsis_5, .op_mask = BLOCKCIPHER_TEST_OP_ENCRYPT, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI Uneven Encryption", .test_data = &aes_test_data_docsis_6, .op_mask = BLOCKCIPHER_TEST_OP_ENCRYPT, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI Full Block Decryption", .test_data = &aes_test_data_docsis_4, .op_mask = BLOCKCIPHER_TEST_OP_DECRYPT, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI Runt Block Decryption", .test_data = &aes_test_data_docsis_5, .op_mask = BLOCKCIPHER_TEST_OP_DECRYPT, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI Uneven Decryption", .test_data = &aes_test_data_docsis_6, .op_mask = BLOCKCIPHER_TEST_OP_DECRYPT, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI OOP Full Block Encryption", .test_data = &aes_test_data_docsis_4, .op_mask = BLOCKCIPHER_TEST_OP_ENCRYPT, .feature_mask = BLOCKCIPHER_TEST_FEATURE_OOP, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI OOP Runt Block Encryption", .test_data = &aes_test_data_docsis_5, .op_mask = BLOCKCIPHER_TEST_OP_ENCRYPT, .feature_mask = BLOCKCIPHER_TEST_FEATURE_OOP, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI OOP Uneven Block Encryption", .test_data = &aes_test_data_docsis_6, .op_mask = BLOCKCIPHER_TEST_OP_ENCRYPT, .feature_mask = BLOCKCIPHER_TEST_FEATURE_OOP, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI OOP Full Block Decryption", .test_data = &aes_test_data_docsis_4, .op_mask = BLOCKCIPHER_TEST_OP_DECRYPT, .feature_mask = BLOCKCIPHER_TEST_FEATURE_OOP, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI OOP Runt Block Decryption", .test_data = &aes_test_data_docsis_5, .op_mask = BLOCKCIPHER_TEST_OP_DECRYPT, .feature_mask = BLOCKCIPHER_TEST_FEATURE_OOP, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, { .test_descr = "AES-256-DOCSIS-BPI OOP Uneven Block Decryption", .test_data = &aes_test_data_docsis_6, .op_mask = BLOCKCIPHER_TEST_OP_DECRYPT, .feature_mask = BLOCKCIPHER_TEST_FEATURE_OOP, +#if IMB_VERSION_NUM >= IMB_VERSION(0, 53, 3) + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT | + BLOCKCIPHER_TEST_TARGET_PMD_MB +#else + .pmd_mask = BLOCKCIPHER_TEST_TARGET_PMD_QAT +#endif }, }; #endif /* TEST_CRYPTODEV_AES_TEST_VECTORS_H_ */ -- 2.12.3 ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors Mairtin o Loingsigh @ 2020-04-12 16:43 ` Thomas Monjalon 2020-04-14 8:49 ` De Lara Guarch, Pablo 0 siblings, 1 reply; 12+ messages in thread From: Thomas Monjalon @ 2020-04-12 16:43 UTC (permalink / raw) To: dev, akhil.goyal Cc: pablo.de.lara.guarch, Mairtin o Loingsigh, declan.doherty, Adam Dybkowski, fiona.trahe 10/04/2020 17:03, Mairtin o Loingsigh: > This patch adds test vectors for AES-256 and sets AESNI-MB as the > target PMD There is really something wrong in the cryptodev API. Why the PMD must be selected in the tests? Isn't there some capabilities in cryptodev? Whitelisting PMD in the test prevents testing capabilities API. It looks the required refactoring is not progressing enough. I suggest blocking all crypto-related patches until is is fixed. ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors 2020-04-12 16:43 ` Thomas Monjalon @ 2020-04-14 8:49 ` De Lara Guarch, Pablo 2020-04-14 9:06 ` Thomas Monjalon 0 siblings, 1 reply; 12+ messages in thread From: De Lara Guarch, Pablo @ 2020-04-14 8:49 UTC (permalink / raw) To: Thomas Monjalon, dev, akhil.goyal Cc: O'loingsigh, Mairtin, Doherty, Declan, Dybkowski, AdamX, Trahe, Fiona Hi Thomas, > -----Original Message----- > From: dev <dev-bounces@dpdk.org> On Behalf Of Thomas Monjalon > Sent: Sunday, April 12, 2020 5:44 PM > To: dev@dpdk.org; akhil.goyal@nxp.com > Cc: De Lara Guarch, Pablo <pablo.de.lara.guarch@intel.com>; O'loingsigh, > Mairtin <mairtin.oloingsigh@intel.com>; Doherty, Declan > <declan.doherty@intel.com>; Dybkowski, AdamX > <adamx.dybkowski@intel.com>; Trahe, Fiona <fiona.trahe@intel.com> > Subject: Re: [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test > vectors > > 10/04/2020 17:03, Mairtin o Loingsigh: > > This patch adds test vectors for AES-256 and sets AESNI-MB as the > > target PMD > > There is really something wrong in the cryptodev API. > Why the PMD must be selected in the tests? > Isn't there some capabilities in cryptodev? > Whitelisting PMD in the test prevents testing capabilities API. > > It looks the required refactoring is not progressing enough. > I suggest blocking all crypto-related patches until is is fixed. > You are right, but Mairtin was following the same approach as it was taken previously. I agree that this approach is not the best one and we are currently working on a better solution. Thanks, Pablo ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors 2020-04-14 8:49 ` De Lara Guarch, Pablo @ 2020-04-14 9:06 ` Thomas Monjalon 0 siblings, 0 replies; 12+ messages in thread From: Thomas Monjalon @ 2020-04-14 9:06 UTC (permalink / raw) To: akhil.goyal, De Lara Guarch, Pablo Cc: dev, O'loingsigh, Mairtin, Doherty, Declan, Dybkowski, AdamX, Trahe, Fiona 14/04/2020 10:49, De Lara Guarch, Pablo: > From: Thomas Monjalon > > 10/04/2020 17:03, Mairtin o Loingsigh: > > > This patch adds test vectors for AES-256 and sets AESNI-MB as the > > > target PMD > > > > There is really something wrong in the cryptodev API. > > Why the PMD must be selected in the tests? > > Isn't there some capabilities in cryptodev? > > Whitelisting PMD in the test prevents testing capabilities API. > > > > It looks the required refactoring is not progressing enough. > > I suggest blocking all crypto-related patches until is is fixed. > > > > You are right, but Mairtin was following the same approach as it was taken previously. I know. His patch was accepted. > I agree that this approach is not the best one and we are currently working > on a better solution. Now that's clear, let's not merge any new crypto patch until the capabilities are working and tested. ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2020-04-17 20:59 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-04-08 14:04 [dpdk-dev] [PATCH v3] crypto/aesni_mb: support DOCSIS AES-256 Mairtin o Loingsigh 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 1/2] " Mairtin o Loingsigh 2020-04-14 8:45 ` De Lara Guarch, Pablo 2020-04-15 9:04 ` [dpdk-dev] [PATCH v5] " Mairtin o Loingsigh 2020-04-15 11:07 ` De Lara Guarch, Pablo 2020-04-15 11:51 ` [dpdk-dev] [PATCH v6] " Mairtin o Loingsigh 2020-04-17 16:31 ` Akhil Goyal 2020-04-17 20:59 ` Akhil Goyal 2020-04-10 15:03 ` [dpdk-dev] [PATCH v4 2/2] test/crypto: Add AES-256 DOCSIS test vectors Mairtin o Loingsigh 2020-04-12 16:43 ` Thomas Monjalon 2020-04-14 8:49 ` De Lara Guarch, Pablo 2020-04-14 9:06 ` Thomas Monjalon
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).