DPDK patches and discussions
 help / color / mirror / Atom feed
From: Luca Boccassi <bluca@debian.org>
To: Jerin Jacob Kollanukkaran <jerinj@marvell.com>,
	"thomas@monjalon.net" <thomas@monjalon.net>,
	"yskoh@mellanox.com" <yskoh@mellanox.com>
Cc: "shahafs@mellanox.com" <shahafs@mellanox.com>,
	 "honnappa.nagarahalli@arm.com" <honnappa.nagarahalli@arm.com>,
	"Gavin.Hu@arm.com" <Gavin.Hu@arm.com>,
	 "tspeier@qti.qualcomm.com" <tspeier@qti.qualcomm.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH] config: change default cache line size for ARMv8 with meson
Date: Wed, 09 Jan 2019 15:41:33 +0000	[thread overview]
Message-ID: <1547048493.6022.65.camel@debian.org> (raw)
In-Reply-To: <043755be57a2f0b30a16d620180bbaa1f5c3144e.camel@marvell.com>

On Wed, 2019-01-09 at 15:34 +0000, Jerin Jacob Kollanukkaran wrote:
> > > Please check below thread and patch.
> > > 
> > > http://mails.dpdk.org/archives/dev/2019-January/122676.html
> > > https://patches.dpdk.org/patch/49477/
> > > 
> > > Debian folks are building like this for the _generic_ image.
> > > What ever works for every distros, I am fine with that.
> > > 
> > > meson configure -Dmachine=default
> > > meson build
> > > cd build
> > > ninja
> > > ninja install
> > 
> > I think we agree on the idea of having different configs
> > for unmodified A72 core and generic build working for all.
> 
> Yes, I agree. config or some scheme to address the generic and
> default
> usecase.
> 
> > The remaining bits to discuss are:
> > 	- do we want to use the armv8 config for unmodified A72?
> > 	- what should be the name of the generic config?
> 
> If all distros following "meson configure -Dmachine=default" scheme
> why not follow that to make generic image. i.e when machine=default
> set then Cache lize size 128B CL specific stuff be kicked in else
> it probe the value based on MIDR from sysfs.
> 
> 
> > When digging more the config files in meson, I found this:
> > 	
> > http://mesonbuild.com/Cross-compilation.html#cross-file-locations
> > It says that distros or compilers should provide some config files.
> > It means we should check if some standard names are emerging
> > and try to follow the same naming, or even re-use existing config
> > files.
> 
> I think this for cross compilation. if I understand it correctly,
> distro folks build arm64 image on arm64 target(not on x86) with
> generic
> images with lowest denominator cpu flags and such that works for all
> platforms.
> 
> Something similar found in arch linux website for arm64.
> https://archlinuxarm.org/platforms/armv8/generic
> 
> See Platforms: ARMv8
> 
> Anyway I am leaving this decision to distro folks or whoever is
> expert
> in meson based distro build generic creation.

Just a comment to confirm that yes, in general distributions do not do
cross-builds, but native builds. That is certainly the case for Debian,
and I am 99% sure it is also the case for Ubuntu and SUSE.

-- 
Kind regards,
Luca Boccassi

  reply	other threads:[~2019-01-09 15:41 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-01-09  9:39 Yongseok Koh
2019-01-09 10:09 ` [dpdk-dev] [EXT] " Jerin Jacob Kollanukkaran
2019-01-09 10:19   ` Luca Boccassi
2019-01-09 10:52     ` Jerin Jacob Kollanukkaran
2019-01-09 13:14       ` Luca Boccassi
2019-01-09 10:22   ` Yongseok Koh
2019-01-09 10:49     ` Jerin Jacob Kollanukkaran
2019-01-09 11:28       ` Thomas Monjalon
2019-01-09 12:47         ` Jerin Jacob Kollanukkaran
2019-01-09 13:30           ` Thomas Monjalon
2019-01-09 14:23             ` Jerin Jacob Kollanukkaran
2019-01-09 14:57               ` Thomas Monjalon
2019-01-09 15:34                 ` [dpdk-dev] " Jerin Jacob Kollanukkaran
2019-01-09 15:41                   ` Luca Boccassi [this message]
2019-01-09 16:36                     ` [dpdk-dev] [EXT] " Pavan Nikhilesh Bhagavatula
2019-01-09 16:52                       ` Luca Boccassi
2019-01-09 17:01                         ` Pavan Nikhilesh Bhagavatula
2019-01-14  4:32                 ` [dpdk-dev] [EXT] " Yongseok Koh
2019-01-14  7:44                   ` Honnappa Nagarahalli
2019-01-16  2:02                     ` Honnappa Nagarahalli
2019-01-19  7:09                       ` Yongseok Koh
2019-01-22 18:51                         ` Honnappa Nagarahalli
2019-01-23  8:56                           ` Jerin Jacob Kollanukkaran
2019-01-23 16:24                             ` Honnappa Nagarahalli
2019-01-23 17:19                               ` Jerin Jacob Kollanukkaran

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1547048493.6022.65.camel@debian.org \
    --to=bluca@debian.org \
    --cc=Gavin.Hu@arm.com \
    --cc=dev@dpdk.org \
    --cc=honnappa.nagarahalli@arm.com \
    --cc=jerinj@marvell.com \
    --cc=shahafs@mellanox.com \
    --cc=thomas@monjalon.net \
    --cc=tspeier@qti.qualcomm.com \
    --cc=yskoh@mellanox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).