Hi Eelco, Flavio, Thanks a lot for all the comments these would surely avoid bugs coming in future . We have more MFEX patches in stores hope to see your review on them as well in Future 😊
BR Amber > -----Original Message----- > From: Eelco Chaudron <[email protected]> > Sent: Friday, July 16, 2021 2:52 PM > To: Stokes, Ian <[email protected]> > Cc: Flavio Leitner <[email protected]>; Amber, Kumar > <[email protected]>; [email protected]; [email protected]; > Van Haaren, Harry <[email protected]>; Ferriter, Cian > <[email protected]> > Subject: Re: [PATCH v14 10/11] dpif-netdev/mfex: Add AVX512 based optimized > miniflow extract > > > > On 16 Jul 2021, at 10:49, Stokes, Ian wrote: > > >> On Thu, Jul 15, 2021 at 09:36:16PM +0530, kumar Amber wrote: > >>> From: Harry van Haaren <[email protected]> > >>> > >>> This commit adds AVX512 implementations of miniflow extract. > >>> By using the 64 bytes available in an AVX512 register, it is > >>> possible to convert a packet to a miniflow data-structure in a small > >>> quantity instructions. > >>> > >>> The implementation here probes for Ether()/IP()/UDP() traffic, and > >>> builds the appropriate miniflow data-structure for packets that > >>> match the probe. > >>> > >>> The implementation here is auto-validated by the miniflow extract > >>> autovalidator, hence its correctness can be easily tested and > >>> verified. > >>> > >>> Note that this commit is designed to easily allow addition of new > >>> traffic profiles in a scalable way, without code duplication for > >>> each traffic profile. > >>> > >>> Signed-off-by: Harry van Haaren <[email protected]> > >>> Acked-by: Eelco Chaudron <[email protected]> > >>> > >>> --- > >> > >> Acked-by: Flavio Leitner <[email protected]> > > > > Thanks Flavio, > > > > @[email protected] are these changes OK with you and can we add an > ACK? (sorry to push on this, given the deadline today to branch we'd like to > merge, there are a few other patch series I'd like to rebase on it and merge > also > so would be great to close this off if there are no objections). > > > > Thanks > > Ian > > Hi Ian, > > Only some comments on patches 3 and 6, see comments in the respective patch > email. > > If you do fix them on merge, make sure the indentation changes do not make it > grow over 80 chars. > > Cheers, > > Eelco _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
