[kvm-devel] headersinstall of kvm.h does not work

2008-03-07 Thread Christian Borntraeger
Hello Avi, in commit fb56dbb31c4738a3918db81fd24da732ce3b4ae6 you changed include/linux/Kbuild: snip KVM: Export include/linux/kvm.h only if $ARCH actually supports KVM Currently, make headers_check barfs due to , which includes, not existing. Rather than add a zillion s, export kvm.h

Re: [kvm-devel] headersinstall of kvm.h does not work

2008-03-07 Thread Avi Kivity
Christian Borntraeger wrote: > Hello Avi, > > in commit fb56dbb31c4738a3918db81fd24da732ce3b4ae6 you changed > include/linux/Kbuild: > snip > KVM: Export include/linux/kvm.h only if $ARCH actually supports KVM > Currently, make headers_check barfs due to , which > includes, not existing.

Re: [kvm-devel] headersinstall of kvm.h does not work

2008-03-07 Thread Christian Borntraeger
Am Freitag, 7. März 2008 schrieb Avi Kivity: > As I'm about to disappear for a week, consider a patch to remove the > config dependency and add asm-*/kvm.h pre-acked for mainline. Maybe the > presence of those empty asm-*/kvm.h files will encourage further kvm > ports to *. Something like the

Re: [kvm-devel] headersinstall of kvm.h does not work

2008-03-07 Thread Avi Kivity
Christian Borntraeger wrote: > Am Freitag, 7. März 2008 schrieb Avi Kivity: > >> As I'm about to disappear for a week, consider a patch to remove the >> config dependency and add asm-*/kvm.h pre-acked for mainline. Maybe the >> presence of those empty asm-*/kvm.h files will encourage further

Re: [kvm-devel] headersinstall of kvm.h does not work

2008-03-07 Thread David Woodhouse
On Fri, 2008-03-07 at 13:26 +0100, Christian Borntraeger wrote: > +unifdef-$(CONFIG_HAVE_KVM) += kvm.h > unifdef-y += llc.h > unifdef-y += loop.h > snip-- > > This patch does not work. Kbuild (scripts/Makefile.headersinst) does > not check the config file, so kvm.h is never installed. >