Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

re: CVS commit: src/sys/conf



Paul Goyette writes:
> On Thu, 5 Mar 2020, Paul Goyette wrote:
> 
> >> also aarch64 has no EXEC_AOUT.
> >
> > Does aarch64 have COMPAT_NETBSD32?  If so, I will adjust my fix.  (I
> > built 67 different architectures, including all of the ones that the
> > releng build cluster is building, and did not have any problems with
> > aarch64 or any other.)
> 
> As near as I can tell, aarch64 sets MACHINE_CPU to "aarch64", so it
> will not try to build COMPAT_NETBSD32 due to the following tests in
> $SRC/sys/compat/netbsd32/netbsd32.mk
> 
> .if ${MACHINE_ARCH} == "x86_64" \
>      || ${MACHINE_CPU} == "arm" \
>      || ${MACHINE_ARCH} == "sparc64" \
>      || (!empty(MACHINE_ARCH:Mmips64*) && !defined(BSD_MK_COMPAT_FILE))
> COMPAT_USE_NETBSD32?=yes
> .else
> COMPAT_USE_NETBSD32?=no
> .endif
> 
> 
> Perhaps the <<<< line should be changed to "aarch64" instead of "arm" ?

what <<< line? :-)

arm64 should be adde here, yes.  arm should remain -- it's
a fake netbsd32 in that it handles the old 32 bit ABI on
32 bit arm systems, so it's still useful.  we're not able
to support both that and EABI on arm64 currently (not sure
how it would be easy without making it possible to compile
the netbsd32 code twice, since there are additional layout
differences that are difficult to handle without letting
the compiler do it for you.)

sorry, i should have added arm64 here when i was looking
at it but i missed it.


Home | Main Index | Thread Index | Old Index