DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] crypto/octeontx2: update min headroom and tailroom
@ 2021-08-17 17:45 Akhil Goyal
  2021-09-29  4:46 ` Anoob Joseph
  0 siblings, 1 reply; 3+ messages in thread
From: Akhil Goyal @ 2021-08-17 17:45 UTC (permalink / raw)
  To: anoobj, dev; +Cc: adwivedi, Akhil Goyal

The driver consume 208B of tailroom but has min requirement
as 8B and headroom needed is 48B, but minimum requirement
is set as 24B. This patch correct minimum requirements
which application should honour.

Signed-off-by: Akhil Goyal <gakhil@marvell.com>
---
 drivers/crypto/octeontx2/otx2_cryptodev_ops.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/crypto/octeontx2/otx2_cryptodev_ops.h b/drivers/crypto/octeontx2/otx2_cryptodev_ops.h
index 1970187f88..043bda5a94 100644
--- a/drivers/crypto/octeontx2/otx2_cryptodev_ops.h
+++ b/drivers/crypto/octeontx2/otx2_cryptodev_ops.h
@@ -7,8 +7,8 @@
 
 #include <rte_cryptodev_pmd.h>
 
-#define OTX2_CPT_MIN_HEADROOM_REQ	24
-#define OTX2_CPT_MIN_TAILROOM_REQ	8
+#define OTX2_CPT_MIN_HEADROOM_REQ	48
+#define OTX2_CPT_MIN_TAILROOM_REQ	208
 
 extern struct rte_cryptodev_ops otx2_cpt_ops;
 
-- 
2.25.1


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] crypto/octeontx2: update min headroom and tailroom
  2021-08-17 17:45 [dpdk-dev] [PATCH] crypto/octeontx2: update min headroom and tailroom Akhil Goyal
@ 2021-09-29  4:46 ` Anoob Joseph
  2021-10-05  8:19   ` Akhil Goyal
  0 siblings, 1 reply; 3+ messages in thread
From: Anoob Joseph @ 2021-09-29  4:46 UTC (permalink / raw)
  To: Akhil Goyal, dev; +Cc: Ankur Dwivedi, Akhil Goyal

> 
> The driver consume 208B of tailroom but has min requirement as 8B and
> headroom needed is 48B, but minimum requirement is set as 24B. This patch
> correct minimum requirements which application should honour.
> 
> Signed-off-by: Akhil Goyal <gakhil@marvell.com>

Acked-by: Anoob Joseph <anoobj@marvell.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] crypto/octeontx2: update min headroom and tailroom
  2021-09-29  4:46 ` Anoob Joseph
@ 2021-10-05  8:19   ` Akhil Goyal
  0 siblings, 0 replies; 3+ messages in thread
From: Akhil Goyal @ 2021-10-05  8:19 UTC (permalink / raw)
  To: Anoob Joseph, dev; +Cc: Ankur Dwivedi

> > The driver consume 208B of tailroom but has min requirement as 8B and
> > headroom needed is 48B, but minimum requirement is set as 24B. This
> patch
> > correct minimum requirements which application should honour.
> >
> > Signed-off-by: Akhil Goyal <gakhil@marvell.com>
> 
> Acked-by: Anoob Joseph <anoobj@marvell.com>

Applied to dpdk-next-crypto


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-10-05  8:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-17 17:45 [dpdk-dev] [PATCH] crypto/octeontx2: update min headroom and tailroom Akhil Goyal
2021-09-29  4:46 ` Anoob Joseph
2021-10-05  8:19   ` Akhil Goyal

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).