Re: [Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-18 Thread Igor Mammedov
On Tue, 18 Oct 2016 08:39:05 -0200 Eduardo Habkost wrote: > On Tue, Oct 18, 2016 at 11:12:04AM +0200, Igor Mammedov wrote: > > On Mon, 17 Oct 2016 19:44:52 -0200 > > Eduardo Habkost wrote: > > > > > On Thu, Oct 13, 2016 at 11:52:39AM +0200, Igor

Re: [Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-18 Thread Eduardo Habkost
On Tue, Oct 18, 2016 at 11:12:04AM +0200, Igor Mammedov wrote: > On Mon, 17 Oct 2016 19:44:52 -0200 > Eduardo Habkost wrote: > > > On Thu, Oct 13, 2016 at 11:52:39AM +0200, Igor Mammedov wrote: > > [...] > > > @@ -236,7 +237,11 @@ void build_legacy_cpu_hotplug_aml(Aml *ctx,

Re: [Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-18 Thread Eduardo Habkost
On Tue, Oct 18, 2016 at 11:02:54AM +0200, Igor Mammedov wrote: > On Mon, 17 Oct 2016 19:44:52 -0200 > Eduardo Habkost wrote: > > > On Thu, Oct 13, 2016 at 11:52:39AM +0200, Igor Mammedov wrote: > > [...] > > > @@ -236,7 +237,11 @@ void build_legacy_cpu_hotplug_aml(Aml *ctx,

Re: [Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-18 Thread Igor Mammedov
On Mon, 17 Oct 2016 19:44:52 -0200 Eduardo Habkost wrote: > On Thu, Oct 13, 2016 at 11:52:39AM +0200, Igor Mammedov wrote: > [...] > > @@ -236,7 +237,11 @@ void build_legacy_cpu_hotplug_aml(Aml *ctx, > > MachineState *machine, > > /* The current AML generator can cover

Re: [Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-18 Thread Igor Mammedov
On Mon, 17 Oct 2016 19:44:52 -0200 Eduardo Habkost wrote: > On Thu, Oct 13, 2016 at 11:52:39AM +0200, Igor Mammedov wrote: > [...] > > @@ -236,7 +237,11 @@ void build_legacy_cpu_hotplug_aml(Aml *ctx, > > MachineState *machine, > > /* The current AML generator can cover

Re: [Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-17 Thread Eduardo Habkost
On Thu, Oct 13, 2016 at 11:52:39AM +0200, Igor Mammedov wrote: [...] > @@ -236,7 +237,11 @@ void build_legacy_cpu_hotplug_aml(Aml *ctx, MachineState > *machine, > /* The current AML generator can cover the APIC ID range [0..255], > * inclusive, for VCPU hotplug. */ >

[Qemu-devel] [PATCH v3 05/13] pc: leave max apic_id_limit only in legacy cpu hotplug code

2016-10-13 Thread Igor Mammedov
that's enough to make old code that depends on it to prevent QEMU starting with more than 255 CPUs. Signed-off-by: Igor Mammedov --- hw/acpi/cpu_hotplug.c | 7 ++- hw/i386/pc.c | 7 --- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git