Yes, for auth cases, the PMD may give errors. However, cipher only cases would work without issues.

You can send a patch to fix this. But make sure not to add that inside the time keeping loop.

From: Suanming Mou <suanmingm@nvidia.com>
Sent: Friday, July 21, 2023 5:23 PM
To: Akhil Goyal <gakhil@marvell.com>; ciara.power@intel.com
Cc: dev@dpdk.org
Subject: [EXT] RE: crypto decryption support in test-crypto-perf pmd-cycle-count test

External Email

---

Thanks Akhil.

Yes, I see.

But for decryption, we should prepare the correct cipher data, right? Or that test does not make any sense.

I assume decryption with incorrect data will cause PMD reject work properly.

From: Akhil Goyal <gakhil@marvell.com>
Sent: Friday, July 21, 2023 7:49 PM
To: Suanming Mou <suanmingm@nvidia.com>; ciara.power@intel.com
Cc: dev@dpdk.org
Subject: RE: crypto decryption support in test-crypto-perf pmd-cycle-count test

Hi Suanming,

The pmd-cycle-count mode is for counting the cycles for enqueue and dequeue.

It does not matter what is the content of the packet, as we are not validating the content

after dequeue of the packet. PMD/Hw will assume it as plain/encrypted payload and

will process it as per the session attached.

Hence, doing memcpy to mbuf is not required.

Regards,

Akhil

From: Suanming Mou <suanmingm@nvidia.com>
Sent: Friday, July 21, 2023 4:06 PM
To: ciara.power@intel.com; Akhil Goyal <gakhil@marvell.com>
Cc: dev@dpdk.org
Subject: [EXT] crypto decryption support in test-crypto-perf pmd-cycle-count test

External Email

---

Hi Guys,

Sorry for the inconvenience.

I have one small question with pmd-cycle-count test in test-crypto-perf application.

As in pmd-cycle-count run code, I don’t see the cperf_mbuf_set() function is called to fill the test data.

So for decryption, I assume it will always fails. Feel like pmd-cycle-count test does not expect to support decryption. Or is it a bug?

I have no idea with the application design background, is it expected?

Thanks,

Suanming Mou