[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 2/5] target-i386: Remove unused APIC ID defau
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH v4 2/5] target-i386: Remove unused APIC ID default code |
Date: |
Fri, 6 Mar 2015 17:27:29 -0300 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Thu, Mar 05, 2015 at 03:46:13PM -0300, Eduardo Habkost wrote:
> 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 <address@hidden>
> > >
> > > 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 <address@hidden>
> >
> > 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)
I just learned from another thread that Peter will be back to work on
March 10th, so I will submit the pull request on Monday.
--
Eduardo
[Qemu-devel] [PATCH v4 4/5] target-i386: Move APIC ID compatibility code to pc.c, Eduardo Habkost, 2015/03/03
[Qemu-devel] [PATCH v4 5/5] target-i386: Require APIC ID to be explicitly set before CPU realize, Eduardo Habkost, 2015/03/03
[Qemu-devel] [PATCH v4 3/5] target-i386: Move CPUX86State.cpuid_apic_id to X86CPU.apic_id, Eduardo Habkost, 2015/03/03