From mboxrd@z Thu Jan 1 00:00:00 1970 From: ehrhardt-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org Subject: [PATCH] kvm: fix kvm_para.h header dependency Date: Tue, 29 Jul 2008 16:23:29 +0200 Message-ID: <1217341409-755-1-git-send-email-ehrhardt@linux.vnet.ibm.com> Cc: hollisb-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org, avi-atKUWr5tajBWk0Htik3J/w@public.gmane.org To: kvm-ppc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org Return-path: Sender: kvm-ppc-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: kvm.vger.kernel.org From: Christian Ehrhardt I currently work on paravirtual extensions for powerpc and found that the current header dependencies create a classic "chicken or egg problem" leading to the point that you can't use definitions out of kvm_host.h in kvm_para.h I found it because I wanted to use kvm_vcpu, after looking around I found that neither kvm_host.h nor any file including it today needs anything from kvm_para.h. C Files needing definitions of kvm_para.h should include it diretly anyway instead of kvm_host.h and hoping for nice follow up includes in the header. Compilation of the patch tested for x86, powerpc and s390 (thanks to Christian Borntraeger). Easy solution to allow e.g. kvm_vcpu in kvm_para.h, but I might overlook something, comments welcome. Signed-off-by: Christian Ehrhardt --- [diffstat] kvm_host.h | 1 - 1 file changed, 1 deletion(-) [diff] diff --git a/include/linux/kvm_host.h b/include/linux/kvm_host.h index a18aaad..af29039 100644 --- a/include/linux/kvm_host.h +++ b/include/linux/kvm_host.h @@ -19,7 +19,6 @@ #include #include -#include #include -- To unsubscribe from this list: send the line "unsubscribe kvm-ppc" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html