From: Neil Horman <nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
To: Matthew Hall <mhall-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
Cc: dev-VfR2kkLFssw@public.gmane.org
Subject: Re: [PATCH 0/4] Fix build issues with CONFIG_RTE_BUILD_COMBINE_LIBS=y
Date: Thu, 2 Oct 2014 20:52:41 -0400 [thread overview]
Message-ID: <20141003005241.GA17181@localhost.localdomain> (raw)
In-Reply-To: <20141002211055.GA30270-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
On Thu, Oct 02, 2014 at 02:10:55PM -0700, Matthew Hall wrote:
> On Thu, Oct 02, 2014 at 04:24:51PM -0400, Neil Horman wrote:
> > This seems somewhat irrelevant to the patch. The default configuration is
> > already the way you want it to be, shared library performance is actually very
> > close to static performance, and yes, people can choose how they want to build.
> > Not sure what point your trying to make here.
> > Neil
>
> According to my reading, one of the subpatches threatened to disable the
> single static libs when the grouped one is built. If you are working with
> multiple DPDK apps, and one expects COMBINE_LIBS and the other does not, you
> can't use both with the same copy of DPDK anymore if that subpatch gets
> committed.
>
Ah, sorry, I misread, and thought you were advoating for making static builds
the default over dynamic builds, which made no sense to me, as that was already
the case.
Neil
> Matthew.
>
next prev parent reply other threads:[~2014-10-03 0:52 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-02 15:56 [PATCH 0/4] Fix build issues with CONFIG_RTE_BUILD_COMBINE_LIBS=y Sergio Gonzalez Monroy
[not found] ` <1412265386-26291-1-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-10-02 15:56 ` [PATCH 1/4] Link combined shared library using CC Sergio Gonzalez Monroy
2014-10-02 15:56 ` [PATCH 2/4] Do not generate individual libs when configured with RTE_BUILD_COMBINE_LIBS=y Sergio Gonzalez Monroy
[not found] ` <1412265386-26291-3-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-10-02 20:00 ` Matthew Hall
2014-10-02 15:56 ` [PATCH 3/4] Link apps only against combined lib or individual libs, not both Sergio Gonzalez Monroy
2014-10-02 15:56 ` [PATCH 4/4] Cleanup Sergio Gonzalez Monroy
2014-10-02 17:26 ` [PATCH 0/4] Fix build issues with CONFIG_RTE_BUILD_COMBINE_LIBS=y Neil Horman
[not found] ` <20141002172634.GE4900-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-10-02 20:04 ` Matthew Hall
[not found] ` <20141002200420.GB29590-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
2014-10-02 20:24 ` Neil Horman
[not found] ` <20141002202451.GF4900-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-10-02 21:10 ` Matthew Hall
[not found] ` <20141002211055.GA30270-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
2014-10-03 0:52 ` Neil Horman [this message]
2014-10-03 10:31 ` Sergio Gonzalez Monroy
[not found] ` <20141003103110.GB28988-IWE99D/oH1/+pXziaqXtF9h3ngVCH38I@public.gmane.org>
2014-10-03 11:28 ` Neil Horman
[not found] ` <20141003112833.GA24059-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-10-03 23:52 ` Stephen Hemminger
2014-10-04 2:30 ` Neil Horman
2014-10-03 7:15 ` Thomas Monjalon
2014-10-03 8:10 ` Sergio Gonzalez Monroy
[not found] ` <20141003081019.GA28988-IWE99D/oH1/+pXziaqXtF9h3ngVCH38I@public.gmane.org>
2014-10-03 8:27 ` Thomas Monjalon
2014-10-03 11:32 ` Neil Horman
[not found] ` <20141003113234.GB24059-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-10-03 18:17 ` Matthew Hall
[not found] ` <20141003181713.GC1741-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
2014-10-03 19:15 ` Neil Horman
[not found] ` <20141003191546.GD24059-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-10-03 21:21 ` Matthew Hall
[not found] ` <20141003212150.GA2637-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
2014-10-06 14:45 ` Neil Horman
2014-10-03 18:13 ` Matthew Hall
2014-10-03 18:00 ` Matthew Hall
2014-10-06 10:52 ` [PATCH v2 0/4] Update build process Sergio Gonzalez Monroy
[not found] ` <1412592755-3370-1-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-10-06 10:52 ` [PATCH v2 1/4] Link combined shared library using CC Sergio Gonzalez Monroy
2014-10-06 10:52 ` [PATCH v2 2/4] Link apps only against single/combined library Sergio Gonzalez Monroy
2014-10-06 10:52 ` [PATCH v2 3/4] Update library build process Sergio Gonzalez Monroy
[not found] ` <1412592755-3370-4-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-10-06 20:46 ` Matthew Hall
[not found] ` <20141006204607.GA18992-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
2014-10-07 9:55 ` Sergio Gonzalez Monroy
[not found] ` <20141007095511.GA27326-IWE99D/oH1/+pXziaqXtF9h3ngVCH38I@public.gmane.org>
2014-10-08 22:36 ` Matthew Hall
[not found] ` <20141008223655.GB29243-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org>
2014-10-09 9:44 ` Sergio Gonzalez Monroy
2014-10-08 15:38 ` Thomas Monjalon
2014-10-06 10:52 ` [PATCH v2 4/4] Link apps/DSOs against EXECENV_LDLIBS with --as-needed Sergio Gonzalez Monroy
[not found] ` <1412592755-3370-5-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-10-08 15:38 ` Thomas Monjalon
2014-10-09 9:23 ` Sergio Gonzalez Monroy
2014-10-06 14:49 ` [PATCH v2 0/4] Update build process Neil Horman
[not found] ` <20141006144946.GD22304-B26myB8xz7F8NnZeBjwnZQMhkBWG/bsMQH7oEaQurus@public.gmane.org>
2014-10-06 15:01 ` Sergio Gonzalez Monroy
[not found] ` <20141006150133.GA778-IWE99D/oH1/+pXziaqXtF9h3ngVCH38I@public.gmane.org>
2014-10-06 16:05 ` Neil Horman
2014-10-09 13:04 ` [PATCH v3 0/6] Update libs " Sergio Gonzalez Monroy
[not found] ` <1412859898-29068-1-git-send-email-sergio.gonzalez.monroy-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
2014-10-09 13:04 ` [PATCH v3 1/6] Link combined shared library using CC Sergio Gonzalez Monroy
2014-10-09 13:04 ` [PATCH v3 2/6] Link apps only against single/combined library Sergio Gonzalez Monroy
2014-10-09 13:04 ` [PATCH v3 3/6] Remove CONFIG_RTE_BUILD_COMBINE_LIBS and related Sergio Gonzalez Monroy
2014-10-09 13:04 ` [PATCH v3 4/6] Update library build process Sergio Gonzalez Monroy
2014-10-09 13:04 ` [PATCH v3 5/6] Avoid duplicated code Sergio Gonzalez Monroy
2014-10-09 13:04 ` [PATCH v3 6/6] Link apps/DSOs against EXECENV_LDLIBS with --as-needed Sergio Gonzalez Monroy
2014-10-13 16:01 ` [PATCH v3 0/6] Update libs build process Gonzalez Monroy, Sergio
[not found] ` <91383E96CE459D47BCE92EFBF5CE73B0C7E2B6-kPTMFJFq+rEMvF1YICWikbfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-10-21 9:43 ` Gonzalez Monroy, Sergio
[not found] ` <91383E96CE459D47BCE92EFBF5CE73B0C7F7E6-kPTMFJFq+rEMvF1YICWikbfspsVTdybXVpNB7YpNyf8@public.gmane.org>
2014-10-22 16:14 ` Gonzalez Monroy, Sergio
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=20141003005241.GA17181@localhost.localdomain \
--to=nhorman-2xusbdqka4r54taoqtywwq@public.gmane.org \
--cc=dev-VfR2kkLFssw@public.gmane.org \
--cc=mhall-Hv3ogNYU3JfZZajBQzqCxQ@public.gmane.org \
/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).