On 27.11.2019 8:11, Lance Yang (Arm Technology China) wrote: > >> -----Original Message----- >> From: Ilya Maximets <i.maxim...@ovn.org> >> Sent: Friday, November 22, 2019 6:36 PM >> To: Lance Yang (Arm Technology China) <lance.y...@arm.com>; Ilya Maximets >> <i.maxim...@ovn.org>; ovs-dev@openvswitch.org >> Cc: Jieqiang Wang (Arm Technology China) <jieqiang.w...@arm.com>; Gavin Hu >> (Arm >> Technology China) <gavin...@arm.com>; Jingzhao Ni (Arm Technology China) >> <jingzhao...@arm.com>; dwil...@us.ibm.com; nd <n...@arm.com>; Ruifeng Wang >> (Arm >> Technology China) <ruifeng.w...@arm.com>; Yanqin Wei (Arm Technology China) >> <yanqin....@arm.com> >> Subject: Re: [ovs-dev] [PATCH v1 4/4] travis: Enable OVS Travis CI for arm >> >> On 22.11.2019 10:35, Lance Yang (Arm Technology China) wrote: >>> Hi Ilya, >>> >>> Thanks for your reply. Please see the inline reply. >>> >>> Best regards, >>> Lance Yang >>> >>>> -----Original Message----- >>>> From: Ilya Maximets <i.maxim...@ovn.org> >>>> Sent: Friday, November 22, 2019 3:47 AM >>>> To: Lance Yang (Arm Technology China) <lance.y...@arm.com>; ovs- >>>> d...@openvswitch.org >>>> Cc: Jieqiang Wang (Arm Technology China) <jieqiang.w...@arm.com>; >>>> Gavin Hu (Arm Technology China) <gavin...@arm.com>; Jingzhao Ni (Arm >>>> Technology China) <jingzhao...@arm.com>; dwil...@us.ibm.com; nd >>>> <n...@arm.com>; Ruifeng Wang (Arm Technology China) >>>> <ruifeng.w...@arm.com> >>>> Subject: Re: [ovs-dev] [PATCH v1 4/4] travis: Enable OVS Travis CI >>>> for arm >>>> >>>> On 20.11.2019 9:15, Lance Yang wrote: >>>>> Enable part of travis jobs with gcc compiler for arm64 architecture >>>>> >>>>> 1. Add arm jobs into the matrix in .travis.yml and define dpdk cache >>>>> directory for arm64 architecture. >>>>> 2. Temporarily disable sparse checker because of static code >>>>> checking failure on arm64 >>>> >>>> Could you provide the sparse error messages? >>>> Maybe we could fix them instead of disabling. >>>> >>>>> >>>>> Successful travis build jobs report: >>>>> https://travis-ci.org/yzyuestc/ovs/builds/614299933 >>>> >>>> There are some issues in Travis in above build. It seems that Travis >>>> fails to upload the resulted cache directory due to missing md5deep: >>>> >>>> "" >>> [Lance] >>> The issue occurred because sparse checker does not recognize some arm neon >>> intrinsics: >>> >>> 1. Some components in Open vSwitch source code directly includes the header >>> file >> <arm_neon.h>. >>> >>> 2. When compiling OvS with DPDK, OvS includes some DPDK headers, which >>> sometimes includes <arm_neon.h>. You can see the details of the error >>> in the report at: https://travis-ci.org/yzyuestc/ovs/jobs/615414391 >>> after line 2693 >>> >>> For the first issue where OvS directly imported the arm_neon.h, I can fix >>> it in my patch set >> v2. However, for the second issue that DPDK project has imported the arm >> neon intrinsics, it >> seems that I am unable to fix this issue until DPDK solved it. >> >> Hi. >> >> Thanks for the information. >> Could, you, please, check if the following patch fixes the issue: >> https://patchwork.ozlabs.org/patch/1199398/ >> ? >> >>> >>>> store build cache >>>> 0.02s5.41sInstalling md5deep >>>> Reading package lists... >>>> Building dependency tree... >>>> Reading state information... >>>> md5deep is already the newest version (4.4-2). >>>> 0 upgraded, 0 newly installed, 0 to remove and 8 not upgraded. >>>> /home/travis/.casher/bin/casher: line 230: md5deep: command not found >>>> nothing changed "" >>>> >>> [Lance] >>> Thanks for pointing out the issue. The upload cache on arm is not >>> successful, that means arm jobs will always clone and build DPDK. I >>> just posted this issue to Travis CI community. You can see: >>> https://travis-ci.community/t/no-cache-support-on-arm64/5416/21 >> >> Thanks! Let's wait for their reply. >> >> Best regards, Ilya Maximets. > [Lance] > > Hi Ilya, > > It seems that travis CI has fixed the md5deep: command not found issue. You > can refer the link: > https://travis-ci.community/t/no-cache-support-on-arm64/5416/23?u=yzyuestc. > And if you check > https://travis-ci.community/t/no-cache-support-on-arm64/5416/26?u=yzyuestc, > you will find Travis has deployed it to production. > > I triggered a build and found it works, the report on line 5093: > https://travis-ci.org/yzyuestc/ovs/jobs/616991450#L5093 It shows the correct > directory has been uploaded successfully.
Cool. Thank you. Will you send a v2 of this series? Best regards, Ilya maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev