Thanks a lot Eelco. MFEX v7 will be available shorty and I have addressed all the reviews comments hopefully.
Br Amber > -----Original Message----- > From: Eelco Chaudron <[email protected]> > Sent: Thursday, July 8, 2021 6:03 PM > To: Ferriter, Cian <[email protected]> > Cc: [email protected]; [email protected]; [email protected]; Van > Haaren, Harry <[email protected]>; Amber, Kumar > <[email protected]>; Stokes, Ian <[email protected]> > Subject: Re: [v6 11/11] dpif-netdev/mfex: add more AVX512 traffic profiles > > > > On 6 Jul 2021, at 15:11, Cian Ferriter wrote: > > > From: Harry van Haaren <[email protected]> > > > > This commit adds 3 new traffic profile implementations to the existing > > avx512 miniflow extract infrastructure. The profiles added are: > > - Ether()/IP()/TCP() > > - Ether()/Dot1Q()/IP()/UDP() > > - Ether()/Dot1Q()/IP()/TCP() > > > > The design of the avx512 code here is for scalability to add more > > traffic profiles, as well as enabling CPU ISA. Note that an > > implementation is primarily adding static const data, which the > > compiler then specializes away when the profile specific function is > > declared > below. > > > > As a result, the code is relatively maintainable, and scalable for new > > traffic profiles as well as new ISA, and does not lower performance > > compared with manually written code for each profile/ISA. > > > > Note that confidence in the correctness of each implementation is > > achieved through autovalidation, unit tests with known packets, and > > fuzz tested packets. > > > > Signed-off-by: Harry van Haaren <[email protected]> > > These changes look good to me! > > This concludes my review of v6. I will get my system ready to do some > performance runs, and hopefully, I can get some results tomorrow or Monday. > > Acked-by: Eelco Chaudron <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
