[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 20/21] qemu-kvm: Move kvm_set_boot_cpu_id
From: |
Jan Kiszka |
Subject: |
[Qemu-devel] Re: [PATCH 20/21] qemu-kvm: Move kvm_set_boot_cpu_id |
Date: |
Tue, 02 Feb 2010 15:20:02 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Gleb Natapov wrote:
> On Tue, Feb 02, 2010 at 09:19:06AM +0100, Jan Kiszka wrote:
>> Setting the boot CPU ID is arch-specific KVM stuff. So push it where it
>> belongs to.
>>
> pc_init1 is also arch-specific, no? TCG should also be able to
> have BSP apic_id != 0.
But not kvm-specific.
I don't understand your second remark. Can you help me how TCG is
affected by kvm_set_boot_cpu_id?
>
>> Signed-off-by: Jan Kiszka <address@hidden>
>> ---
>> hw/pc.c | 3 ---
>> qemu-kvm-x86.c | 3 ++-
>> 2 files changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/hw/pc.c b/hw/pc.c
>> index 6c15a9f..3df6195 100644
>> --- a/hw/pc.c
>> +++ b/hw/pc.c
>> @@ -803,9 +803,6 @@ static void pc_init1(ram_addr_t ram_size,
>> #endif
>> }
>>
>> - if (kvm_enabled()) {
>> - kvm_set_boot_cpu_id(0);
>> - }
>> for (i = 0; i < smp_cpus; i++) {
>> env = pc_new_cpu(cpu_model);
>> }
>> diff --git a/qemu-kvm-x86.c b/qemu-kvm-x86.c
>> index 9de018e..0f34451 100644
>> --- a/qemu-kvm-x86.c
>> +++ b/qemu-kvm-x86.c
>> @@ -695,7 +695,8 @@ int kvm_arch_qemu_create_context(void)
>> if (kvm_check_extension(kvm_state, KVM_CAP_ADJUST_CLOCK))
>> vmstate_register(0, &vmstate_kvmclock, &kvmclock_data);
>> #endif
>> - return 0;
>> +
>> + return kvm_set_boot_cpu_id(0);
>> }
>>
>> static void set_msr_entry(struct kvm_msr_entry *entry, uint32_t index,
>> --
>> 1.6.0.2
>>
>> --
>> To unsubscribe from this list: send the line "unsubscribe kvm" in
>> the body of a message to address@hidden
>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
> --
> Gleb.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
[Qemu-devel] [PATCH 01/21] qemu-kvm: Drop vmport changes, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 18/21] qemu-kvm: Cleanup/fix TSC and PV clock writeback, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 17/21] qemu-kvm: Use VCPU event state for reset and vmsave/load, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 16/21] KVM: x86: Restrict writeback of VCPU state, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 09/21] qemu-kvm: Use upstream kvm_pit_in_kernel, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 10/21] KVM: Move and rename regs_modified, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 12/21] qemu-kvm: Use upstream kvm_vcpu_dirty, Jan Kiszka, 2010/02/02
[Qemu-devel] [PATCH 11/21] KVM: Rework of guest debug state writing, Jan Kiszka, 2010/02/02