DPDK patches and discussions
 help / color / mirror / Atom feed
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
>

      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).