Re: CVS commit: src/sys

[Antti Kantee <pooka%netbsd.org@localhost>]

On Mon Jan 05 2009 at 17:49:57 -0500, Arnaud Lacombe wrote:
> On Mon, Jan 5, 2009 at 5:43 PM, Antti Kantee <pooka%netbsd.org@localhost> 
> wrote:
> > On Mon Jan 05 2009 at 17:42:23 -0500, Arnaud Lacombe wrote:
> >> On Mon, Jan 5, 2009 at 4:42 PM, Antti Kantee <pooka%netbsd.org@localhost> 
> >> wrote:
> >> >
> >> > Module Name:    src
> >> > Committed By:   pooka
> >> > Date:           Mon Jan  5 21:42:37 UTC 2009
> >> >
> >> > Modified Files:
> >> >        src/sys/kern: kern_malloc.c
> >> >        src/sys/rump/librump/rumpkern: Makefile.rumpkern emul.c
> >> >        src/sys/rump/librump/rumpuser: Makefile rumpuser.c
> >> >        src/sys/sys: malloc.h
> >> >
> >> > Log Message:
> >> > Rename malloc() to kern_malloc() to avoid name conflict with libc.
> >> > Now it is possible to use the kernel linker with rump.
> >> >
> >> Isn't that a bit too intrusive ? There is plenty of way to avoid name
> >> conflict without touch the function name.
> >
> > Intrusive to what?
> >
> to the kernel itself. rump(4) is an abstraction layer. It should not
> require such change and should not lead to bump of the kernel version.

Do you have a technical argument or are you handwaving just for fun?

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.