> -----Original Message----- > From: William Tu <[email protected]> > Sent: Saturday, July 4, 2020 10:18 PM > To: Van Haaren, Harry <[email protected]> > Cc: ovs-dev <[email protected]>; Stokes, Ian <[email protected]>; > Ilya > Maximets <[email protected]>; Federico Iezzi <[email protected]> > Subject: Re: [PATCH v5 0/6] DPCLS Subtable ISA Optimization > > On Thu, Jul 2, 2020 at 8:18 AM Van Haaren, Harry > <[email protected]> wrote: > > > > > -----Original Message----- > > > From: William Tu <[email protected]> > > > Sent: Wednesday, July 1, 2020 10:53 PM > > > To: Van Haaren, Harry <[email protected]> > > > Cc: ovs-dev <[email protected]>; Stokes, Ian <[email protected]>; > Ilya > > > Maximets <[email protected]>; Federico Iezzi <[email protected]> > > > Subject: Re: [PATCH v5 0/6] DPCLS Subtable ISA Optimization > > > > > > On Wed, Jul 1, 2020 at 7:22 AM Harry van Haaren > > > <[email protected]> wrote: > > > > > > > > v5 work done: > > > > - Add NEWS section > > > > - Integrate v4 patches 5 and 6 into a single commit > > > > - Address v4 feedback (details in each patch) > > > > -- Fixed multiple travis build issues/warnings > > > > -- Improved APIs/code, refactored #ifdefs > > > > > > > > @William Tu: > > > > Thanks for pushing v4 commits to travis for testing, and reporting back. > > > > There are 2 items that I could not reproduce, and could not reliably > > > > state they are real issues, or 2nd order issues. This v5 has addressed > > > > the actual errors from Travis output, there is a chance these issues > > > > are not resolved by the surrounding fixes: > > > > 1) RTE_CPUFLAG_AVX512F flag not defined in a DPDK=1 build? > > > > 2) Binutils check for as with --64 parameter is invalid on older as? > > > > > > > > Would you re-push this branch and we can verify? Thanks in advance! > > > Hi Harry, > > > This is the v5 on travis. > > > https://travis-ci.org/github/williamtu/ovs-travis/builds/704102981 > > > William > > > > Thanks William! > > > > CPUFLAG was a cross-build issue, resolved with a #ifdef __x86_64__ > > Binutils check has been fixed with a simple rm during configure time > > > > Working on the remaining 2 known issues, and will push a v6 ASAP once > > resolved. > > > > You asked about the "make check-system-userspace" command on the v4, > > I see this failing on ovs-master, and (still) failing with the AVX512 > > patchset applied. > > > Hi Harry, > > I checked your v6 using "make check-system-userspace" (with > autovalidtor enabled) > and it works OK! I will continue looking at your v6.
Ah great, thanks for checking. It seems my system is somehow misconfigured to run those tests in that case, I'll investigate that. > Regards, > William Cheers, -Harry <snip> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
