Re: [kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-25 Thread Avi Kivity
Christian Borntraeger wrote: Am Montag, 24. März 2008 schrieb Avi Kivity: Christian Borntraeger wrote: Am Freitag, 21. März 2008 schrieb Anthony Liguori: This patch breaks QEMU build when doing a 'make sync'. When you do a top-level ./configure, libkvm is built with

Re: [kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-24 Thread Christian Borntraeger
Am Freitag, 21. März 2008 schrieb Anthony Liguori: This patch breaks QEMU build when doing a 'make sync'. When you do a top-level ./configure, libkvm is built with kerneldir pointing to kvm-userspace/kernel/include. While linux/kvm.h is present there, there isn't a linux/compiler.h. I

Re: [kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-24 Thread Avi Kivity
Christian Borntraeger wrote: Am Freitag, 21. März 2008 schrieb Anthony Liguori: This patch breaks QEMU build when doing a 'make sync'. When you do a top-level ./configure, libkvm is built with kerneldir pointing to kvm-userspace/kernel/include. While linux/kvm.h is present there, there

Re: [kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-23 Thread Avi Kivity
Anthony Liguori wrote: This patch breaks QEMU build when doing a 'make sync'. When you do a top-level ./configure, libkvm is built with kerneldir pointing to kvm-userspace/kernel/include. While linux/kvm.h is present there, there isn't a linux/compiler.h. The host kernelpath isn't

Re: [kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-21 Thread Anthony Liguori
This patch breaks QEMU build when doing a 'make sync'. When you do a top-level ./configure, libkvm is built with kerneldir pointing to kvm-userspace/kernel/include. While linux/kvm.h is present there, there isn't a linux/compiler.h. The host kernelpath isn't normally part of the libkvm or

Re: [kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-18 Thread Avi Kivity
Christian Borntraeger wrote: include/linux/kvm.h defines struct kvm_dirty_log to [...] union { void __user *dirty_bitmap; /* one bit per page */ __u64 padding; }; __user requires compiler.h to compile. Currently, this works on x86 only

[kvm-devel] [PATCH] kvm.h: __user requires compiler.h

2008-03-12 Thread Christian Borntraeger
include/linux/kvm.h defines struct kvm_dirty_log to [...] union { void __user *dirty_bitmap; /* one bit per page */ __u64 padding; }; __user requires compiler.h to compile. Currently, this works on x86 only coincidentally due to other