> -----Original Message-----
> From: Amber, Kumar <[email protected]>
> Sent: Thursday 26 May 2022 11:09
> To: [email protected]
> Cc: [email protected]; [email protected]; Ferriter, Cian 
> <[email protected]>; Stokes, Ian
> <[email protected]>; [email protected]; [email protected]; Van 
> Haaren, Harry
> <[email protected]>; Amber, Kumar <[email protected]>
> Subject: [PATCH v8 3/4] dpif-netdev/mfex: Add AVX512 ipv6 traffic profiles
> 
> Add AVX512 Ipv6 optimized profile for vlan/IPv6/UDP and
> vlan/IPv6/TCP, IPv6/UDP and IPv6/TCP.
> 
> MFEX autovalidaton test-case already has the IPv6 support for
> validating against the scalar mfex.
> 
> Signed-off-by: Kumar Amber <[email protected]>
> Signed-off-by: Harry van Haaren <[email protected]>
> Co-authored-by: Harry van Haaren <[email protected]>
> 
> ---
> v8:
> - Rename defines for packet offsets.
> v7:
> - Fix Lenght checks for plen.
> v5:
> - Add variable length checks for IPv6 and TCP.
> v4:
> - Rebase to master.
> v2:
> - Fix CI build error.
> - Fix check-patch sign-offs.
> ---

Hi Amber,

Thanks for fixing the length checks, the minimum IPV6 packet test I was running 
before which was failing, is now passing, the length checks look good to me.

I'm seeing ~20% improvement in performance for a simple P2P with IPV6 packets
Before:
AVX512 DPIF + DPCLS and Scalar MFEX

After:
AVX512 DPIF + MFEX + DPCLS

I'm happy to Ack the patch:
Acked-by: Cian Ferriter <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to