From: Dani Liberman <dliber...@habana.ai>

In struct cpucp_packet both nonce and data_max_size members are in an
union overlapping each other. This is a problem as they both are used
in attestation and info_signed packets.
The solution here is to move the nonce member to a different union under
the same structure.

Signed-off-by: Dani Liberman <dliber...@habana.ai>
Reviewed-by: Ofir Bitton <obit...@habana.ai>
---
 include/linux/habanalabs/cpucp_if.h | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/include/linux/habanalabs/cpucp_if.h 
b/include/linux/habanalabs/cpucp_if.h
index 1ac1d68193e3..0913415243e8 100644
--- a/include/linux/habanalabs/cpucp_if.h
+++ b/include/linux/habanalabs/cpucp_if.h
@@ -859,9 +859,6 @@ struct cpucp_packet {
                 * result cannot be used to hold general purpose data.
                 */
                __le32 status_mask;
-
-               /* random, used once number, for security packets */
-               __le32 nonce;
        };
 
        union {
@@ -870,6 +867,9 @@ struct cpucp_packet {
 
                /* For Generic packet sub index */
                __le32 pkt_subidx;
+
+               /* random, used once number, for security packets */
+               __le32 nonce;
        };
 };
 
-- 
2.34.1

Reply via email to