> Hi Ian, > > Pls find the separated patch for DPCLS at : > http://patchwork.ozlabs.org/project/openvswitch/patch/20210629164941.1563 > [email protected]/ > > Regards > Amber
Just spotted it, thanks. Regards Ian > > > -----Original Message----- > > From: Van Haaren, Harry <[email protected]> > > Sent: Tuesday, June 29, 2021 10:16 PM > > To: Stokes, Ian <[email protected]>; Amber, Kumar > > <[email protected]>; [email protected] > > Cc: [email protected]; Ferriter, Cian <[email protected]> > > Subject: RE: [ovs-dev] [v4 12/12] dpif/dpcls: limit count subtable search > > info > > logs > > > > > -----Original Message----- > > > From: Stokes, Ian <[email protected]> > > > Sent: Tuesday, June 29, 2021 5:40 PM > > > To: Amber, Kumar <[email protected]>; [email protected] > > > Cc: [email protected]; Ferriter, Cian <[email protected]>; Van > > > Haaren, Harry <[email protected]> > > > Subject: RE: [ovs-dev] [v4 12/12] dpif/dpcls: limit count subtable > > > search info logs > > > > > > > From: Harry van Haaren <[email protected]> > > > > > > > > This commit avoids many instances of "using subtable X for miniflow > > (x,y)" > > > > in the ovs-vswitchd log when using the DPCLS Autovalidator. This > > > > occurs when no specialized subtable is found, and the generic "_any" > > > > version of the avx512 subtable search implementation was used. This > > > > change logs the subtable usage once, avoiding duplicates. > > > > > > > > > > Good point here, I think people forget there is a cost to logs and no > > > need to flood them. > > > > > > Just to confirm, I think this log is already upstream? What I mean is > > > that it is not added by either the DPIF or MFEX patch series so this > > > is the earliest we can make the change on it? > > > > This change can be made earlier. The logs spam gets worse if we use the > > autovalidator, so it was identified as an issue to fix when testing with > > MFEX > > autovalidator && DPCLS autovalidator, hence why here in the patchset. > > > > Can submit separately if preferred. > > > > > > > Regards > > > Ian > > > > Thanks for review, -Harry > > > > <snip patch contents> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
