> -----Original Message-----
> From: Aaron Conole <acon...@redhat.com>
> Sent: Friday, January 10, 2020 23:13
> To: Ruifeng Wang <ruifeng.w...@arm.com>
> Cc: maicolgabr...@hotmail.com; tho...@monjalon.net;
> ferruh.yi...@intel.com; arybche...@solarflare.com; dev@dpdk.org;
> david.march...@redhat.com; Gavin Hu <gavin...@arm.com>; Honnappa
> Nagarahalli <honnappa.nagaraha...@arm.com>; nd <n...@arm.com>
> Subject: Re: [PATCH v4 2/2] ci: add travis ci support for aarch64
> 
> Ruifeng Wang <ruifeng.w...@arm.com> writes:
> 
> > Add Travis compilation jobs for aarch64. gcc/clang compilations for
> > static/shared libraries are added.
> >
> > Some limitations for current aarch64 Travis support:
> > 1. Container is used. Huge page is not available due to security reason.
> > 2. Missing kernel header package in Xenial distribution.
> >
> > Solutions to address the limitations:
> > 1. Not to add unit test for now. And run tests with no-huge in future.
> > 2. Use Bionic distribution for all aarch64 jobs.
> >
> > Signed-off-by: Ruifeng Wang <ruifeng.w...@arm.com>
> > Reviewed-by: Gavin Hu <gavin...@arm.com>
> > ---
> 
> I think the above text should probably include 'native' since there are 
> already
> aarch64 builds that go (but they are cross compiled).  I also am glad that you
> didn't remove them.
> 
Agree. 
I will submit new version with commit message update, and take your Ack.
Thank you.

> Acked-by: Aaron Conole <acon...@redhat.com>

Reply via email to