From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marvin Liu Subject: [PATCH] examples/vm_power_manager: fix build with libvirt version < 0.9.3 Date: Mon, 11 Apr 2016 11:45:57 +0800 Message-ID: <1460346357-26592-1-git-send-email-yong.liu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: Marvin Liu To: dev@dpdk.org Return-path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id EA22C137D for ; Mon, 11 Apr 2016 05:46:05 +0200 (CEST) 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" vm_power_manager utilize libvirt API virDomainGetVcpuPinInfo for retrieve domU vcpu information. This API implemented from version 0.9.3. Suse11 SP3 32bit default libvirt version is 0.8.8, so there'll be build error. Add judgement in sample Makefile to skip unsupport environment. examples/vm_power_manager/channel_manager.c: In function =E2=80=98update_pcpus_mask=E2=80=99: channel_manager.c:117:3: error: implicit declaration of function =E2=80=98virDomainGetVcpuPinInfo=E2=80=99 Fixes: 2e099bc5d104 ("fix split of compiler and linker options") Signed-off-by: Marvin Liu diff --git a/examples/vm_power_manager/Makefile b/examples/vm_power_manag= er/Makefile index 113dbc4..49a6b9b 100644 --- a/examples/vm_power_manager/Makefile +++ b/examples/vm_power_manager/Makefile @@ -33,9 +33,19 @@ ifeq ($(RTE_SDK),) $(error "Please define RTE_SDK environment variable") endif =20 +LIBVIRT_COMMON =3D libvirt-common.h +LIBVIRT_HEADER =3D libvirt.h +INCLUDE_PATH =3D /usr/include/libvirt/ + +HEADER_FILE =3D $(shell if [ -f $(INCLUDE_PATH)$(LIBVIRT_COMMON) ]; then= echo $(LIBVIRT_COMMON);else echo $(LIBVIRT_HEADER); fi;) +LIBVIRT_INCLUDE =3D $(INCLUDE_PATH)$(HEADER_FILE) + +LIBVIR_VER =3D $(shell gawk '/LIBVIR_VERSION_NUMBER .*/{print $$3}' $(LI= BVIRT_INCLUDE)) + # Default target, can be overridden by command line or environment RTE_TARGET ?=3D x86_64-native-linuxapp-gcc =20 +ifeq ($(shell test $(LIBVIR_VER) -ge 00009003 && echo 1), 1) include $(RTE_SDK)/mk/rte.vars.mk =20 # binary name @@ -57,3 +67,10 @@ CFLAGS_main.o +=3D -Wno-return-type endif =20 include $(RTE_SDK)/mk/rte.extapp.mk + +else +.PHONY: all clean +all: +$(warning "vm_power_manager required libvirt version >=3D 0.9.3, please = update libvirt-devel first") +clean: +endif --=20 1.9.3