From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yuanhan Liu Subject: Re: [RFC] Link ibrte_vhost to librte_pmd_vhost Date: Mon, 25 Apr 2016 20:47:10 -0700 Message-ID: <20160426034710.GD7832@yliu-dev.sh.intel.com> References: <571DDDD8.6000000@igel.co.jp> <571DE345.4000609@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Tetsuya Mukawa , "dev@dpdk.org" , "Xie, Huawei" , Thomas Monjalon To: Panu Matilainen Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 4E1BD3208 for ; Tue, 26 Apr 2016 05:44:58 +0200 (CEST) Content-Disposition: inline In-Reply-To: <571DE345.4000609@redhat.com> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Mon, Apr 25, 2016 at 12:28:37PM +0300, Panu Matilainen wrote: > > >Another way is applying a below patch. > >--- a/drivers/net/vhost/Makefile > >+++ b/drivers/net/vhost/Makefile > >@@ -38,6 +38,7 @@ LIB = librte_pmd_vhost.a > > > > CFLAGS += -O3 > > CFLAGS += $(WERROR_FLAGS) > >+LDLIBS += -lrte_vhost > > > > EXPORT_MAP := rte_pmd_vhost_version.map > > > >This is same way to link libpcap to librte_pmd_pcap. > >What do you think about adding it to vhost PMD? > > Yes, this is absolutely the right thing to do. > > Ultimately this should be done for all dependencies in all libraries, but > missing dependencies are even more pronounced in plugins so the sooner this > goes in, the better. > > Acked-by: Panu Matilainen Panu, thanks for the input. Tetsuya, please submit a formal patch so that I can merge. --yliu