From mboxrd@z Thu Jan 1 00:00:00 1970 From: Xiao Guangrong Subject: Re: [PATCH 3/3] KVM: perf: kvm events analysis tool Date: Mon, 13 Feb 2012 13:00:11 +0800 Message-ID: <4F3898DB.2030907@linux.vnet.ibm.com> References: <4F338CAA.10807@linux.vnet.ibm.com> <4F338D56.6010505@linux.vnet.ibm.com> <4F387DA4.7020503@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Avi Kivity , Marcelo Tosatti , Ingo Molnar , Arnaldo Carvalho de Melo , LKML , KVM To: David Ahern Return-path: In-Reply-To: <4F387DA4.7020503@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On 02/13/2012 11:04 AM, David Ahern wrote: > Compile fails on Fedora 16, x86_64 with latest tip-perf-core branch: >=20 >=20 > In file included from builtin-kvm-events.c:19:0: > ../../arch/x86/include/asm/svm.h:133:1: error: packed attribute is > unnecessary for =E2=80=98vmcb_seg=E2=80=99 [-Werror=3Dpacked] > ../../arch/x86/include/asm/svm.h:178:1: error: packed attribute is > unnecessary for =E2=80=98vmcb_save_area=E2=80=99 [-Werror=3Dpacked] > ../../arch/x86/include/asm/svm.h:183:1: error: packed attribute is > unnecessary for =E2=80=98vmcb=E2=80=99 [-Werror=3Dpacked] > In file included from builtin-kvm-events.c:20:0: > ../../arch/x86/include/asm/vmx.h:334:0: error: "REG_R8" redefined [-W= error] > /usr/include/sys/ucontext.h:46:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:335:0: error: "REG_R9" redefined [-W= error] > /usr/include/sys/ucontext.h:48:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:336:0: error: "REG_R10" redefined [-= Werror] > /usr/include/sys/ucontext.h:50:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:337:0: error: "REG_R11" redefined [-= Werror] > /usr/include/sys/ucontext.h:52:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:338:0: error: "REG_R12" redefined [-= Werror] > /usr/include/sys/ucontext.h:54:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:339:0: error: "REG_R13" redefined [-= Werror] > /usr/include/sys/ucontext.h:56:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:340:0: error: "REG_R14" redefined [-= Werror] > /usr/include/sys/ucontext.h:58:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:341:0: error: "REG_R15" redefined [-= Werror] > /usr/include/sys/ucontext.h:60:0: note: this is the location of the > previous definition > ../../arch/x86/include/asm/vmx.h:443:13: error: expected declaration > specifiers or =E2=80=98...=E2=80=99 before numeric constant > In file included from builtin-kvm-events.c:21:0: > ../../arch/x86/include/asm/kvm_host.h:15:22: fatal error: linux/mm.h:= No > such file or directory > cc1: all warnings being treated as errors >=20 The first patch(patch 1/3) should be applied! Thank you, David! :)