On 5/11/2020 8:49 PM, Maxime Coquelin wrote: > > > On 5/11/20 8:48 PM, Ferruh Yigit wrote: >> From: Maxime Coquelin <maxime.coque...@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.coque...@redhat.com> >> Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> >> --- >> Cc: Bruce Richardson <bruce.richard...@intel.com> >> Cc: Radu Nicolau <radu.nico...@intel.com> >> Cc: Luca Boccassi <bl...@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: > > In case you're waiting for it: > Acked-by: Maxime Coquelin <maxime.coque...@redhat.com> >
Applied to dpdk-next-net/master, thanks.