[dpdk-dev] [PATCH v2] net/virtio: fix AVX512 datapath selection
Liu, Yong
yong.liu at intel.com
Tue May 12 05:29:01 CEST 2020
> -----Original Message-----
> From: Maxime Coquelin <maxime.coquelin at redhat.com>
> Sent: Tuesday, May 12, 2020 3:50 AM
> To: Yigit, Ferruh <ferruh.yigit at intel.com>; Wang, Zhihong
> <zhihong.wang at intel.com>; Ye, Xiaolong <xiaolong.ye at intel.com>; Liu,
> Yong <yong.liu at intel.com>
> Cc: dev at dpdk.org; Thomas Monjalon <thomas at monjalon.net>; David
> Marchand <david.marchand at redhat.com>; Richardson, Bruce
> <bruce.richardson at intel.com>; Nicolau, Radu <radu.nicolau at intel.com>;
> Luca Boccassi <bluca at debian.org>
> Subject: Re: [PATCH v2] net/virtio: fix AVX512 datapath selection
>
>
>
> On 5/11/20 8:48 PM, Ferruh Yigit wrote:
> > From: Maxime Coquelin <maxime.coquelin at redhat.com>
> >
> > The AVX512 packed ring datapath selection was only done
> > at build time, but it should also be checked at runtime
> > that the CPU supports it.
> >
> > This patch add a CPU flags check so that non-vectorized
> > path is selected at runtime if AVX512 is not supported.
> >
> > Also in meson build enable vectorization only for relevant file, not for
> > all driver.
> >
> > Fixes: ccb10995c2ad ("net/virtio: add election for vectorized path")
> >
> > Signed-off-by: Maxime Coquelin <maxime.coquelin at redhat.com>
> > Signed-off-by: Ferruh Yigit <ferruh.yigit at intel.com>
> > ---
> > Cc: Bruce Richardson <bruce.richardson at intel.com>
> > Cc: Radu Nicolau <radu.nicolau at intel.com>
> > Cc: Luca Boccassi <bluca at debian.org>
> >
> > For meson I mainly adapted implementation from other driver, not able
> to
> > test or verify myself.
> > ---
> > drivers/net/virtio/meson.build | 9 +++++++--
> > drivers/net/virtio/virtio_ethdev.c | 6 ++++--
> > 2 files changed, 11 insertions(+), 4 deletions(-)
>
> Thanks Ferruh, I cannot test either right now but it looks good to me:
>
Hi Maxime & Ferruh,
IMHO, meson build update is the essential part for fixing unexpected AVX512 instructions.
Change in virtio_ethdev may cause building issues on ppc and arm platform. Is it convenient to revert that change?
Regards,
Marvin
> In case you're waiting for it:
> Acked-by: Maxime Coquelin <maxime.coquelin at redhat.com>
>
> Maxime
More information about the dev
mailing list