All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] test: support to check AVX512F
@ 2017-09-08  7:44 Zhiyong Yang
  2017-09-18 15:07 ` Bruce Richardson
  2017-09-19  2:25 ` [PATCH v2] test: add check for AVX512F Zhiyong Yang
  0 siblings, 2 replies; 5+ messages in thread
From: Zhiyong Yang @ 2017-09-08  7:44 UTC (permalink / raw)
  To: dev; +Cc: 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

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [PATCH] test: support to check AVX512F
  2017-09-08  7:44 [PATCH] test: support to check AVX512F Zhiyong Yang
@ 2017-09-18 15:07 ` Bruce Richardson
  2017-09-19  2:12   ` Yang, Zhiyong
  2017-09-19  2:25 ` [PATCH v2] test: add check for AVX512F Zhiyong Yang
  1 sibling, 1 reply; 5+ messages in thread
From: Bruce Richardson @ 2017-09-18 15:07 UTC (permalink / raw)
  To: Zhiyong Yang; +Cc: dev

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>

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH] test: support to check AVX512F
  2017-09-18 15:07 ` Bruce Richardson
@ 2017-09-19  2:12   ` Yang, Zhiyong
  0 siblings, 0 replies; 5+ messages in thread
From: Yang, Zhiyong @ 2017-09-19  2:12 UTC (permalink / raw)
  To: Richardson, Bruce; +Cc: dev

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

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [PATCH v2] test: add check for AVX512F
  2017-09-08  7:44 [PATCH] test: support to check AVX512F Zhiyong Yang
  2017-09-18 15:07 ` Bruce Richardson
@ 2017-09-19  2:25 ` Zhiyong Yang
  2017-10-09 14:40   ` Thomas Monjalon
  1 sibling, 1 reply; 5+ messages in thread
From: Zhiyong Yang @ 2017-09-19  2:25 UTC (permalink / raw)
  To: dev; +Cc: bruce.richardson, 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

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [PATCH v2] test: add check for AVX512F
  2017-09-19  2:25 ` [PATCH v2] test: add check for AVX512F Zhiyong Yang
@ 2017-10-09 14:40   ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2017-10-09 14:40 UTC (permalink / raw)
  To: Zhiyong Yang; +Cc: dev, bruce.richardson

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

^ permalink raw reply	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-10-09 14:40 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-08  7:44 [PATCH] test: support to check AVX512F Zhiyong Yang
2017-09-18 15:07 ` Bruce Richardson
2017-09-19  2:12   ` Yang, Zhiyong
2017-09-19  2:25 ` [PATCH v2] test: add check for AVX512F Zhiyong Yang
2017-10-09 14:40   ` Thomas Monjalon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.