On 2011-05-03 17:04, Alexander Graf wrote:
> 
> 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 <[email protected]>
>>>>> ---
>>>>> 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?
> 

I'm copying from the output dir auf headers_install (INSTALL_HDR_PATH).
What step does this patch target at?

Jan

-- 
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to