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

[Lance Yang (Arm Technology China)]

> -----Original Message-----
> From: Ilya Maximets <i.maxim...@ovn.org>
> Sent: Friday, November 29, 2019 3:06 AM
> 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 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.
[Lance]

Yes. Once we update for v2, we will send you the series.

All the best.
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.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev