In message <20030507055103.GA31797@in.ibm.com> you write:
> I tried to run a test to compare this implementation, but got an oops.
> Here is the oops and the patch I was trying...
> + if (!init_committed_space)
init_committed_space is a function. You meant to call it 8)
> btw, why the change from kmalloc_percpu(size) to kmalloc_percpu(type)?
> You do kmalloc(sizeof (long)) for the usual kmalloc, but
> kmalloc_percpu(long) for percpu data...looks strange no?
Yes, I'd probably want to change the name, if Andrew had agreed to the
concept. But the type is very convenient, because you want to know
the alignment (kmalloc doesn't care, it just pads to cachline).
Cheers,
Rusty.
-- Anyone who quotes me in their sig is an idiot. -- Rusty Russell. - 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/
This archive was generated by hypermail 2b29 : Wed May 07 2003 - 22:00:34 EST