* [dpdk-dev] [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 ` [dpdk-dev] [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 [flat|nested] 5+ messages in thread* Re: [dpdk-dev] [PATCH] test: support to check AVX512F
2017-09-08 7:44 [dpdk-dev] [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 ` [dpdk-dev] [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: [dpdk-dev] [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
* [dpdk-dev] [PATCH v2] test: add check for AVX512F
2017-09-08 7:44 [dpdk-dev] [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 [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 [dpdk-dev] [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 ` [dpdk-dev] [PATCH v2] test: add check for AVX512F Zhiyong Yang
2017-10-09 14:40 ` Thomas Monjalon
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).