All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jan Kiszka <jan.kiszka@web.de>
To: Chris Friesen <chris.friesen@genband.com>
Cc: David Ahern <dsahern@gmail.com>, Avi Kivity <avi@redhat.com>,
	mtosatti@redhat.com, kvm@vger.kernel.org
Subject: Re: help?  need to enable VT-d for 2.6.27
Date: Wed, 31 Aug 2011 09:30:27 +0200	[thread overview]
Message-ID: <4E5DE313.1050509@web.de> (raw)
In-Reply-To: <4E5D6C34.6040907@genband.com>

[-- Attachment #1: Type: text/plain, Size: 1420 bytes --]

On 2011-08-31 01:03, Chris Friesen wrote:
> 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 the
>>> following:
>>>
>>> /home/cfriesen/Download/kvm-kmod-3.0b/x86/x86.c: In function
>>> ‘vcpu_enter_guest’:
>>> /home/cfriesen/Download/kvm-kmod-3.0b/x86/x86.c:5539:26: error:
>>> ‘prof_on’ 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
>>> appears 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 x86.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
> enabled on my kernel.  The problem is that our OS vendor has included a
> patch to convert prof_on to immediate values, which requires a change in
> x86.c.

Ah, now I remember: LTTng fallouts. I think there was just some
EXPORT_SYMBOL missing in the kernel, but my memory is weak.

Jan


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 262 bytes --]

  reply	other threads:[~2011-08-31  7:32 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-08-30 16:29 help? need to enable VT-d for 2.6.27 Chris Friesen
2011-08-30 17:55 ` David Ahern
2011-08-30 18:11   ` Chris Friesen
2011-08-30 18:31     ` David Ahern
2011-08-30 19:39       ` Chris Friesen
2011-08-30 19:54         ` David Ahern
2011-08-30 19:59         ` Jan Kiszka
2011-08-30 23:03           ` Chris Friesen
2011-08-31  7:30             ` Jan Kiszka [this message]
2011-08-31  0:03           ` [BUG] error compiling qemu-kvm-0.15.0 without vnc Chris Friesen
2011-08-31  7:40             ` Jan Kiszka
2011-08-31  7:40               ` [Qemu-devel] " Jan Kiszka
2011-08-31  8:14               ` Kevin Wolf
2011-08-31  8:14                 ` [Qemu-devel] " Kevin Wolf
2011-08-31 20:11                 ` Justin M. Forbes
2011-08-31 20:11                   ` Justin M. Forbes

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E5DE313.1050509@web.de \
    --to=jan.kiszka@web.de \
    --cc=avi@redhat.com \
    --cc=chris.friesen@genband.com \
    --cc=dsahern@gmail.com \
    --cc=kvm@vger.kernel.org \
    --cc=mtosatti@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.