From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Jones Subject: Re: [PATCH v4 0/4] KVM: arm64: BUG FIX: Correctly handle zero register transfers Date: Mon, 7 Dec 2015 15:50:02 -0600 Message-ID: <20151207215002.GG4402@hawk.localdomain> References: <20151205003334.GB3117@hawk.localdomain> <011701d130cb$f2efc830$d8cf5890$@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: 'Marc Zyngier' , kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org To: Pavel Fedin Return-path: Content-Disposition: inline In-Reply-To: <011701d130cb$f2efc830$d8cf5890$@samsung.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu List-Id: kvm.vger.kernel.org On Mon, Dec 07, 2015 at 11:47:44AM +0300, Pavel Fedin wrote: > Hello! > > > But, if Pavel doesn't > > mind trying them out on his system, then it'd be good to know if they > > reproduce there. I'd like to find out if it's a test case problem or > > something else strange going on with environments. > > Does not build, applied to master: > --- cut --- > aarch64-unknown-linux-gnu-gcc -std=gnu99 -ffreestanding -Wextra -O2 -I lib -I lib/libfdt -g -MMD -MF arm/.xzr-test.d -Wall > -fomit-frame-pointer -fno-stack-protector -c -o arm/xzr-test.o arm/xzr-test.c > arm/xzr-test.c: In function 'check_xzr_sysreg': > arm/xzr-test.c:13:2: warning: implicit declaration of function 'mmu_disable' [-Wimplicit-function-declaration] > mmu_disable(); /* Tell KVM to set HCR_TVM for this VCPU */ > ^ > aarch64-unknown-linux-gnu-gcc -std=gnu99 -ffreestanding -Wextra -O2 -I lib -I lib/libfdt -g -MMD -MF arm/.xzr-test.d -Wall > -fomit-frame-pointer -fno-stack-protector -nostdlib -o arm/xzr-test.elf \ > -Wl,-T,arm/flat.lds,--build-id=none,-Ttext=40080000 \ > arm/xzr-test.o arm/cstart64.o lib/libcflat.a lib/libfdt/libfdt.a /usr/lib/gcc/aarch64-unknown-linux-gnu/4.9.0/libgcc.a > lib/arm/libeabi.a > arm/xzr-test.o: In function `check_xzr_sysreg': > /cygdrive/d/Projects/kvm-unit-tests/arm/xzr-test.c:13: undefined reference to `mmu_disable' > --- cut --- Have you done a git pull of your kvm-unit-tests repo lately? The patch that introduces mmu_disable was commit a few months ago or so. Other than your repo just not having mmu_disable(), then I can't think of why it compiles for me and not you. If you have done a recent git pull, then maybe do a 'make distclean; ./configure; make' Thanks, drew > > Kind regards, > Pavel Fedin > Expert Engineer > Samsung Electronics Research center Russia > > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html