From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Friesen Subject: Re: help? need to enable VT-d for 2.6.27 Date: Tue, 30 Aug 2011 17:03:16 -0600 Message-ID: <4E5D6C34.6040907@genband.com> References: <4E5D0FF3.6030705@genband.com> <4E5D23F5.3070905@gmail.com> <4E5D27B5.8090004@genband.com> <4E5D2C68.2050801@gmail.com> <4E5D3C67.1020908@genband.com> <4E5D413D.1070700@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: David Ahern , Avi Kivity , mtosatti@redhat.com, kvm@vger.kernel.org To: Jan Kiszka Return-path: Received: from exprod7og111.obsmtp.com ([64.18.2.175]:55454 "EHLO exprod7og111.obsmtp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753859Ab1H3XDW (ORCPT ); Tue, 30 Aug 2011 19:03:22 -0400 In-Reply-To: <4E5D413D.1070700@web.de> Sender: kvm-owner@vger.kernel.org List-ID: On 08/30/2011 01:59 PM, Jan Kiszka wrote: > On 2011-08-30 21:39, Chris Friesen wrote: >> 2) If I create such a link, it builds a few files, and then gives th= e >> following: >> >> /home/cfriesen/Download/kvm-kmod-3.0b/x86/x86.c: In function >> =91vcpu_enter_guest=92: >> /home/cfriesen/Download/kvm-kmod-3.0b/x86/x86.c:5539:26: error: >> =91prof_on=92 undeclared (first use in this function) >> /home/cfriesen/Download/kvm-kmod-3.0b/x86/x86.c:5539:26: note: each >> undeclared identifier is reported only once for each function it app= ears in >> >> >> This appears to be due to the fact that KVM_PROFILING is defined >> unconditionally in my kernel, but "prof_on" is only defined if >> CONFIG_PROFILING is enabled. Commenting out that chunk of code in x= 86.c >> lets everything build. > > If CONFIG_PROFILING is off, prof_on is defined to 0. But we may miss > some include on older kernels. Does adding linux/profile.h to x86.c > solve the issue as well? It would appear that I got the explanation wrong. CONFIG_PROFILING is=20 enabled on my kernel. The problem is that our OS vendor has included a= =20 patch to convert prof_on to immediate values, which requires a change i= n=20 x86.c. Never mind, nothing to see here. Chris --=20 Chris Friesen Software Developer GENBAND chris.friesen@genband.com www.genband.com