* Re: [PATCH] kvm: ppc: detect old headers [not found] ` <4DBFF595.8060607@siemens.com> @ 2011-05-03 12:33 ` Alexander Graf 2011-05-03 13:02 ` Jan Kiszka 0 siblings, 1 reply; 2+ messages in thread From: Alexander Graf @ 2011-05-03 12:33 UTC (permalink / raw) To: Jan Kiszka; +Cc: QEMU-devel Developers, Scott Wood, kvm@vger.kernel.org list On 03.05.2011, at 14:31, Jan Kiszka wrote: > On 2011-05-03 14:28, Alexander Graf wrote: >> >> On 03.05.2011, at 14:14, Jan Kiszka wrote: >> >>> On 2011-05-03 13:54, Alexander Graf wrote: >>>> When compiling Qemu with older kernel headers, the PVR setting >>>> mechanism isn't available yet. Unfortunately, back then I didn't add >>>> a capability we could check against, so all we can do is add a configure >>>> test to see if we support PVR setting. For BookE, we don't care yet. >>>> >>>> This fixes compilation errors with KVM enabled on older kernel headers >>>> (like 2.6.32). >>> >>> Why not finally import the latest kvm kernel headers into qemu? Would >>> save us a lot of current and future configure and #ifdef dances. >> >> Sure, sounds like a good topic for today's call? > > Fine with me. Patch should be done by then as well. *shrug* I'm fairly indifferent on that topic. It would help users, so they can easier compile things, but requires us to keep the headers in sync. Do you have any good way of automating the process? Alex ^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] kvm: ppc: detect old headers 2011-05-03 12:33 ` [PATCH] kvm: ppc: detect old headers Alexander Graf @ 2011-05-03 13:02 ` Jan Kiszka 0 siblings, 0 replies; 2+ messages in thread From: Jan Kiszka @ 2011-05-03 13:02 UTC (permalink / raw) To: Alexander Graf Cc: QEMU-devel Developers, Scott Wood, kvm@vger.kernel.org list On 2011-05-03 14:33, Alexander Graf wrote: > > On 03.05.2011, at 14:31, Jan Kiszka wrote: > >> On 2011-05-03 14:28, Alexander Graf wrote: >>> >>> On 03.05.2011, at 14:14, Jan Kiszka wrote: >>> >>>> On 2011-05-03 13:54, Alexander Graf wrote: >>>>> When compiling Qemu with older kernel headers, the PVR setting >>>>> mechanism isn't available yet. Unfortunately, back then I didn't add >>>>> a capability we could check against, so all we can do is add a configure >>>>> test to see if we support PVR setting. For BookE, we don't care yet. >>>>> >>>>> This fixes compilation errors with KVM enabled on older kernel headers >>>>> (like 2.6.32). >>>> >>>> Why not finally import the latest kvm kernel headers into qemu? Would >>>> save us a lot of current and future configure and #ifdef dances. >>> >>> Sure, sounds like a good topic for today's call? >> >> Fine with me. Patch should be done by then as well. > > *shrug* I'm fairly indifferent on that topic. It would help users, so they can easier compile things, but requires us to keep the headers in sync. Do you have any good way of automating the process? There will be a 'make update-kvm-headers' target, imported from kvm-kmod. Can be run against some recent kernel, and the result will just have to be committed & posted. Moreover, I will drop alternative ways of pulling in headers (except via CFLAGS overwriting). That will typically bite the patch submitter who requires a header update and make her/him submit latest headers as well. So far at least for the theory. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2011-05-03 13:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1304423686-18909-1-git-send-email-agraf@suse.de>
[not found] ` <4DBFF1A3.2030702@siemens.com>
[not found] ` <78134E6D-50FE-483B-8B9A-6A577AC4E47B@suse.de>
[not found] ` <4DBFF595.8060607@siemens.com>
2011-05-03 12:33 ` [PATCH] kvm: ppc: detect old headers Alexander Graf
2011-05-03 13:02 ` Jan Kiszka
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).