Subject: Re: CVS commit: syssrc/sys/sys
To: Klaus Klein <email@example.com>
From: Todd Vierling <firstname.lastname@example.org>
Date: 07/06/2001 11:25:50
On 6 Jul 2001, Klaus Klein wrote:
: > bcopy() is documented as allowing overlapping memory regions. Define it
: > in terms of the standard name memmove() to account for this.
: > Fixes kern/13369
: 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
Yes, I know. However, PR kern/13369 was created because of kernel source
code which used bcopy under the assumptions listed in bcopy(3), since we
don't have a bcopy(9).
The other alternatives to this change are:
1. reintroduce ovbcopy() as a macro,
2. eliminate all uses of bcopy() in the kernel (incl. imported sources),
3. create a bcopy(9) manpage which describes the differences.
I'd rather see most instances of bcopy() eliminated (partial #2) here, and
then Just Deal with the slight overhead of the bcopy() compatibility
#define. This is the intent of my commit.
My second choice would be to create the manpage as in #3.
-- Todd Vierling <email@example.com> * Wasabi NetBSD: Run with it.
-- NetBSD 1.5 now available on CD-ROM -- http://www.wasabisystems.com/