Date: Wed, 22 Feb 2006 14:01:55 +0530
From: Srivatsa Vaddagiri <vatsa@in.ibm.com>
To: Martin Peschke <mp3@de.ibm.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: How to allocate per-cpu data for online CPUs only (and safely)?
Message-ID: <20060222083155.GB5111@in.ibm.com>
Reply-To: vatsa@in.ibm.com
References: <43FB942E.5070309@de.ibm.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <43FB942E.5070309@de.ibm.com>
User-Agent: Mutt/1.5.11
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1922
Lines: 50

On Tue, Feb 21, 2006 at 11:29:02PM +0100, Martin Peschke wrote:
> I am trying to optimize the memory footprint of some code of mine.
> the disadvantage of getting hold of memory for CPU's which aren't there
> (yet).
> 
> I could imagine using CPU-hotplug notifications as triggers for
> additional allocations or for cleaning up unneeded memory. But
> alloc_percpu() appears to conflict with that idea.
> 
> I was briefly tempted to derive some code from alloc_percpu() more to my
> liking, until I was scared off by this comment in alloc_percpu():
> 
>         /*
>          * Cannot use for_each_online_cpu since a cpu may come online
>          * and we have no way of figuring out how to fix the array
>          * that we have allocated then....
>          */
> 
> Well, and then there is kernel/profile.c, for example, which boldly
> ignors alloc_percpu()'s qualms and allocates and releases per-cpu data
> as needed.
> 
> Is that the way to go?
> If so, why alloc_percpu()'s reservations?

The above comment in alloc_percpu regarding use of for_each_online_cpu
probably refers to the fact that once we handover 'pdata' to the caller
of alloc_percpu, there is no way to go back and fix it (i.e allocate
objects) for CPUs that come online later. This could have been possible if 
alloc_percpu kept track of all per-CPU allocations (chain togther all pdata's 
in a linked list?), but I guess they didn't do it to keep it simple.

I would say the code in kernel/profile.c is fine enough to emulate.

> Or, does that comment imply that the exploiter isn't expected to take
> care of CPU hotplug events?
> Am I missing anything?
> 
> Thank you.
> 
> Martin

-- 
Regards,
vatsa
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/