Re: CVS commit: src/sys/lib/libkern

[christos%astron.com@localhost (Christos Zoulas)]

In article <20130311101612.GD29198%sirion.coris.org.uk@localhost>,
Julian Coleman  <jdc%coris.org.uk@localhost> wrote:
>Hi,
>
>> Module Name: src
>> Committed By:        christos
>> Date:                Sun Mar 10 07:31:03 UTC 2013
>> 
>> Modified Files:
>>      src/sys/lib/libkern: Makefile.libkern
>> 
>> Log Message:
>> On sparc64 memcpy and memset are provided by memcpyset.s (should have been
>> memcpyset.S, but...). Don't include them in the build because it breaks the
>> modular build where the kernel library is built as an object.
>
>This breaks the build on sparc64:
>
>  .../librumpfs_cd9660.so: undefined reference to `rumpns_memset'
>  .../librumpfs_cd9660.so: undefined reference to `rumpns_memcpy'
>
>It looks like we didn't previously use the assembler source for rump on
>sparc64.  Is the fix to use the assembler for rump, or to add the C files
>to librump/rumpkern/Makefile.rumpkern only for sparc64?

Whatever the fix is, it should be concentrated in the libkern Makefile
so that there are no moving parts to it.

christos