Hi Maxime, > -----Original Message----- > From: Maxime Coquelin <maxime.coque...@redhat.com> > Sent: Monday, October 5, 2020 3:34 PM > To: Joyce Kong <joyce.k...@arm.com> > Cc: jer...@marvell.com; dev@dpdk.org; nd <n...@arm.com>; Honnappa > Nagarahalli <honnappa.nagaraha...@arm.com>; Ruifeng Wang > <ruifeng.w...@arm.com>; Phil Yang <phil.y...@arm.com> > Subject: Re: [RFC 0/3] Vectorize packed ring RX path with NEON > > 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 >
Thanks for your comments, and I will send the new revision soon. > > > 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 > >
