2005-01-10 20:03:09

by YhLu

[permalink] [raw]
Subject: RE: 256 apic id for amd64

Try this one.

-----Original Message-----
From: Andi Kleen [mailto:[email protected]]
Sent: Monday, January 10, 2005 11:44 AM
To: YhLu
Cc: 'Mikael Pettersson'; [email protected]; [email protected];
[email protected]; [email protected]; [email protected]
Subject: Re: 256 apic id for amd64

On Mon, Jan 10, 2005 at 11:41:02AM -0800, YhLu wrote:
> Please refer the patch.

That's unreadable. Can you generate against a recent BK snapshot
(2.6.10 + latest from ftp://ftp.kernel.org/pub/linux/kernel/v2.6/snapshots/)

and use diff -u ?

-Andi


Attachments:
x86_64_phy_proc_id.patch (1.11 kB)

2005-01-10 20:20:10

by Andi Kleen

[permalink] [raw]
Subject: Re: 256 apic id for amd64

On Mon, Jan 10, 2005 at 12:09:48PM -0800, YhLu wrote:
> Try this one.

I don't think it will work at all on Intel HT systems, since
nobody initializes c->x86_num_cores there. phys_proc_id[]
is supposed to be the same on two HT siblings.

You'll either need to initialize c->x86_num_cores on Intel too.
But since Intel seems to have dual cores upcomming and you'll
want HyperThreading support too it would need to be a new field.

Alternatively you can split the function for AMD and Intel and
use the new algorithm on AMD only. Perhaps that's better.
In the later case I would only use it when CMP_LEGACY is set.

-Andi