On Fri, 10 Oct 2014 10:47:09 +0200 Cornelia Huck <cornelia.h...@de.ibm.com> wrote:
> On Thu, 9 Oct 2014 12:17:30 +0200 > Paolo Bonzini <pbonz...@redhat.com> wrote: > > > From: Eduardo Habkost <ehabk...@redhat.com> > > > > Now that we create an accel object before calling machine_init, we can > > simply use the accel object to save all KVMState data, instead of > > allocationg KVMState manually. > > > > Reviewed-by: Paolo Bonzini <pbonz...@redhat.com> > > Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > > --- > > kvm-all.c | 13 +++++++++---- > > 1 file changed, 9 insertions(+), 4 deletions(-) > > This patch breaks compilation on s390 for me: > > CC s390x-softmmu/kvm-all.o > /home/cohuck/git/qemu/kvm-all.c:110: error: redefinition of typedef ‘KVMState’ > /home/cohuck/git/qemu/include/sysemu/kvm.h:161: error: previous declaration > of ‘KVMState’ was here Seems to be compiler version dependant. With the SLES11SP3 compiler (calls itself 4.3.4), I get the failure above. With newer compilers (4.6+), it builds fine.
