Ilias Apalodimas(apalos) replied on github web page:

platform/linux-generic/include/odp_ipsec_internal.h
line 5
@@ -92,7 +92,7 @@ int _odp_ipsec_status_send(odp_queue_t queue,
 #define ODP_CONFIG_IPSEC_SAS   8
 
 struct ipsec_sa_s {
-       odp_atomic_u32_t state ODP_ALIGNED_CACHE;
+       odp_atomic_u32_t ODP_ALIGNED_CACHE state;


Comment:
The real bug was only present on typedef of structs/unions since the attribute 
was ignored if  placed after the terminating }.
typedef unsigned long long __u64 __attribute__((aligned(16))); 
should be equal to 
typedef unsigned long long  __attribute__((aligned(16))) __u64;

The rest were changed to have a unified approach

> Petri Savolainen(psavol) wrote:
> The new attribute placement rule was for structs, union, etc, but how about 
> integers ?


https://github.com/Linaro/odp/pull/398#discussion_r162366329
updated_at 2018-01-18 15:03:10

Reply via email to