[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <79E93560F4A5FD42BB769DAAF8BEF62A01ADBF81@scsmsx411.amr.corp.intel.com>
Date: Fri, 25 May 2007 08:24:12 -0700
From: "Yu, Fenghua" <fenghua.yu@...el.com>
To: "Russell King" <rmk+lkml@....linux.org.uk>
Cc: "Andrew Morton" <akpm@...ux-foundation.org>,
"Ravikiran G Thirumalai" <kiran@...lex86.org>,
"Siddha, Suresh B" <suresh.b.siddha@...el.com>, <clameter@....com>,
<linux-kernel@...r.kernel.org>, <schwidefsky@...ibm.com>,
<heiko.carstens@...ibm.com>
Subject: RE: [PATCH 1/2] Define new percpu interface for shared data -- version 3
>What's "shared percpu data" ? It sounds to me like a contradiction in
>terms. Isn't percpu data supposed to only be accessed by the CPU which
>owns it to prevent cache line bouncing? In which case, what's the
point
>of sharing that data with other CPUs? Surely "shared percpu data" is
>just the same as normal data?
Like said in the patch description:
"per cpu data section contains two types of data. One set which is
exclusively accessed by the local cpu and the other set which is per
cpu,
but also shared by remote cpus."
The patches separate shared and aligned per cpu data from others to
eliminate cache pouncing between remote and local access.
The patches do not separate shared but non-aligned percpu data, e.g.
cpu_idle_state, flush_state in x86-64. We can do that later if needed.
I think the shared percpu data are "mainly" accessed by local cpu. We
need
to concentrate on their local cpu access i.e. define in percpu. If we
treat them as "normal" data, we may lose percpu benefits e.g. numa
allocation.
Thanks.
-Fenghua
-
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