On Thu, Mar 05, 2015 at 07:35:17PM +0100, Andreas Färber wrote: > Am 05.03.2015 um 14:43 schrieb Eduardo Habkost: > > On Tue, Mar 03, 2015 at 11:13:41PM -0300, Eduardo Habkost wrote: > >> The existing apic_id = cpu_index code has no visible effect: the PC code > >> already initializes the APIC ID according to the topology on > >> pc_new_cpu(), and linux-user memcpy()s the CPU state (including > >> cpuid_apic_id) on cpu_copy(). > >> > >> Remove the dead code and simply let APIC ID to to be 0 by default. This > >> doesn't change behavior of PC because apic-id is already explicitly set, > >> and doesn't affect linux-user because APIC ID was already always 0. > >> > >> Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> > > > > This patch is holding the rest of the series, so a Reviewed-by or > > Acked-by would be welcome. > > > > This change removes the 254-CPU limit from {i386,x86_64}-linux-user that > > Peter and I discussed previously. > > Reviewed-by: Andreas Färber <afaer...@suse.de> > > Are you going to send a new pull for the 2 plus these 5 now?
Yes. I plan to send a pull request tomorrow. (If we get reviews in time, the pull request may include the instance_init series as well) -- Eduardo