Re: CVS commit: src

[David Brownlee <abs%NetBSD.org@localhost>]

On Sat, 14 Mar 2009, Takeshi Nakayama wrote:

> > > > David Brownlee <abs%netbsd.org@localhost> wrote
>
> > Module Name:    src
> > Committed By:   abs
> > Date:           Thu Mar 12 17:16:59 UTC 2009
> >
> > Modified Files:
> >         src/common/lib/libc/arch/sparc/atomic: Makefile.inc
> >         src/compat: Makefile.compat
> >         src/etc/etc.sparc64: Makefile.inc
> >         src/sys/arch/i386/stand/boot: Makefile.boot
> >         src/sys/arch/i386/stand/bootxx: Makefile.bootxx
> >         src/sys/arch/i386/stand/fatboot: Makefile.fat
> >         src/sys/arch/i386/stand/mbr: Makefile.mbr
> >
> > Log Message:
> > Prefer MACHINE_ARCH to MACHINE in some tests
>
> Please revert the change in common/lib/libc/arch/sparc/atomic/Makefile.inc.
>
> The test intend to check whether kernel use SPARC V9 instructions
> or V7/V8 ones.  We must use V9 ones in case of 32-bit sparc64
> kernels (MACHINE_ARCH=sparc MACHINE=sparc64), so using MACHINE is
> correct.

        Reverted - many thanks for picking up on that. Have also added
        a comment explaining the (odd) situation :)

--
                David/absolute       -- www.NetBSD.org: No hype required --