Messages in this thread | | | From | Rusty Russell <> | Subject | Re: [PATCH] Allocate module.ref array dynamically | Date | Wed, 12 Nov 2008 20:29:23 +1030 |
| |
On Wednesday 12 November 2008 13:44:51 Christoph Lameter wrote: > Please use my new email address.... Otherwise I will not see this.
Oops, updated thanks.
> > There's something in linux-next, but I'm not sure of the status. > > Christoph, is this anticipated to make the next merge window, or am I > > best off merging a patch like Eric's for the moment? > > Yes the patch in -next is for the next merge window. This should > actually have been in .28.
Perhaps I'm missing the overarching plan here?
You've introduced a third set of per-cpu primitives, yet the second set still has 0 users.
Your new basic interface is: CPU_ALLOC/CPU_FREE/CPU_PTR/THIS_CPU/__THIS_CPU
I don't think the CAPS adds anything. I'd like to see standard docbook comments. It's not clear from your documentation whether this allocates for all possible or only all online CPUs, and the difference between THIS_CPU and __THIS_CPU is not immediately obvious.
How about re-using alloc_percpu/free_percpu/per_cpu_ptr APIs? Rename THIS_CPU to __get_cpu_ptr and implement get_cpu_ptr and put_cpu_ptr wrappers (a-la get_cpu_var).
I love this work, but I think it stumbles on the final polish. If that's just a "not done yet", I'd be happy to try to put some patches together.
Thanks! Rusty.
| |