RE: [EXTERNAL] [PATCH] app/crypto-perf: fix aad offset alignment

2025-05-21 Thread Akhil Goyal
> > > > > > Hi, > > > > AAD offset in AES-GCM crypto test was calculated by adding 16-byte > > > > alignment after the IV, which is only needed in AES-CCM. > > > > > > Agreed that CCM has a requirement for 16B alignment. > > > But for GCM, does it break any protocol? Can we not align to byte > > >

RE: [EXTERNAL] [PATCH] app/crypto-perf: fix aad offset alignment

2025-05-18 Thread Shani Peretz
> -Original Message- > From: Shani Peretz > Sent: Wednesday, 26 March 2025 9:14 > To: Akhil Goyal ; dev@dpdk.org > Cc: Suanming Mou ; sta...@dpdk.org; Brian > Dooley ; Pablo de Lara > > Subject: RE: [EXTERNAL] [PATCH] app/crypto-perf: fix aad offset alignment

RE: [EXTERNAL] [PATCH] app/crypto-perf: fix aad offset alignment

2025-03-26 Thread Shani Peretz
> -Original Message- > From: Akhil Goyal > Sent: Monday, 17 March 2025 12:23 > To: Shani Peretz ; dev@dpdk.org > Cc: Suanming Mou ; sta...@dpdk.org; Brian > Dooley ; Pablo de Lara > > Subject: RE: [EXTERNAL] [PATCH] app/crypto-perf: fix aad offset alignment

RE: [EXTERNAL] [PATCH] app/crypto-perf: fix aad offset alignment

2025-03-17 Thread Akhil Goyal
Hi, > AAD offset in AES-GCM crypto test was calculated by adding > 16-byte alignment after the IV, which is only needed in AES-CCM. Agreed that CCM has a requirement for 16B alignment. But for GCM, does it break any protocol? Can we not align to byte boundary for performance? This is a performanc

[PATCH] app/crypto-perf: fix aad offset alignment

2025-03-11 Thread Shani Peretz
AAD offset in AES-GCM crypto test was calculated by adding 16-byte alignment after the IV, which is only needed in AES-CCM. The patch correct the AAD offset calculation in AES-GCM algorithm tests. Fixes: 0b242422d385 ("app/crypto-perf: set AAD after the crypto operation") Cc: sta...@dpdk.org Sig