[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <86802c440803061123h2f2232d4jca7b81d7d2e4ca66@mail.gmail.com>
Date: Thu, 6 Mar 2008 11:23:09 -0800
From: "Yinghai Lu" <yhlu.kernel@...il.com>
To: "Andi Kleen" <andi@...stfloor.org>
Cc: "Ingo Molnar" <mingo@...e.hu>,
"Andrew Morton" <akpm@...ux-foundation.org>,
"Thomas Gleixner" <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...or.com>,
"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] x86_64: show apicid for cpu in proc
On 06 Mar 2008 12:41:46 +0100, Andi Kleen <andi@...stfloor.org> wrote:
> "Yinghai Lu" <yhlu.kernel@...il.com> writes:
>
> > cpus_weight(per_cpu(cpu_core_map, cpu)));
> > seq_printf(m, "core id\t\t: %d\n", c->cpu_core_id);
> > seq_printf(m, "cpu cores\t: %d\n", c->booted_cores);
>
> > + seq_printf(m, "apicid\t: %02x\n", c->apicid);
>
> Seems a little redundant with the boot log, but ok.
>
> But it's unclear to the user which apic ID is meant by that. There are
> two different ones. The one in the LAPIC mapping and the one reported by
> CPUID. They are not necessarily the same. I think you should clarify that
> by renaming the field or perhaps report both.
report the apicid and initial apicid at the same time?
but in current code, we didn't use the initial apicid...., actually
initial apic id is more better to get the responding to node/core.
will look at it.
YH
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists