Hi Joyce, On 9/11/20 2:09 PM, Joyce Kong wrote: > This patch set introduced vectorized RX path for packed ring > with NEON.
Overall, the series looks good. I will have to trust you on the NEON implementation. Ideally, having a detailed review from someone from ARM would be good. Could you please send a new revision with the build issues reported by Intel CI fixed? http://mails.dpdk.org/archives/test-report/2020-September/152501.html Thanks, Maxime > Joyce Kong (3): > net/virtio: move AVX based Rx and Tx code to separate file > net/virtio: add vectorized packed ring Rx NEON path > net/virtio: add election for packed vector Rx NEON path > > doc/guides/nics/virtio.rst | 4 +- > drivers/net/virtio/meson.build | 2 + > drivers/net/virtio/virtio_ethdev.c | 19 +- > drivers/net/virtio/virtio_rxtx.c | 7 +- > drivers/net/virtio/virtio_rxtx_packed.c | 37 +++ > drivers/net/virtio/virtio_rxtx_packed.h | 300 +++++++++++++++++++ > drivers/net/virtio/virtio_rxtx_packed_avx.c | 264 +--------------- > drivers/net/virtio/virtio_rxtx_packed_neon.c | 202 +++++++++++++ > drivers/net/virtio/virtio_user_ethdev.c | 2 + > 9 files changed, 566 insertions(+), 271 deletions(-) > create mode 100644 drivers/net/virtio/virtio_rxtx_packed.c > create mode 100644 drivers/net/virtio/virtio_rxtx_packed.h > create mode 100644 drivers/net/virtio/virtio_rxtx_packed_neon.c >
