[dpdk-dev] [EXT] [PATCH 0/6] build: fix build for arm64

Jerin Jacob Kollanukkaran jerinj at marvell.com
Tue Apr 16 07:57:58 CEST 2019



> -----Original Message-----
> From: Yongseok Koh <yskoh at mellanox.com>
> Sent: Tuesday, April 16, 2019 2:26 AM
> To: Jerin Jacob Kollanukkaran <jerinj at marvell.com>
> Cc: bruce.richardson at intel.com; Pavan Nikhilesh Bhagavatula
> <pbhagavatula at marvell.com>; Shahaf Shuler <shahafs at mellanox.com>;
> dev at dpdk.org; Thomas Monjalon <thomas at monjalon.net>;
> gavin.hu at arm.com; Honnappa.Nagarahalli at arm.com
> Subject: Re: [EXT] [PATCH 0/6] build: fix build for arm64
> 
> 
> > On Apr 13, 2019, at 12:12 AM, Jerin Jacob Kollanukkaran
> <jerinj at marvell.com> wrote:
> >
> > Other than 1/1, I don't think, this patches series fixing any build for arm64.
> 
> That's one of reasons for the title.
> 
> > It is adding features required for Mellanox BlueField support.
> 
> Hard to agree.

The data should say:
meson: disable octeontx for buggy compilers on arm64
meson: change default cache line size for cortex-a72
net/mlx: fix library search in meson build
meson: add Mellanox BlueField cross-compile config
build: add option for armv8 crypto extension
mk: disable armv8 crypto extension for Mellanox BlueField

> 
> > Please change subject to more appropriate name.
> 
> If the title of the cover letter (which isn't merged anyway but informative) still
> bothers you, let me know. I'd rather remove the cover letter like your patchset.

If those are fixes then it needs to back ported to stable tree.
What bothers to me as maintainer that it painting a different picture that
arm64 still has build issues for meson. It is not the case.
So it its good to say what the actual content is, Let me ask you way around,
If it is just informational not going to merged then why to change to something
more appropriate.

I don't have any strong opinion keep the cover letter or not? You can decide.




> 
> >> -----Original Message-----
> >> From: Yongseok Koh <yskoh at mellanox.com>
> >> Sent: Saturday, April 13, 2019 4:55 AM
> >> To: bruce.richardson at intel.com; Jerin Jacob Kollanukkaran
> >> <jerinj at marvell.com>; Pavan Nikhilesh Bhagavatula
> >> <pbhagavatula at marvell.com>; shahafs at mellanox.com
> >> Cc: dev at dpdk.org; thomas at monjalon.net; gavin.hu at arm.com;
> >> Honnappa.Nagarahalli at arm.com
> >> Subject: [EXT] [PATCH 0/6] build: fix build for arm64
> >>
> >> External Email
> >>
> >> ---------------------------------------------------------------------
> >> -
> >> This patchset depends on
> >> "meson: add infra to support machine specific flags" [1]
> >>
> >> [1]
> >> https://eur03.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpatc
> >>
> hes.dpdk.org%2Fpatch%2F52606%2F&data=02%7C01%7Cyskoh%40mellan
> ox.c
> >>
> om%7C0c76f968187240046bfd08d6bfdf7283%7Ca652971c7d2e4d9ba6a4d1492
> 56f4
> >>
> 61b%7C0%7C0%7C636907363757515638&sdata=X3fQ%2B%2B7RIs8MbTy
> %2Bok5W
> >> Vm6wSmslrcyRuIuECAVhxXA%3D&reserved=0
> >>
> >> Yongseok Koh (6):
> >>  meson: disable octeontx for buggy compilers on arm64
> >>  meson: change default cache line size for cortex-a72
> >>  net/mlx: fix library search in meson build
> >>  meson: add Mellanox BlueField cross-compile config
> >>  build: add option for armv8 crypto extension
> >>  mk: disable armv8 crypto extension for Mellanox BlueField
> >>
> >> config/arm/arm64_bluefield_linux_gcc          | 16 ++++++++++++++++
> >> config/arm/meson.build                        | 18 +++++++++++-------
> >> config/common_armv8a_linux                    |  1 +
> >> config/defconfig_arm64-bluefield-linuxapp-gcc |  6 ++++++
> >> drivers/crypto/armv8/Makefile                 |  4 ++++
> >> drivers/event/meson.build                     |  6 +++++-
> >> drivers/net/mlx4/meson.build                  | 19 +++++++++++--------
> >> drivers/net/mlx5/meson.build                  | 19 +++++++++++--------
> >> meson_options.txt                             |  2 ++
> >> mk/machine/armv8a/rte.vars.mk                 |  4 ++++
> >> 10 files changed, 71 insertions(+), 24 deletions(-)  create mode
> >> 100644 config/arm/arm64_bluefield_linux_gcc
> >>
> >> --
> >> 2.21.0.196.g041f5ea
> >



More information about the dev mailing list