crypto/octeontx2: fix structure alignment
Checks
Commit Message
The structure cpt_request_info needs only 8 byte alignment.
This patch replaces __rte_cache_aligned of cpt_request_info
with __rte_aligned(8) and removes __rte_aligned(8) in
cpt_meta_info structure.
Fixes: fab634eb87ca ("crypto/octeontx2: support security session data path")
Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
---
drivers/common/cpt/cpt_common.h | 2 +-
drivers/crypto/octeontx2/otx2_cryptodev.h | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
Comments
>
> The structure cpt_request_info needs only 8 byte alignment.
> This patch replaces __rte_cache_aligned of cpt_request_info with
> __rte_aligned(8) and removes __rte_aligned(8) in cpt_meta_info structure.
>
> Fixes: fab634eb87ca ("crypto/octeontx2: support security session data path")
>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
Acked-by: Anoob Joseph <anoobj@marvell.com>
Thanks Ferruh for pointing this out.
> >
> > The structure cpt_request_info needs only 8 byte alignment.
> > This patch replaces __rte_cache_aligned of cpt_request_info with
> > __rte_aligned(8) and removes __rte_aligned(8) in cpt_meta_info structure.
> >
> > Fixes: fab634eb87ca ("crypto/octeontx2: support security session data path")
> >
> > Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
>
> Acked-by: Anoob Joseph <anoobj@marvell.com>
>
> Thanks Ferruh for pointing this out.
Applied to dpdk-next-crypto
Thanks.
@@ -76,6 +76,6 @@ struct cpt_request_info {
/** Control path fields */
uint64_t time_out;
uint8_t extra_time;
-} __rte_cache_aligned;
+} __rte_aligned(8);
#endif /* _CPT_COMMON_H_ */
@@ -38,7 +38,7 @@ struct cpt_meta_info {
uint64_t deq_op_info[4];
uint64_t comp_code_sz;
union cpt_res_s cpt_res __rte_aligned(16);
- struct cpt_request_info cpt_req __rte_aligned(8);
+ struct cpt_request_info cpt_req;
};
#define CPT_LOGTYPE otx2_cpt_logtype