From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <tredaelli@redhat.com>
Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28])
 by dpdk.org (Postfix) with ESMTP id 6E7BC2A62
 for <dev@dpdk.org>; Thu, 30 Nov 2017 12:21:56 +0100 (CET)
Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.phx2.redhat.com
 [10.5.11.13])
 (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))
 (No client certificate requested)
 by mx1.redhat.com (Postfix) with ESMTPS id A71C281DF7;
 Thu, 30 Nov 2017 11:21:55 +0000 (UTC)
Received: from localhost (ovpn-204-167.brq.redhat.com [10.40.204.167])
 by smtp.corp.redhat.com (Postfix) with ESMTP id 9B4F860DCE;
 Thu, 30 Nov 2017 11:21:48 +0000 (UTC)
Date: Thu, 30 Nov 2017 12:21:47 +0100
From: Timothy Redaelli <tredaelli@redhat.com>
To: Thomas Monjalon <thomas@monjalon.net>
Cc: dev@dpdk.org, echaudro@redhat.com, Jingjing Wu <jingjing.wu@intel.com>
Message-ID: <20171130122147.1d10053f@redhat.com>
In-Reply-To: <3208097.sCyqhKb3JL@xps>
References: <e6a0258739baaf4058a367f065b77f03f594ad42.1510935549.git.tredaelli@redhat.com>
 <f76597fd-8ded-5b61-eb47-ba95f4215059@redhat.com>
 <3208097.sCyqhKb3JL@xps>
Organization: Red Hat
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
X-Scanned-By: MIMEDefang 2.79 on 10.5.11.13
X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16
 (mx1.redhat.com [10.5.110.25]); Thu, 30 Nov 2017 11:21:55 +0000 (UTC)
Subject: Re: [dpdk-dev] [PATCH] app/testpmd: link with virtio PMD when using
 shared libraries
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Thu, 30 Nov 2017 11:21:56 -0000

On Wed, 29 Nov 2017 23:23:40 +0100
Thomas Monjalon <thomas@monjalon.net> wrote:

> 20/11/2017 11:27, Eelco Chaudron:
> > On 17/11/17 17:19, Timothy Redaelli wrote:  
> > > testpmd is often used inside a VM to test the OVS PVP scenario.
> > > This commit makes testpmd to link to virtio PMD when DPDK is
> > > built as shared libraries too.
> > >
> > > Signed-off-by: Timothy Redaelli <tredaelli@redhat.com>  
> > 
> > Acked-by: Eelco Chaudron <echaudro@redhat.com>  
> 
> I really doubt it is the right fix.
> This PMD should be dynamically linked with dlopen as a plugin.
> It can be done with -d option or by specifying the plugin directory
> at compilation time in CONFIG_RTE_EAL_PMD_PATH.

Hi,
I checked that CONFIG_RTE_EAL_PMD_PATH is set correctly when we build
dpdk in Fedora, but unlucky the symlinks for each pmd from the PMD_PATH
to the real librte_pmd_* library (installed in /usr/lib64) on Fedora is
broken since it points to the .so file directly, that is only installed
by dpdk-devel package, instead of correctly point to the .so.[0-9] one
that is installed by the dpdk package and so my patch is not needed
and I'll send the fix on the dpdk Fedora spec file instead.

Just a little question: if testpmd loads the pmds dynamically using
dlopen, why in app/test-pmd/Makefile some PMD are linked at
build time?

Thank you and sorry for the noise.