On 03.05.2011, at 16:44, Jan Kiszka wrote: > On 2011-05-03 16:36, Alexander Graf wrote: >> >> On 03.05.2011, at 16:20, Jan Kiszka wrote: >> >>> On 2011-05-03 14:37, Alexander Graf wrote: >>>> When running "make headers_install", we don't install kvm headers on >>>> PPC. This confuses Qemu, as it really wants to include kernel headers >>>> to know the ioctl structs. >>>> >>>> So let's add the arch/powerpc/include/asm headers that are important >>>> for KVM to the headers_install list. >>>> >>>> Signed-off-by: Alexander Graf <agraf@xxxxxxx> >>>> --- >>>> arch/powerpc/include/asm/Kbuild | 2 ++ >>>> 1 files changed, 2 insertions(+), 0 deletions(-) >>>> >>>> diff --git a/arch/powerpc/include/asm/Kbuild b/arch/powerpc/include/asm/Kbuild >>>> index d51df17..568b000 100644 >>>> --- a/arch/powerpc/include/asm/Kbuild >>>> +++ b/arch/powerpc/include/asm/Kbuild >>>> @@ -34,3 +34,5 @@ header-y += termios.h >>>> header-y += types.h >>>> header-y += ucontext.h >>>> header-y += unistd.h >>>> +header-y += kvm.h >>>> +header-y += kvm_para.h >>> >>> Weird, just worked for me with current kvm master without your patch. >> >> Oh? You did do make headers_install and it did push the correct headers to usr? > > I strongly suspect. It's all documented in my header patch. Your header patch manually copies the respective files, so you're not using that code path there? Alex -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html