Re: [Xen-devel] [PATCH] x86/apic: remove the legacy bigsmp APIC driver

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 27.08.13 at 07:44, Matt Wilson <msw@xxxxxxxxxx> wrote:
> The bigsmp APIC driver was used on legacy (32-bit only) x86 platforms
> with 8 or more CPUs. In Linux the X86_BIGSMP Kconfig option depends on
> X86_32. Since we no longer support 32-bit hypervisors on the x86
> platform we can remove this code.

The fact that this is 32-bit only in Linux doesn't mean anything.
Linux has other 64-bit only models, which we don't have (in
favor of, originally, having a single shared implementation).

On a system without x2APIC but more than 8 CPUs - how would
you see us bringing up the CPUs beyond 8? This is particularly
shown by your adjustment to MP_processor_info_x(), which is
code removal without replacement.

Despite Keir's ack, unless I'm overlooking something I'm explicitly
nack-ing this.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel