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>
Maxime