[v6,1/8] eal/x86: introduce AVX 512-bit type
Checks
Commit Message
New data type to manipulate 512 bit AVX values.
Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
---
lib/librte_eal/x86/include/rte_vect.h | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
Comments
On Mon, Jul 13, 2020 at 1:11 PM Vladimir Medvedkin
<vladimir.medvedkin@intel.com> wrote:
>
> New data type to manipulate 512 bit AVX values.
>
> Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
> ---
> lib/librte_eal/x86/include/rte_vect.h | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/lib/librte_eal/x86/include/rte_vect.h b/lib/librte_eal/x86/include/rte_vect.h
> index df5a60762..30dcfd5e7 100644
> --- a/lib/librte_eal/x86/include/rte_vect.h
> +++ b/lib/librte_eal/x86/include/rte_vect.h
> @@ -13,6 +13,7 @@
>
> #include <stdint.h>
> #include <rte_config.h>
> +#include <rte_common.h>
> #include "generic/rte_vect.h"
>
> #if (defined(__ICC) || \
> @@ -90,6 +91,24 @@ __extension__ ({ \
> })
> #endif /* (defined(__ICC) && __ICC < 1210) */
>
> +#ifdef __AVX512F__
> +
> +#define RTE_X86_ZMM_SIZE (sizeof(__m512i))
> +#define RTE_X86_ZMM_MASK (ZMM_SIZE - 1)
Please fix:
#define RTE_X86_ZMM_MASK (RTE_X86_ZMM_SIZE - 1)
> +
> +typedef union __rte_x86_zmm {
> + __m512i z;
> + ymm_t y[RTE_X86_ZMM_SIZE / sizeof(ymm_t)];
> + xmm_t x[RTE_X86_ZMM_SIZE / sizeof(xmm_t)];
> + uint8_t u8[RTE_X86_ZMM_SIZE / sizeof(uint8_t)];
> + uint16_t u16[RTE_X86_ZMM_SIZE / sizeof(uint16_t)];
> + uint32_t u32[RTE_X86_ZMM_SIZE / sizeof(uint32_t)];
> + uint64_t u64[RTE_X86_ZMM_SIZE / sizeof(uint64_t)];
> + double pd[RTE_X86_ZMM_SIZE / sizeof(double)];
> +} __rte_aligned(RTE_X86_ZMM_SIZE) __rte_x86_zmm_t;
> +
> +#endif /* __AVX512F__ */
> +
> #ifdef __cplusplus
> }
> #endif
> --
> 2.17.1
>
On 13/07/2020 12:33, David Marchand wrote:
> On Mon, Jul 13, 2020 at 1:11 PM Vladimir Medvedkin
> <vladimir.medvedkin@intel.com> wrote:
>>
>> New data type to manipulate 512 bit AVX values.
>>
>> Signed-off-by: Vladimir Medvedkin <vladimir.medvedkin@intel.com>
>> Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com>
>> ---
>> lib/librte_eal/x86/include/rte_vect.h | 19 +++++++++++++++++++
>> 1 file changed, 19 insertions(+)
>>
>> diff --git a/lib/librte_eal/x86/include/rte_vect.h b/lib/librte_eal/x86/include/rte_vect.h
>> index df5a60762..30dcfd5e7 100644
>> --- a/lib/librte_eal/x86/include/rte_vect.h
>> +++ b/lib/librte_eal/x86/include/rte_vect.h
>> @@ -13,6 +13,7 @@
>>
>> #include <stdint.h>
>> #include <rte_config.h>
>> +#include <rte_common.h>
>> #include "generic/rte_vect.h"
>>
>> #if (defined(__ICC) || \
>> @@ -90,6 +91,24 @@ __extension__ ({ \
>> })
>> #endif /* (defined(__ICC) && __ICC < 1210) */
>>
>> +#ifdef __AVX512F__
>> +
>> +#define RTE_X86_ZMM_SIZE (sizeof(__m512i))
>> +#define RTE_X86_ZMM_MASK (ZMM_SIZE - 1)
>
> Please fix:
> #define RTE_X86_ZMM_MASK (RTE_X86_ZMM_SIZE - 1)
>
Oh, thanks!
>
>> +
>> +typedef union __rte_x86_zmm {
>> + __m512i z;
>> + ymm_t y[RTE_X86_ZMM_SIZE / sizeof(ymm_t)];
>> + xmm_t x[RTE_X86_ZMM_SIZE / sizeof(xmm_t)];
>> + uint8_t u8[RTE_X86_ZMM_SIZE / sizeof(uint8_t)];
>> + uint16_t u16[RTE_X86_ZMM_SIZE / sizeof(uint16_t)];
>> + uint32_t u32[RTE_X86_ZMM_SIZE / sizeof(uint32_t)];
>> + uint64_t u64[RTE_X86_ZMM_SIZE / sizeof(uint64_t)];
>> + double pd[RTE_X86_ZMM_SIZE / sizeof(double)];
>> +} __rte_aligned(RTE_X86_ZMM_SIZE) __rte_x86_zmm_t;
>> +
>> +#endif /* __AVX512F__ */
>> +
>> #ifdef __cplusplus
>> }
>> #endif
>> --
>> 2.17.1
>>
>
@@ -13,6 +13,7 @@
#include <stdint.h>
#include <rte_config.h>
+#include <rte_common.h>
#include "generic/rte_vect.h"
#if (defined(__ICC) || \
@@ -90,6 +91,24 @@ __extension__ ({ \
})
#endif /* (defined(__ICC) && __ICC < 1210) */
+#ifdef __AVX512F__
+
+#define RTE_X86_ZMM_SIZE (sizeof(__m512i))
+#define RTE_X86_ZMM_MASK (ZMM_SIZE - 1)
+
+typedef union __rte_x86_zmm {
+ __m512i z;
+ ymm_t y[RTE_X86_ZMM_SIZE / sizeof(ymm_t)];
+ xmm_t x[RTE_X86_ZMM_SIZE / sizeof(xmm_t)];
+ uint8_t u8[RTE_X86_ZMM_SIZE / sizeof(uint8_t)];
+ uint16_t u16[RTE_X86_ZMM_SIZE / sizeof(uint16_t)];
+ uint32_t u32[RTE_X86_ZMM_SIZE / sizeof(uint32_t)];
+ uint64_t u64[RTE_X86_ZMM_SIZE / sizeof(uint64_t)];
+ double pd[RTE_X86_ZMM_SIZE / sizeof(double)];
+} __rte_aligned(RTE_X86_ZMM_SIZE) __rte_x86_zmm_t;
+
+#endif /* __AVX512F__ */
+
#ifdef __cplusplus
}
#endif