Hi Bruce, > Unfortunately, I've had to back this patch out again as it has caused errors > in the CI over the weekend e.g. [1]. No errors were reported with the patch > before I merged it, but I see now that not all tests were successfully run on > it. Can you please take a look at the CI errors reported and submit a new > version of this patch.
Thanks for reporting the issue! I didn't realise that some of the intrinsics, such as vzip1q/vzip2q, are only supported on AArch64. This explains the compilation errors we're seeing on AArch32 when cross-building DPDK for the AArch32 unit tests. I'll address this in the next revision by using vzip intrinsics to ensure compatibility across both AArch64 and AArch32. Best Regards, Jay Wang -----Original Message----- From: Bruce Richardson <[email protected]> Sent: Tuesday, April 7, 2026 11:47 AM To: Jay Wang <[email protected]> Cc: [email protected]; Anatoly Burakov <[email protected]>; [email protected]; nd <[email protected]> Subject: Re: [PATCH v2] net/ice: add NEON-optimised Rx/Tx vector paths On Fri, Apr 03, 2026 at 03:33:35PM +0100, Bruce Richardson wrote: > On Tue, Mar 24, 2026 at 04:25:24PM +0000, Jay Wang wrote: > > This patch adds the NEON-optimised Rx and Tx paths to the ice driver. > > > > Tested on Ampere One platform with Intel E810-C NIC and 100G connection. > > Tested with a single core and testpmd io forwarding mode. Observed > > ~30% performance boost in the above test compared to the default > > scalar path. > > > > Signed-off-by: Jay Wang <[email protected]> > > --- > > .mailmap | 1 + > > drivers/net/intel/ice/ice_ethdev.h | 3 + > > drivers/net/intel/ice/ice_rxtx.c | 53 +- > > drivers/net/intel/ice/ice_rxtx.h | 6 + > > drivers/net/intel/ice/ice_rxtx_vec_neon.c | 761 ++++++++++++++++++++++ > > drivers/net/intel/ice/meson.build | 2 + > > 6 files changed, 824 insertions(+), 2 deletions(-) create mode > > 100644 drivers/net/intel/ice/ice_rxtx_vec_neon.c > > > Acked-by: Bruce Richardson <[email protected]> > > Applied to dpdk-next-net-intel > Unfortunately, I've had to back this patch out again as it has caused errors in the CI over the weekend e.g. [1]. No errors were reported with the patch before I merged it, but I see now that not all tests were successfully run on it. Can you please take a look at the CI errors reported and submit a new version of this patch. Thanks, /Bruce [1] https://lab.dpdk.org/results/dashboard/testruns/logs/1985291/
