Hi Fan, With my release manager hat on:
the first three patches are solely contained (minus seemingly whitespace change in 34660? can it be avoided ?) within wireguard plugin, which has "experimental" status, with which I would be happy to err on the side of keeping the velocity - so once the nit i pointed out is taken care of, I would be happy to merge the cherry-picks into the stable/2202, provided it is "soon" for some very proximate value of "soon" :) the last patch to crypto-dev is an improvement, and looks fairly straightforward addition, backwards compatible, and was +2'd by Damjan back in December. So, from the "technical" standpoint it should have been in, but there was new year break period, etc, etc. So I would again would be fine with merging the cherry-pick into stable/2202, but for this one" the value of "soon" should not exceed "end of this week". In case no-one from the community objects to the above by this Friday 12:00 UTC, let's get these as cherry-picks into stable/2202. (p.s.: the 22.02 RC1 is not tagged yet, but the v22.06-rc0 tag is already on master, I would rather not mess with that). --a On 1/19/22, Fan Zhang <roy.fan.zh...@intel.com> wrote: > Hi, > > Sorry for the late notice, but we have a bunch of patches waiting to be > reviewed/merged if possible for VPP22.02. > > The patches do 2 things > > * Optimizing wireguard performance by introducing burst processing of > packets and chacha-poly encryption/decryption. > * Adding async mode to wireguard so the crypto can be processed by > sw-crypto-scheduler or QAT. > > They are > https://gerrit.fd.io/r/c/vpp/+/34324 > https://gerrit.fd.io/r/c/vpp/+/34660/2 > https://gerrit.fd.io/r/c/vpp/+/34661/2 > https://gerrit.fd.io/r/c/vpp/+/34662/4 > > We know the patches are not perfect - as Matthew already pointed out there > are some improvements can be done (thank you very much Matthew!). > The bottom line is > > * They passed the wireguard unit tests, and we will keep improving > them. > * The code change had >5% performance improvement with small packets. > > But are they too late for VPP22.02? Much appreciate for the help in > advance! > > Regards, > fan > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#20758): https://lists.fd.io/g/vpp-dev/message/20758 Mute This Topic: https://lists.fd.io/mt/88537247/21656 Group Owner: vpp-dev+ow...@lists.fd.io Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
