From: "Robin Jarry" <rjarry@redhat.com>
To: "Bruce Richardson" <bruce.richardson@intel.com>
Cc: <dev@dpdk.org>
Subject: Re: [PATCH dpdk v2 2/2] meson: add drivers to dynamic libraries
Date: Thu, 15 Oct 2026 16:47:48 +0200 [thread overview]
Message-ID: <DM5HT0VSVYFD.CDUXVM1LWY6T@redhat.com> (raw)
In-Reply-To: <atDEkFQs5XyI3xIp@bricha3-mobl1.ger.corp.intel.com>
Bruce Richardson, Oct 15, 2026 at 14:18:
> Yes, static builds link in everything, and that was always the design since
> static builds don't load drivers at runtime, everything has to be linked
> into the final object.
>
> Personally, I don't that strongly object to the change because - as you
> point out - the as-needed flag should prevent overlinking. If we do this,
> I'd suggest making both drivers and libs full lists, rather than
> distinguishing between drivers with public headers and those without,
> because that should allow us to simplify our package config files. Right
> now we have this strange split into two .pc files, as that was the only way
> I could find to get correct behaviour for both static and shared builds,
> where static included everything and shared was libs only. [IIRC, the
> complication was that, when you pass --static to pkg-config, the extra link
> args from the pc file get listed *after* the normal libs, while they need to get
> listed *before* in order to satisfy the dependencies drivers have on the
> libs]
Thanks. I will see what I can do.
next prev parent reply other threads:[~2026-10-15 14:47 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-15 10:09 [PATCH dpdk 0/2] Fix dynamic link of driver symbols with pkgconf Robin Jarry
2026-10-15 10:09 ` [PATCH dpdk 1/2] meson: rename static drivers list Robin Jarry
2026-10-15 10:36 ` Bruce Richardson
2026-10-15 10:09 ` [PATCH dpdk 2/2] meson: add drivers to dynamic libraries Robin Jarry
2026-10-15 10:40 ` Bruce Richardson
2026-10-15 10:41 ` [PATCH dpdk 0/2] Fix dynamic link of driver symbols with pkgconf Bruce Richardson
2026-10-15 11:27 ` Robin Jarry
2026-10-15 11:26 ` [PATCH dpdk v2 " Robin Jarry
2026-10-15 11:26 ` [PATCH dpdk v2 1/2] meson: rename static drivers list Robin Jarry
2026-10-15 11:26 ` [PATCH dpdk v2 2/2] meson: add drivers to dynamic libraries Robin Jarry
2026-10-15 11:52 ` Bruce Richardson
2026-10-15 12:06 ` Robin Jarry
2026-10-15 12:18 ` Bruce Richardson
2026-10-15 14:47 ` Robin Jarry [this message]
2026-10-16 15:38 ` [PATCH dpdk v3 0/2] Fix dynamic link of driver symbols with pkgconf Robin Jarry
2026-10-16 15:38 ` [PATCH dpdk v3 1/2] meson: rename static drivers list Robin Jarry
2026-10-16 15:38 ` [PATCH dpdk v3 2/2] meson: add all drivers to dynamic libraries Robin Jarry
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=DM5HT0VSVYFD.CDUXVM1LWY6T@redhat.com \
--to=rjarry@redhat.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.