<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div class="elementToProof" style="font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);">
<br>
</div>
<div id="appendonsend"></div>
<div class="elementToProof" style="font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);">
Hi Gowrishankar,</div>
<div class="elementToProof" style="font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);">
Thanks for reach out, I don't have those environments setup on my end, but Ubuntu 20.04 and RHEL8 are old but still common LTS packages. </div>
<div class="elementToProof" style="font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);">
Are you able to dump the actual msg buffer ?</div>
<div class="elementToProof" style="font-family: "IntelOne Text"; font-size: 10pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
It is highly possible relates to dsitro-specific patches ? Paddings ? Or crypto polices in RHEL ? </div>
<div class="elementToProof" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Regards</div>
<div class="elementToProof" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div class="elementToProof" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
Kai</div>
<div class="elementToProof" style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<hr style="display: inline-block; width: 98%;">
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<b>From:</b> Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com><br>
<b>Sent:</b> Thursday, June 12, 2025 11:33<br>
<b>To:</b> Ji, Kai <kai.ji@intel.com><br>
<b>Cc:</b> Anoob Joseph <anoobj@marvell.com>; dev@dpdk.org <dev@dpdk.org>; Akhil Goyal <gakhil@marvell.com>; Fan Zhang <fanzhang.oss@gmail.com>; Shally Verma <shallyv@marvell.com>; Kanaka Durga Kotamarthy <kkotamarthy@marvell.com>; Ayuj Verma <ayverma@marvell.com>;
Kusztal, ArkadiuszX <arkadiuszx.kusztal@intel.com><br>
<b>Subject:</b> RE: [PATCH] test/crypto: fix RSA decrypt op validation </div>
<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-size: 11pt;">Hi Kai Ji,<br>
This patch adds a check on cop status after DECRYPT in existing RSA test.There are<br>
few CI failures for this patch and the failures are very specific to Openssl 1.1.1 variants<br>
used in below platforms:<br>
* Ubuntu 20.04 - Openssl 1.1.1f<br>
* RHEL 8 - Openssl 1.1.1k<br>
<br>
When I manually build these Openssl variants in the above platforms (using<br>
Docker images) and execute tests using respective openssl shared libraries,<br>
all tests PASS. If I use Openssl library that is shipped within the above<br>
distributions, the test in patch FAIL.<br>
<br>
I would like to get your opinion first to proceed on getting these patches CI acked.<br>
In my debug, I don't see any issue with the patch and the patch adds better<br>
Validation indeed. May be that, it has uncovered a potential failure in above<br>
Distros. Can you check on your end if these failures disappear in manual try.<br>
What would you suggest to go forward, for openssl PMD ?<br>
<br>
If required, I can start filing a Bugzilla as well.<br>
<br>
Thanks,<br>
Gowrishankar<br>
<br>
><br>
> Following RSA encrypt op, same plaintext buffer is used as output buffer for<br>
> decrypt op, hence comparing plaintext buffer against same buffer pointer in<br>
> crypto op always succeed irrespective of whether decrypt op succeeds or not.<br>
> This patch fixes this issue with a local buffer for crypto op.<br>
><br>
> Fixes: 5ae36995f10 ("test/crypto: move RSA enqueue/dequeue into<br>
> functions")<br>
> Cc: stable@dpdk.org<br>
><br>
> Signed-off-by: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com><br>
<br>
</div>
</body>
</html>