[ovs-dev] [PATCH v1 4/4] travis: Enable OVS Travis CI for arm

Lance Yang (Arm Technology China) Lance.Yang at arm.com
Wed Nov 27 07:11:47 UTC 2019


> -----Original Message-----
> From: Ilya Maximets <i.maximets at ovn.org>
> Sent: Friday, November 22, 2019 6:36 PM
> To: Lance Yang (Arm Technology China) <Lance.Yang at arm.com>; Ilya Maximets
> <i.maximets at ovn.org>; ovs-dev at openvswitch.org
> Cc: Jieqiang Wang (Arm Technology China) <Jieqiang.Wang at arm.com>; Gavin Hu (Arm
> Technology China) <Gavin.Hu at arm.com>; Jingzhao Ni (Arm Technology China)
> <Jingzhao.Ni at arm.com>; dwilder at us.ibm.com; nd <nd at arm.com>; Ruifeng Wang (Arm
> Technology China) <Ruifeng.Wang at arm.com>; Yanqin Wei (Arm Technology China)
> <Yanqin.Wei at 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.maximets at ovn.org>
> >> Sent: Friday, November 22, 2019 3:47 AM
> >> To: Lance Yang (Arm Technology China) <Lance.Yang at arm.com>; ovs-
> >> dev at openvswitch.org
> >> Cc: Jieqiang Wang (Arm Technology China) <Jieqiang.Wang at arm.com>;
> >> Gavin Hu (Arm Technology China) <Gavin.Hu at arm.com>; Jingzhao Ni (Arm
> >> Technology China) <Jingzhao.Ni at arm.com>; dwilder at us.ibm.com; nd
> >> <nd at arm.com>; Ruifeng Wang (Arm Technology China)
> >> <Ruifeng.Wang at 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.

Best regards,
Lance
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


More information about the dev mailing list