Date: Mon, 11 Jul 2016 11:04:20 -0700 From: Mark Millard <markmi@dsl-only.net> To: Nathan Whitehorn <nwhitehorn@freebsd.org> Cc: FreeBSD PowerPC ML <freebsd-ppc@freebsd.org>, freebsd-stable@freebsd.org, FreeBSD Current <freebsd-current@freebsd.org> Subject: Re: stable/11 question: kboot vs. powerpc: only powerpc64? Message-ID: <73A6C8D3-6733-4E75-9C5C-59F8A5BB5039@dsl-only.net> In-Reply-To: <5783A3F5.2030301@freebsd.org> References: <07312336-F627-4372-AEF2-5FA93CF6E4CD@dsl-only.net> <5783A3F5.2030301@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 2016-Jul-11, at 6:49 AM, Nathan Whitehorn <nwhitehorn@freebsd.org> =
wrote:
>=20
> It is not 64-bit only; like the normal loader, it can load both 32-bit =
and 64-bit kernels. Those two flags are probably obsolete at this point =
and were for compatibility with pre-2.17.5 versions of binutils. Can you =
do a test build with the -CFLAGS+=3D -Wa,-mppc64bridge line removed?
> -Nathan
>=20
> On 07/11/16 03:55, Mark Millard wrote:
>> Is the following something that should be updated something like is =
indicated below for 11.0-BETA1? Is kboot powerpc64 specific?
>>=20
>> # svnlite diff /usr/src/sys/boot/powerpc/Makefile
>> Index: /usr/src/sys/boot/powerpc/Makefile
>> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>> --- /usr/src/sys/boot/powerpc/Makefile (revision 302457)
>> +++ /usr/src/sys/boot/powerpc/Makefile (working copy)
>> @@ -1,5 +1,9 @@
>> # $FreeBSD$
>> -SUBDIR=3D boot1.chrp kboot ofw ps3 uboot
>> +SUBDIR=3D boot1.chrp
>> +.if ${MACHINE_ARCH} =3D=3D "powerpc64"
>> +SUBDIR+=3D kboot
>> +.endif
>> +SUBDIR+=3D ofw ps3 uboot
>> .include <bsd.subdir.mk>
>>=20
>>=20
>>=20
>> I ask because I'd submitted 206303 back on 2016-jan-16 reporting that =
TARGET_ARCH=3Dpowerpc WITH_BOOT=3D was stopped by getting a =
-Wc,-mppc64bride and a -mcpu=3Dpowerpc64 (one of the =
base/head/sys/boot/powerpc/kboot/Makefile SRCS being =
ppc64_elf_freebsd.c).
>>=20
>> =3D=3D=3D
>> Mark Millard
>> markmi at dsl-only.net
I do not have access to powerpc's currently so I'm just going to be =
doing cross-build tests for TARGET_ARCH=3Dpowerpc and =
TARGET_ARCH=3Dpowerpc64 (from amd64) based on the below updates.
You initially mention "two flags" but then only explicitly request =
removal of one (the -CFLAGS+=3D -Wa,-mppc64bridge line).
I'm assuming that the -mcpu=3Dpowerpc64 is also to be removed if powerpc =
(non-64) is to be covered. See my intended test below. Let me know if it =
is not what you want.=20
> # svnlite diff sys/boot/powerpc/kboot/Makefile
> Index: sys/boot/powerpc/kboot/Makefile
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- sys/boot/powerpc/kboot/Makefile (revision 302457)
> +++ sys/boot/powerpc/kboot/Makefile (working copy)
> @@ -71,7 +71,7 @@
> # Avoid the open-close-dance for every file access as some firmwares =
perform
> # an auto-negotiation on every open of the network interface and thus =
causes
> # netbooting to take horribly long.
> -CFLAGS+=3D -DNETIF_OPEN_CLOSE_ONCE -mcpu=3Dpowerpc64
> +CFLAGS+=3D -DNETIF_OPEN_CLOSE_ONCE
> =20
> # Always add MI sources
> .PATH: ${.CURDIR}/../../common =
${.CURDIR}/../../../libkern
> @@ -88,9 +88,6 @@
> =20
> LDFLAGS=3D -nostdlib -static -T ${.CURDIR}/ldscript.powerpc
> =20
> -# 64-bit bridge extensions
> -CFLAGS+=3D -Wa,-mppc64bridge
> -
> # Pull in common loader code
> #.PATH: ${.CURDIR}/../../ofw/common
> #.include "${.CURDIR}/../../ofw/common/Makefile.inc"
> # svnlite diff sys/boot/powerpc/Makefile
> #=20
(I.e., I reverted sys/boot/powerpc/Makefile.)
=3D=3D=3D
Mark Millard
markmi at dsl-only.net
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?73A6C8D3-6733-4E75-9C5C-59F8A5BB5039>