From: Marco Varlese <marco.varlese@suse.com>
To: Luca Boccassi <luca.boccassi@gmail.com>,
Bruce Richardson <bruce.richardson@intel.com>,
dev@dpdk.org
Subject: Re: [dpdk-dev] [RFCv2 00/40] Building DPDK with meson and ninja
Date: Fri, 18 Aug 2017 10:00:01 +0200 [thread overview]
Message-ID: <1503043201.3032.0.camel@suse.com> (raw)
In-Reply-To: <1502983469.31476.3.camel@gmail.com>
Hi Luca,
On Thu, 2017-08-17 at 16:24 +0100, Luca Boccassi wrote:
> On Thu, 2017-08-17 at 16:10 +0200, Marco Varlese wrote:
> >
> > Hi Bruce,
> >
> > On Mon, 2017-08-14 at 10:51 +0100, Bruce Richardson wrote:
> > >
> > > Following on from previous RFC [http://dpdk.org/dev/patchwork/patch
> > > /25104/]
> > > here is a second draft implementation for building DPDK with meson
> > > and
> > > ninja. While still not building all of DPDK, and needing patch
> > > cleanup so
> > > that patches don't overwrite previous work, it is more complete in
> > > many
> > > ways than the previous version and includes:
> > >
> > > * dynamic build configuration e.g. building pcap driver only if
> > > pcap is
> > > found, only build af_packet if the target is linux, and only
> > > building QAT
> > > and openssl crypto drivers if libcrypto is found
> > > * support for pmdinfo inside the PMDs(for shared builds) and
> > > binaries (for
> > > static builds)
> > > * generalized/standardized way of building libs and drivers, though
> > > the
> > > drivers code still needs generalization at the driver, rather
> > > than
> > > driver-class level.
> > > * support for having a pkgconfig file for DPDK on install, and
> > > helloworld
> > > and l2fwd can be built using the pkgconfig info (via make, not
> > > ninja)
> > > * support for library versions
> > > * an implementation for FreeBSD as well as Linux
> > > * all libraries are included in the build, as well as a number of
> > > NIC,
> > > crypto, and mempool drivers
> > > * the igb_uio kernel module is build via the kernel Kbuild system
> > > as part
> > > of a meson/ninja DPDK build
> >
> > This is really great to see. I do have one suggestion.
> > Would it be possible to version the libraries built by the build-
> > system
> > following the same version of the DPDK release?
> >
> > For instance, in DPDK 17.08 we currently have:
> > # objdump -p librte_pmd_ixgbe.so.1 |grep SONAME
> > SONAME librte_pmd_ixgbe.so.1
> >
> > Would it make sense to instead have librte_pmd_ixgbe.so.17.08
> >
> > I think it would help to facilitate the installation of multiple DPDK
> > library
> > versions on the same system.
> >
> > For example, we could have the following scenario:
> >
> > 1) OpenVSwithc linked with version 17.02 of DPDK
> > 2) VPP linked with version 17.08 of DPDK
> > 3) DPDK 18.xx installed in the system for any cutting-edge
> > application
> > prototyping.
> >
> > Is this something which could be incorporated as part of this work?
>
> Christian sent a patch a while ago, which was merged, to enable this in
> the current build system, it's the CONFIG_RTE_MAJOR_ABI option, we use
> it in Debian and Ubuntu for the reasons you mentioned.
Thanks for your input, very much appreciated.
I'll give it a try right away.
>
> And if it's not been translated yet, I agree it's an important one.
>
> >
> > >
> > > there is still plenty to do on this, but please review and test it
> > > out.
> > > I've kept the patches split so that the evolution of the work is a
> > > bit
> > > clearer than having it all send in a monolithic patch. All feedback
> > > welcome. I recommend one uses the latest meson release for this,
> > > though the
> > > minimum version supported should be v0.4.
> > >
> > > I've tested static and dynamic builds with clang and gcc on Fedora
> > > 26, and
> > > done test builds on FreeBSD 11 with clang too. Build with other
> > > compilers
> > > and on other OS's may have issues, so YMMV.
> >
> > I'll give it a try on openSUSE and let you know of any issues I might
> > face.
> >
> >
> > Cheers,
> > Marco
> >
> > >
> > >
> > > To do a basic build, using gcc and with shared libs, in a folder
> > > called "gcc-
> > > build":
> > >
> > > user@host:dpdk$ meson gcc-build
> > > ....
> > > user@host:dpdk$ cd gcc-build
> > > user@host:gcc-build$ ninja
> > > user@host:gcc-build$ sudo ninja install
> > >
> > > To do a build with clang, using static libs:
> > >
> > > user@host:dpdk$ CC=clang meson --default_library=static clang-
> > > build
> > > ....
> > > user@host:dpdk$ cd clang-build
> > > user@host:clang-build$ ninja
> > > user@host:clang-build$ sudo ninja install
> > >
> > > Other options can be adjusted by running "mesonconf" in the build
> > > directory, i.e. gcc-build or clang-build in the examples above.
> > >
> > >
> > > Bruce Richardson (40):
> > > build: initial hooks for using meson with DPDK
> > > build: create pkg-config file for DPDK
> > > build: build linuxapp EAL with meson and ninja
> > > build: add EAL support under BSD
> > > build: add core libraries to meson build system
> > > build: add i40e driver to meson build
> > > build: add pmdinfogen to build
> > > build: generate list of sources for pmdinfogen
> > > build: build i40e driver, including pmdinfo
> > > build: install usertools scripts
> > > build: simplify generation of pmd.c files
> > > build: generalize net driver build to higher level
> > > build: remove hard-coded enablement of vector driver
> > > build: add ixgbe driver to build
> > > build: set up standard deps for drivers
> > > build: add af_packet driver to build
> > > build: add pcap PMD support
> > > build: add symbol version map file support to libs
> > > build: build libraries in a loop for brevity
> > > build: version library .so files
> > > eal: add version information to meson build
> > > build: add mempool drivers to build
> > > build: add gro library to meson build
> > > build: tweak naming of pmd dependencies
> > > build: track driver include directories properly
> > > metrics: add metrics lib to meson build
> > > build: track dependencies recursively
> > > testpmd: compile testpmd with meson and ninja
> > > crypto/null: rename the version file to standard
> > > crypto/qat: remove dependency on ether library
> > > build: add cryptodev and some crypto drivers to build
> > > igb_uio: add igb_uio to meson build
> > > ip_frag: rename version file to standard name
> > > build: add most remaining libraries to meson build
> > > build: add packet framework libs to meson build
> > > acl: add acl library to meson build
> > > build: add ark and avp PMDs to build
> > > build: fix static library builds with base code
> > > build: fix driver dependencies for static builds
> > > examples: allow basic sample app build using pkg-config
> > >
> > > app/meson.build | 32 ++++++
> > > app/test-pmd/meson.build | 74
> > > +++++++++++++
> > > buildtools/gen-pmdinfo-cfile.sh | 41 ++++++++
> > > buildtools/meson.build | 34 ++++++
> > > buildtools/pmdinfogen/meson.build | 43 ++++++++
> > > config/meson.build | 69
> > > +++++++++++++
> > > config/rte_config.h | 114
> > > +++++++++++++++++++++
> > > config/x86/meson.build | 70
> > > +++++++++++++
> > > drivers/crypto/meson.build | 108
> > > +++++++++++++++++++
> > > drivers/crypto/null/Makefile | 2 +-
> > > drivers/crypto/null/meson.build | 32 ++++++
> > > ...crypto_version.map => rte_pmd_null_version.map} | 0
> > > drivers/crypto/openssl/meson.build | 38 +++++++
> > > drivers/crypto/qat/meson.build | 41 ++++++++
> > > drivers/crypto/qat/qat_crypto.c | 1 -
> > > drivers/mempool/meson.build | 108
> > > +++++++++++++++++++
> > > drivers/mempool/ring/meson.build | 32 ++++++
> > > drivers/mempool/stack/meson.build | 32 ++++++
> > > drivers/meson.build | 34 ++++++
> > > drivers/net/af_packet/meson.build | 35 +++++++
> > > drivers/net/ark/meson.build | 45 ++++++++
> > > drivers/net/avp/meson.build | 34 ++++++
> > > drivers/net/i40e/base/meson.build | 56
> > > ++++++++++
> > > drivers/net/i40e/meson.build | 61
> > > +++++++++++
> > > drivers/net/ixgbe/base/meson.build | 60
> > > +++++++++++
> > > drivers/net/ixgbe/meson.build | 58
> > > +++++++++++
> > > drivers/net/meson.build | 108
> > > +++++++++++++++++++
> > > drivers/net/pcap/meson.build | 41 ++++++++
> > > examples/helloworld/Makefile | 13 +++
> > > examples/l2fwd/Makefile | 13 +++
> > > lib/librte_acl/meson.build | 60
> > > +++++++++++
> > > lib/librte_bitratestats/meson.build | 34 ++++++
> > > lib/librte_cfgfile/meson.build | 35 +++++++
> > > lib/librte_cmdline/meson.build | 57
> > > +++++++++++
> > > lib/librte_compat/meson.build | 36 +++++++
> > > lib/librte_cryptodev/meson.build | 40 ++++++++
> > > lib/librte_distributor/meson.build | 39 +++++++
> > > lib/librte_eal/bsdapp/eal/meson.build | 67
> > > ++++++++++++
> > > lib/librte_eal/bsdapp/meson.build | 32 ++++++
> > > lib/librte_eal/common/arch/meson.build | 33 ++++++
> > > lib/librte_eal/common/arch/x86/meson.build | 32 ++++++
> > > lib/librte_eal/common/include/arch/meson.build | 33 ++++++
> > > lib/librte_eal/common/include/arch/x86/meson.build | 48 +++++++++
> > > lib/librte_eal/common/include/meson.build | 71
> > > +++++++++++++
> > > lib/librte_eal/common/meson.build | 71
> > > +++++++++++++
> > > lib/librte_eal/linuxapp/eal/meson.build | 72
> > > +++++++++++++
> > > lib/librte_eal/linuxapp/igb_uio/Kbuild | 1 +
> > > lib/librte_eal/linuxapp/igb_uio/meson.build | 47 +++++++++
> > > lib/librte_eal/linuxapp/meson.build | 42 ++++++++
> > > lib/librte_eal/meson.build | 44 ++++++++
> > > lib/librte_efd/meson.build | 34 ++++++
> > > lib/librte_ether/meson.build | 46 +++++++++
> > > lib/librte_eventdev/meson.build | 36 +++++++
> > > lib/librte_gro/meson.build | 34 ++++++
> > > lib/librte_hash/meson.build | 46 +++++++++
> > > lib/librte_ip_frag/Makefile | 2 +-
> > > lib/librte_ip_frag/meson.build | 39 +++++++
> > > ..._ipfrag_version.map => rte_ip_frag_version.map} | 0
> > > lib/librte_jobstats/meson.build | 34 ++++++
> > > lib/librte_kni/meson.build | 35 +++++++
> > > lib/librte_kvargs/meson.build | 35 +++++++
> > > lib/librte_latencystats/meson.build | 34 ++++++
> > > lib/librte_lpm/meson.build | 35 +++++++
> > > lib/librte_mbuf/meson.build | 36 +++++++
> > > lib/librte_mempool/meson.build | 35 +++++++
> > > lib/librte_meter/meson.build | 34 ++++++
> > > lib/librte_metrics/meson.build | 34 ++++++
> > > lib/librte_net/meson.build | 46 +++++++++
> > > lib/librte_pdump/meson.build | 34 ++++++
> > > lib/librte_pipeline/meson.build | 35 +++++++
> > > lib/librte_port/meson.build | 58
> > > +++++++++++
> > > lib/librte_power/meson.build | 35 +++++++
> > > lib/librte_reorder/meson.build | 34 ++++++
> > > lib/librte_ring/meson.build | 34 ++++++
> > > lib/librte_sched/meson.build | 36 +++++++
> > > lib/librte_table/meson.build | 52
> > > ++++++++++
> > > lib/librte_timer/meson.build | 34 ++++++
> > > lib/librte_vhost/meson.build | 36 +++++++
> > > lib/meson.build | 105
> > > +++++++++++++++++++
> > > meson.build | 88
> > > ++++++++++++++++
> > > meson_options.txt | 8 ++
> > > usertools/meson.build | 32 ++++++
> > > 82 files changed, 3536 insertions(+), 3 deletions(-)
> > > create mode 100644 app/meson.build
> > > create mode 100644 app/test-pmd/meson.build
> > > create mode 100755 buildtools/gen-pmdinfo-cfile.sh
> > > create mode 100644 buildtools/meson.build
> > > create mode 100644 buildtools/pmdinfogen/meson.build
> > > create mode 100644 config/meson.build
> > > create mode 100644 config/rte_config.h
> > > create mode 100644 config/x86/meson.build
> > > create mode 100644 drivers/crypto/meson.build
> > > create mode 100644 drivers/crypto/null/meson.build
> > > rename drivers/crypto/null/{rte_pmd_null_crypto_version.map =>
> > > rte_pmd_null_version.map} (100%)
> > > create mode 100644 drivers/crypto/openssl/meson.build
> > > create mode 100644 drivers/crypto/qat/meson.build
> > > create mode 100644 drivers/mempool/meson.build
> > > create mode 100644 drivers/mempool/ring/meson.build
> > > create mode 100644 drivers/mempool/stack/meson.build
> > > create mode 100644 drivers/meson.build
> > > create mode 100644 drivers/net/af_packet/meson.build
> > > create mode 100644 drivers/net/ark/meson.build
> > > create mode 100644 drivers/net/avp/meson.build
> > > create mode 100644 drivers/net/i40e/base/meson.build
> > > create mode 100644 drivers/net/i40e/meson.build
> > > create mode 100644 drivers/net/ixgbe/base/meson.build
> > > create mode 100644 drivers/net/ixgbe/meson.build
> > > create mode 100644 drivers/net/meson.build
> > > create mode 100644 drivers/net/pcap/meson.build
> > > create mode 100644 lib/librte_acl/meson.build
> > > create mode 100644 lib/librte_bitratestats/meson.build
> > > create mode 100644 lib/librte_cfgfile/meson.build
> > > create mode 100644 lib/librte_cmdline/meson.build
> > > create mode 100644 lib/librte_compat/meson.build
> > > create mode 100644 lib/librte_cryptodev/meson.build
> > > create mode 100644 lib/librte_distributor/meson.build
> > > create mode 100644 lib/librte_eal/bsdapp/eal/meson.build
> > > create mode 100644 lib/librte_eal/bsdapp/meson.build
> > > create mode 100644 lib/librte_eal/common/arch/meson.build
> > > create mode 100644 lib/librte_eal/common/arch/x86/meson.build
> > > create mode 100644 lib/librte_eal/common/include/arch/meson.build
> > > create mode 100644
> > > lib/librte_eal/common/include/arch/x86/meson.build
> > > create mode 100644 lib/librte_eal/common/include/meson.build
> > > create mode 100644 lib/librte_eal/common/meson.build
> > > create mode 100644 lib/librte_eal/linuxapp/eal/meson.build
> > > create mode 100644 lib/librte_eal/linuxapp/igb_uio/Kbuild
> > > create mode 100644 lib/librte_eal/linuxapp/igb_uio/meson.build
> > > create mode 100644 lib/librte_eal/linuxapp/meson.build
> > > create mode 100644 lib/librte_eal/meson.build
> > > create mode 100644 lib/librte_efd/meson.build
> > > create mode 100644 lib/librte_ether/meson.build
> > > create mode 100644 lib/librte_eventdev/meson.build
> > > create mode 100644 lib/librte_gro/meson.build
> > > create mode 100644 lib/librte_hash/meson.build
> > > create mode 100644 lib/librte_ip_frag/meson.build
> > > rename lib/librte_ip_frag/{rte_ipfrag_version.map =>
> > > rte_ip_frag_version.map}
> > > (100%)
> > > create mode 100644 lib/librte_jobstats/meson.build
> > > create mode 100644 lib/librte_kni/meson.build
> > > create mode 100644 lib/librte_kvargs/meson.build
> > > create mode 100644 lib/librte_latencystats/meson.build
> > > create mode 100644 lib/librte_lpm/meson.build
> > > create mode 100644 lib/librte_mbuf/meson.build
> > > create mode 100644 lib/librte_mempool/meson.build
> > > create mode 100644 lib/librte_meter/meson.build
> > > create mode 100644 lib/librte_metrics/meson.build
> > > create mode 100644 lib/librte_net/meson.build
> > > create mode 100644 lib/librte_pdump/meson.build
> > > create mode 100644 lib/librte_pipeline/meson.build
> > > create mode 100644 lib/librte_port/meson.build
> > > create mode 100644 lib/librte_power/meson.build
> > > create mode 100644 lib/librte_reorder/meson.build
> > > create mode 100644 lib/librte_ring/meson.build
> > > create mode 100644 lib/librte_sched/meson.build
> > > create mode 100644 lib/librte_table/meson.build
> > > create mode 100644 lib/librte_timer/meson.build
> > > create mode 100644 lib/librte_vhost/meson.build
> > > create mode 100644 lib/meson.build
> > > create mode 100644 meson.build
> > > create mode 100644 meson_options.txt
> > > create mode 100644 usertools/meson.build
> > >
>
prev parent reply other threads:[~2017-08-18 8:00 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-14 9:51 Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 01/40] build: initial hooks for using meson with DPDK Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 02/40] build: create pkg-config file for DPDK Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 03/40] build: build linuxapp EAL with meson and ninja Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 04/40] build: add EAL support under BSD Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 05/40] build: add core libraries to meson build system Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 06/40] build: add i40e driver to meson build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 07/40] build: add pmdinfogen to build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 08/40] build: generate list of sources for pmdinfogen Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 09/40] build: build i40e driver, including pmdinfo Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 10/40] build: install usertools scripts Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 11/40] build: simplify generation of pmd.c files Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 12/40] build: generalize net driver build to higher level Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 13/40] build: remove hard-coded enablement of vector driver Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 14/40] build: add ixgbe driver to build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 15/40] build: set up standard deps for drivers Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 16/40] build: add af_packet driver to build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 17/40] build: add pcap PMD support Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 18/40] build: add symbol version map file support to libs Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 19/40] build: build libraries in a loop for brevity Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 20/40] build: version library .so files Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 21/40] eal: add version information to meson build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 22/40] build: add mempool drivers to build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 23/40] build: add gro library to meson build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 24/40] build: tweak naming of pmd dependencies Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 25/40] build: track driver include directories properly Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 26/40] metrics: add metrics lib to meson build Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 27/40] build: track dependencies recursively Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 28/40] testpmd: compile testpmd with meson and ninja Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 29/40] crypto/null: rename the version file to standard Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 30/40] crypto/qat: remove dependency on ether library Bruce Richardson
2017-08-14 9:51 ` [dpdk-dev] [RFCv2 31/40] build: add cryptodev and some crypto drivers to build Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 32/40] igb_uio: add igb_uio to meson build Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 33/40] ip_frag: rename version file to standard name Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 34/40] build: add most remaining libraries to meson build Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 35/40] build: add packet framework libs " Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 36/40] acl: add acl library " Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 37/40] build: add ark and avp PMDs to build Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 38/40] build: fix static library builds with base code Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 39/40] build: fix driver dependencies for static builds Bruce Richardson
2017-08-14 9:52 ` [dpdk-dev] [RFCv2 40/40] examples: allow basic sample app build using pkg-config Bruce Richardson
2017-08-15 10:56 ` [dpdk-dev] [RFCv2 00/40] Building DPDK with meson and ninja Luca Boccassi
2017-08-15 11:31 ` Bruce Richardson
2017-08-17 14:10 ` Marco Varlese
2017-08-17 15:25 ` Luca Boccassi
2017-08-18 8:35 ` Bruce Richardson
2017-08-18 8:52 ` Marco Varlese
2017-08-18 9:17 ` Marco Varlese
2017-08-18 9:33 ` Luca Boccassi
[not found] ` <1502983469.31476.3.camel@gmail.com>
2017-08-18 8:00 ` Marco Varlese [this message]
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=1503043201.3032.0.camel@suse.com \
--to=marco.varlese@suse.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=luca.boccassi@gmail.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).