Source-Changes archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/sys

On Mon, Jan 5, 2009 at 6:40 PM, Antti Kantee <> wrote:
> Do you have a technical argument or are you handwaving just for fun?
do I look to handwave just for fun ?

A technical point ? Let's see... I am a module and I reference
malloc() (or free()). You're right, we don't have shipped a release
with MODULAR yet so this is not valid. I can always argue that using a
macro to rename a function is ugly, that it mislead tools like cscope
to point to the wrong definition of malloc(). Or the fact that memory
allocator was already a hell, now, you will have malloc() which is not
really malloc() but an alias to kern_malloc(), but also kmem_alloc().

Btw, your commit log is incomplete as you also modified free. I keep
the best for the end: you are breaking all code that reference free or
malloc (the symbol, not the function).

> This change solves a real problem.  If you have a way to provide malloc
> in the kernel symbol table so that the kernel linker picks it up, provide
> the kernel ABI and preserve the ability to call libc malloc portably,
> please present it.
I'm not developing rump, you are.

 - Arnaud

Home | Main Index | Thread Index | Old Index