On 16.05.2016 18:59, Peter Maydell wrote:
> On 16 May 2016 at 17:53, Peter Maydell <peter.mayd...@linaro.org> wrote:
>> ppc64 (this is the ppc64be host in the GCC compile farm if
>> you have an account there):
>>
>> /home/pm215/qemu/hw/intc/xics_kvm.c: In function ‘icp_get_kvm_state’:
>> /home/pm215/qemu/hw/intc/xics_kvm.c:54:12: error: variable ‘reg’ has
>> initializer but incomplete type
>> struct kvm_one_reg reg = {
>> ^
>> /home/pm215/qemu/hw/intc/xics_kvm.c:55:9: error: unknown field ‘id’
>> specified in initializer
>> .id = KVM_REG_PPC_ICP_STATE,
>> ^
>> /home/pm215/qemu/hw/intc/xics_kvm.c:55:15: error:
>> ‘KVM_REG_PPC_ICP_STATE’ undeclared (first use in this function)
>> .id = KVM_REG_PPC_ICP_STATE,
>> ^
>>
>> etc -- looks like missing a kvm include somewhere.
>
> I logged in by hand to do a -k build; the only other error was:
> /home/pm215/qemu/hw/ppc/spapr_pci.c: In function ‘spapr_phb_realize’:
> /home/pm215/qemu/hw/ppc/spapr_pci.c:1426:5: error: implicit
> declaration of function ‘kvm_enabled’
> [-Werror=implicit-function-declaration]
> if (kvm_enabled()) {
> ^
> /home/pm215/qemu/hw/ppc/spapr_pci.c:1426:5: error: nested extern
> declaration of ‘kvm_enabled’ [-Werror=nested-externs]
I've checked the build on our POWER server, too, and I get the same
problems. You can fix it with this patch:
diff --git a/hw/intc/xics_kvm.c b/hw/intc/xics_kvm.c
index 9029d9e..55fd801 100644
--- a/hw/intc/xics_kvm.c
+++ b/hw/intc/xics_kvm.c
@@ -31,6 +31,7 @@
#include "cpu.h"
#include "hw/hw.h"
#include "trace.h"
+#include "sysemu/kvm.h"
#include "hw/ppc/spapr.h"
#include "hw/ppc/xics.h"
#include "kvm_ppc.h"
diff --git a/hw/ppc/spapr_pci.c b/hw/ppc/spapr_pci.c
index 573e635..d4bcb5a 100644
--- a/hw/ppc/spapr_pci.c
+++ b/hw/ppc/spapr_pci.c
@@ -44,7 +44,7 @@
#include "hw/pci/pci_bus.h"
#include "hw/ppc/spapr_drc.h"
#include "sysemu/device_tree.h"
-
+#include "sysemu/kvm.h"
#include "hw/vfio/vfio.h"
/* Copied from the kernel arch/powerpc/platforms/pseries/msi.c */
(Note: In xics_kvm.c, the sysemu/kvm.h header file has to be included
before the kvm_ppc.h header file to avoid some other conflicts).
Thomas
