« Previous | Next » 

Revision 14a10fc3


Added by Andreas Färber over 7 years ago

cpu: Partially revert "cpu: Change qemu_init_vcpu() argument to CPUState"

Commit c643bed99 moved qemu_init_vcpu() calls to common CPUState code.
This causes x86 cpu-add to fail with "KVM: setting VAPIC address failed".

The reason for the failure is that CPUClass::kvm_fd is not yet
initialized in the following call graph:
->device_reset (hotplugged == 1)
This causes attempted KVM vCPU ioctls to fail.

By contrast, in the non-hotplug case the APIC is reset much later, when
the vCPU is already initialized.

As a quick and safe solution, move the qemu_init_vcpu() call back into
the targets' realize functions.

Reported-by: Chen Fan <>
Acked-by: Igor Mammedov <> (for i386)
Tested-by: Jia Liu <> (for openrisc)
Signed-off-by: Andreas Färber <>


  • added
  • modified
  • copied
  • renamed
  • deleted

View differences