I just did it: https://gerrit.fd.io/r/#/c/14711/
-- Damjan > On 7 Sep 2018, at 09:25, Damjan Marion via Lists.Fd.Io > <[email protected]> wrote: > > > Dear Kingwei, > > That should be easy to fix. Can you just remove it and submit patch, so i can > merge it? > > Thanks, > > -- > Damjan > >> On 7 Sep 2018, at 02:39, Kingwel Xie <[email protected] >> <mailto:[email protected]>> wrote: >> >> Hi Damijan, >> >> Thanks for the great job done. It is now much faster. >> >> We noticed a difference between using cmake and automake in the latest code: >> >> Vppinfra/qsort.c is included in vppinfra/CMakefile.txt but not in >> vppinfra.am, which creates a situation that cmake image would be linked to >> the qsort.c but automake linked to glibc. >> >> The reason why we noticed is there is a buffer overrun bug in qsort.c that >> causes cli crashed occasionally. >> >> Please comment how to fix. Personally I’d like to remove qsort.c, like >> before. >> >> Regards, >> Kingwel >> >> From: [email protected] <mailto:[email protected]> <[email protected] >> <mailto:[email protected]>> On Behalf Of Damjan Marion via Lists.Fd.Io >> Sent: Sunday, September 02, 2018 8:48 PM >> To: vpp-dev <[email protected] <mailto:[email protected]>> >> Cc: [email protected] <mailto:[email protected]> >> Subject: [vpp-dev] cmake is on >> >> >> Dear all, >> >> We just switched from autotools to cmake and retired all autotools related >> files in src/. >> >> All verify jobs are ok, and we also tried it on 3 different x86 and 2 >> different ARM Aarch64 machines. >> >> Due to blast radius, i will not be surprised that some small issues pop out, >> but i don't expect anything hard to fix. >> >> Let us know if you hit something... >> >> PS As a part of this change, CentOS 7 build are now using devtoolset-7, so >> they are compiled with gcc-7, which also means images have support for >> Skylake Servers (AVX512). >> >> -- >> Damjan >> >> -=-=-=-=-=-=-=-=-=-=-=- >> Links: You receive all messages sent to this group. >> >> View/Reply Online (#10420): https://lists.fd.io/g/vpp-dev/message/10420 >> <https://lists.fd.io/g/vpp-dev/message/10420> >> Mute This Topic: https://lists.fd.io/mt/25155374/675642 >> <https://lists.fd.io/mt/25155374/675642> >> Group Owner: [email protected] <mailto:[email protected]> >> Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub >> <https://lists.fd.io/g/vpp-dev/unsub> [[email protected] >> <mailto:[email protected]>] >> -=-=-=-=-=-=-=-=-=-=-=- > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > > View/Reply Online (#10421): https://lists.fd.io/g/vpp-dev/message/10421 > Mute This Topic: https://lists.fd.io/mt/25155374/675642 > Group Owner: [email protected] > Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [[email protected]] > -=-=-=-=-=-=-=-=-=-=-=-
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#10423): https://lists.fd.io/g/vpp-dev/message/10423 Mute This Topic: https://lists.fd.io/mt/25155374/21656 Group Owner: [email protected] Unsubscribe: https://lists.fd.io/g/vpp-dev/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
