From: Panu Matilainen <pmatilai@redhat.com>
To: Bruce Richardson <bruce.richardson@intel.com>,
Christian Ehrhardt <christian.ehrhardt@canonical.com>
Cc: Daniele Di Proietto <diproiettod@vmware.com>,
James Page <james.page@canonical.com>, dev <dev@dpdk.org>,
Luca Boccassi <luca.boccassi@gmail.com>
Subject: Re: [dpdk-dev] Did we reduce unnecessary linkage too well?
Date: Fri, 30 Sep 2016 13:31:23 +0300 [thread overview]
Message-ID: <db155f73-e6b4-7e67-0a03-faea4eae371d@redhat.com> (raw)
In-Reply-To: <20160930101522.GB67296@bricha3-MOBL3>
On 09/30/2016 01:15 PM, Bruce Richardson wrote:
> On Thu, Sep 29, 2016 at 09:26:48AM +0200, Christian Ehrhardt wrote:
>> On Thu, Sep 29, 2016 at 9:20 AM, Panu Matilainen <pmatilai@redhat.com>
>> wrote:
>>
>>>
>>> Yup. Set CONFIG_RTE_EAL_PMD_PATH to the path where your PMDs are
>>> installed. Note that since the plugin autoloader in DPDK doesn't make
>>> assumptions about names, it'll try to load *everything* in that path, so
>>> you don't want it pointing to eg /usr/lib directly.
>
> Is this something we should look to change? To me having some sort of
> naming convention might not be a bad thing, so that we can point it at generic
> folders.
Plugins for program/library X are nearly always in a sub-directory of
their own, outside the linker path because ... well, they're plugins and
not something you should link to, and having them in separate
directories makes it possible to have multiple versions co-exist on the
system by simply placing the plugins into a versioned directory.
That's why the current plugin autoloader is the way it is - it's the
de-facto standard for dealing with plugins.
The DPDK case is a bit convoluted since some of the alleged plugins also
provide library APIs, so at least those DSOs need to be present in
linker paths. Plugins usually also lack soname version, because that
doesn't make much sense for plugins, libraries with APIs are different
there too.
Anyway, naming conventions are flimsy and fall apart in situations that
the directory approach easily handles. So I dont see a point in changing
that. What *would* be good is creating and populating that directory
from DPDK "make install" step automatically, at least when
RTE_EAL_PMD_PATH is set.
- Panu -
>
> /Bruce
>
prev parent reply other threads:[~2016-09-30 10:31 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-29 6:58 Christian Ehrhardt
2016-09-29 7:20 ` Panu Matilainen
2016-09-29 7:26 ` Christian Ehrhardt
2016-09-30 10:15 ` Bruce Richardson
2016-09-30 10:29 ` Christian Ehrhardt
2016-09-30 10:31 ` Panu Matilainen [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=db155f73-e6b4-7e67-0a03-faea4eae371d@redhat.com \
--to=pmatilai@redhat.com \
--cc=bruce.richardson@intel.com \
--cc=christian.ehrhardt@canonical.com \
--cc=dev@dpdk.org \
--cc=diproiettod@vmware.com \
--cc=james.page@canonical.com \
--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).