[PATCH] test: support to check AVX512F

[PATCH] test: support to check AVX512F

[Zhiyong Yang]

The CPUs which support AVX512 have been released. Add the support
of checking AVX512F instruction set.

Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
---
 test/test/test_cpuflags.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/test/test/test_cpuflags.c b/test/test/test_cpuflags.c
index 0e5ebe788..08c166139 100644
--- a/test/test/test_cpuflags.c
+++ b/test/test/test_cpuflags.c
@@ -171,6 +171,9 @@ test_cpuflags(void)
 	printf("Check for AVX2:\t\t");
 	CHECK_FOR_FLAG(RTE_CPUFLAG_AVX2);
 
+	printf("Check for AVX512F:\t");
+	CHECK_FOR_FLAG(RTE_CPUFLAG_AVX512F);
+
 	printf("Check for TRBOBST:\t");
 	CHECK_FOR_FLAG(RTE_CPUFLAG_TRBOBST);
 
-- 
2.13.3

Re: [PATCH] test: support to check AVX512F

[Bruce Richardson]

suggest title as: test: add check for AVX512F

On Fri, Sep 08, 2017 at 03:44:54PM +0800, Zhiyong Yang wrote:
> The CPUs which support AVX512 have been released. Add the support
> of checking AVX512F instruction set.
>
s/the support of/support for/

> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> ---
Simple enough patch which adds one more check to the test. Probably
worth having. Reports the presence of the flag correctly on my board.

With commit message rewording:

Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Re: [PATCH] test: support to check AVX512F

[Yang, Zhiyong]

Hi Bruce,

> -----Original Message-----
> From: Richardson, Bruce
> Sent: Monday, September 18, 2017 11:08 PM
> To: Yang, Zhiyong <zhiyong.yang@intel.com>
> Cc: dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] test: support to check AVX512F
> 
> suggest title as: test: add check for AVX512F
> 
> On Fri, Sep 08, 2017 at 03:44:54PM +0800, Zhiyong Yang wrote:
> > The CPUs which support AVX512 have been released. Add the support of
> > checking AVX512F instruction set.
> >
> s/the support of/support for/
> 
> > Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> > ---
> Simple enough patch which adds one more check to the test. Probably worth
> having. Reports the presence of the flag correctly on my board.
> 
> With commit message rewording:
> 
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Thanks for your reviews , I will send V2 out sooner according to your comments.

Thanks
Zhiyong

[PATCH v2] test: add check for AVX512F

[Zhiyong Yang]

The CPUs which support AVX512 have been released. Add support for
checking AVX512F instruction set.

Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
---

Changes in V2:
reword the title and commit message according to Bruce's comments.

 test/test/test_cpuflags.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/test/test/test_cpuflags.c b/test/test/test_cpuflags.c
index 0e5ebe788..08c166139 100644
--- a/test/test/test_cpuflags.c
+++ b/test/test/test_cpuflags.c
@@ -171,6 +171,9 @@ test_cpuflags(void)
 	printf("Check for AVX2:\t\t");
 	CHECK_FOR_FLAG(RTE_CPUFLAG_AVX2);
 
+	printf("Check for AVX512F:\t");
+	CHECK_FOR_FLAG(RTE_CPUFLAG_AVX512F);
+
 	printf("Check for TRBOBST:\t");
 	CHECK_FOR_FLAG(RTE_CPUFLAG_TRBOBST);
 
-- 
2.13.3

Re: [PATCH v2] test: add check for AVX512F

[Thomas Monjalon]

19/09/2017 04:25, Zhiyong Yang:
> The CPUs which support AVX512 have been released. Add support for
> checking AVX512F instruction set.
> 
> Signed-off-by: Zhiyong Yang <zhiyong.yang@intel.com>
> Acked-by: Bruce Richardson <bruce.richardson@intel.com>

Applied, thanks