Re: CVS commit: syssrc/sys/sys

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

In article <87d77fdmo3.fsf%minor-variation.er.reziprozitaet.de@localhost>,
Klaus Klein <kleink%reziprozitaet.de@localhost> wrote:

I agree with klaus. Please back this out, because it introduces overhead
that is unneeded in 99% of the usage.

christos

>Todd Vierling <tv%netbsd.org@localhost> writes:
>
>> Module Name: syssrc
>> Committed By:        tv
>> Date:                Thu Jul  5 21:51:10 UTC 2001
>> 
>> Modified Files:
>>      syssrc/sys/sys: systm.h
>> 
>> Log Message:
>> bcopy() is documented as allowing overlapping memory regions.  Define it
>> in terms of the standard name memmove() to account for this.
>> Fixes kern/13369 and the redundant lib/13370.
>
>This change looks wrong to me.  While the libc bcopy() implementation
>is indeed documented as allowing overlapping memory objects, this is
>not the case for its kernel pendant, which led (historically) to the
>alternate version ovbcopy() being used for possibly overlapping
>objects.
>
>
>- Klaus