[dpdk-dev] [PATCH v3 1/4] build: port arm32 options from make to meson

[Ruifeng Wang]

> -----Original Message-----
> From: Juraj Linkeš <juraj.linkes at pantheon.tech>
> Sent: Thursday, April 16, 2020 8:22 PM
> To: Luca Boccassi <bluca at debian.org>; bruce.richardson at intel.com;
> aconole at redhat.com; maicolgabriel at hotmail.com; Ruifeng Wang
> <Ruifeng.Wang at arm.com>
> Cc: dev at dpdk.org
> Subject: RE: [dpdk-dev] [PATCH v3 1/4] build: port arm32 options from make
> to meson
> 
> 
> 
> > -----Original Message-----
> > From: Luca Boccassi <bluca at debian.org>
> > Sent: Thursday, April 16, 2020 2:01 PM
> > To: Juraj Linkeš <juraj.linkes at pantheon.tech>;
> > bruce.richardson at intel.com; aconole at redhat.com;
> > maicolgabriel at hotmail.com; Ruifeng.Wang at arm.com
> > Cc: dev at dpdk.org
> > Subject: Re: [dpdk-dev] [PATCH v3 1/4] build: port arm32 options from
> > make to meson
> >
> > On Thu, 2020-04-16 at 11:25 +0000, Juraj Linkeš wrote:
> > > > -----Original Message-----
> > > > From: Luca Boccassi <bluca at debian.org>
> > > > Sent: Thursday, April 16, 2020 10:53 AM
> > > > To: Juraj Linkeš <juraj.linkes at pantheon.tech>;
> > > > bruce.richardson at intel.com; aconole at redhat.com;
> > > > maicolgabriel at hotmail.com; Ruifeng.Wang at arm.com
> > > > Cc: dev at dpdk.org
> > > > Subject: Re: [dpdk-dev] [PATCH v3 1/4] build: port arm32 options
> > > > from make to meson
> > > >
> > > > On Thu, 2020-04-16 at 06:16 +0000, Juraj Linkeš wrote:
> > > > > Hi Luca,
> > > > >
> > > > > > -----Original Message-----
> > > > > > From: Luca Boccassi <bluca at debian.org>
> > > > > > Sent: Wednesday, April 15, 2020 1:45 PM
> > > > > > To: Juraj Linkeš <juraj.linkes at pantheon.tech>;
> > > > > > bruce.richardson at intel.com; aconole at redhat.com;
> > > > > > maicolgabriel at hotmail.com; Ruifeng.Wang at arm.com
> > > > > > Cc: dev at dpdk.org
> > > > > > Subject: Re: [dpdk-dev] [PATCH v3 1/4] build: port arm32
> > > > > > options from make to meson
> > > > > >
> > > > > > On Wed, 2020-04-15 at 11:03 +0200, Juraj Linkeš wrote:
> > > > > > > config/defconfig_arm-armv7a-linux-gcc contains a number of
> > > > > > > CONFIG_RTE_LIBRTE_ options. Implement those in applicable
> > > > > > > meson.build files.
> > > > > > >
> > > > > > > Signed-off-by: Juraj Linkeš <juraj.linkes at pantheon.tech>
> > > > > > > ---
> > > > > > >  drivers/net/ark/meson.build     | 6 ++++++
> > > > > > >  drivers/net/avp/meson.build     | 5 +++++
> > > > > > >  drivers/net/cxgbe/meson.build   | 6 ++++++
> > > > > > >  drivers/net/e1000/meson.build   | 6 ++++++
> > > > > > >  drivers/net/enic/meson.build    | 6 ++++++
> > > > > > >  drivers/net/fm10k/meson.build   | 6 ++++++
> > > > > > >  drivers/net/hinic/meson.build   | 6 ++++++
> > > > > > >  drivers/net/i40e/meson.build    | 6 ++++++
> > > > > > >  drivers/net/ionic/meson.build   | 6 ++++++
> > > > > > >  drivers/net/ixgbe/meson.build   | 6 ++++++
> > > > > > >  drivers/net/qede/meson.build    | 6 ++++++
> > > > > > >  drivers/net/vmxnet3/meson.build | 6 ++++++
> > > > > > >  lib/librte_vhost/meson.build    | 2 +-
> > > > > > >  13 files changed, 72 insertions(+), 1 deletion(-)
> > > > > >
> > > > > > This is disabling a lot of drivers that are building just fine
> > > > > > on
> > > > > > armv7 at the moment, and thus it is a backward-incomaptible
> change.
> > > > > >
> > > > > > Eg:
> > > > > >
> > > > > > https://packages.debian.org/sid/armhf/librte-pmd-cxgbe20.0/fil
> > > > > > el
> > > > > > ist
> > > > > >
> > > > > > --
> > > > > > Luca Boccassi <bluca at debian.org>
> > > > >
> > > > > I guess that means config/defconfig_arm-armv7a-linux-gcc is
> > > > > outdated? I've
> > > > tried to build all of the drivers and they build except for ixgbe.
> > > > Seeing as
> > > > https://packages.debian.org/sid/armhf/libs/librte-pmd-ixgbe20.0
> > > > exists, would disabling ixgbe be also a backwards incompatible change?
> > > > > Thanks,
> > > > > Juraj
> > > >
> > > > Yes. That defconfig is specific to the deprecated GNU Make build
> > > > system, which I haven't used in a long time. I do not know if
> > > > anybody keeps it
> > updated.
> > > >
> > > > What is the issue you are seeing with ixgbe?
> > >
> > > There are a lot of issues, please have a look here:
> > > https://travis-ci.com/github/jlinkes/dpdk/jobs/319965296#L1932
> > >
> > > Juraj
> >
> > Given it worked fine in 19.11, I'd say they are either regressions or
> > issues in the travis setup itself.
> 
> I'd wager it's because of
> https://github.com/DPDK/dpdk/commit/41b059a20215284e8aa618d2ce1e2a
> 803e64c1d0#diff-f44e12b03243963c21a46a37392568db
> 
> When I don't build with ixgbe_rxtx_vec_neon.c it builds fine, indicating real
> problems with the code.
> 
> Ruifeng, would you please take a look at the failures here? https://travis-
> ci.com/github/jlinkes/dpdk/jobs/319965296#L1932
> 
Yes. I see couple of problems with ixgbe_rxtx_vec_neon.c when building for AArch32. 
I will submit patches to fix these.

Thanks.
/Ruifeng
> Juraj