From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH] mk: fix missing link of librte_vhost in shared, non-combined config Date: Wed, 11 Feb 2015 13:25:41 +0200 Message-ID: <54DB3C35.9000104@redhat.com> References: <6ca22ee1257e34ee2b89f2fb354d6c382b8f3e29.1423644785.git.pmatilai@redhat.com> <91383E96CE459D47BCE92EFBF5CE73B004F4AB6C@IRSMSX108.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit To: "Gonzalez Monroy, Sergio" , "dev-VfR2kkLFssw@public.gmane.org" Return-path: In-Reply-To: <91383E96CE459D47BCE92EFBF5CE73B004F4AB6C-kPTMFJFq+rEMvF1YICWikbfspsVTdybXVpNB7YpNyf8@public.gmane.org> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" On 02/11/2015 12:51 PM, Gonzalez Monroy, Sergio wrote: > Hi Panu, > >> -----Original Message----- >> From: dev [mailto:dev-bounces-VfR2kkLFssw@public.gmane.org] On Behalf Of Panu Matilainen >> Sent: Wednesday, February 11, 2015 8:53 AM >> To: dev-VfR2kkLFssw@public.gmane.org >> Subject: [dpdk-dev] [PATCH] mk: fix missing link of librte_vhost in shared, >> non-combined config >> >> When building shared, non-combined library, librte_vhost does not get >> linked in, causing among other things vhost example to fail building to do >> undefined symbols on linkage. >> >> Signed-off-by: Panu Matilainen >> --- >> mk/rte.app.mk | 4 ++++ >> 1 file changed, 4 insertions(+) >> >> diff --git a/mk/rte.app.mk b/mk/rte.app.mk index 95dbb0b..55d989e 100644 >> --- a/mk/rte.app.mk >> +++ b/mk/rte.app.mk >> @@ -125,6 +125,10 @@ LDLIBS += -lm >> LDLIBS += -lrt >> endif >> >> +ifeq ($(CONFIG_RTE_LIBRTE_VHOST), y) >> +LDLIBS += -lrte_vhost >> +endif >> + >> endif # ! CONFIG_RTE_BUILD_COMBINE_LIBS >> >> ifeq ($(CONFIG_RTE_LIBRTE_PMD_PCAP),y) >> -- >> 2.1.0 > > I think that vhost is being linked in the wrong place (plugins section). > The plugins only get linked when building static libraries. > I think the patch should also remove vhost from the plugins section. Right, so vhost isn't a pluggable driver in the sense that pmds are. I wont claim to be familiar with all this virt-related puzzle pieces :) I'll send an updated patch, I was just looking to fix build in my particular config and ignored the rest. On a related note, shouldn't librte_pmd_bond and librte_pmd_xenvirt be included in the plugins section along with all the other pmds? - Panu -