Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 21 Nov 2019 00:58:07 +0800
From:      Li-Wen Hsu <lwhsu@freebsd.org>
To:        John Baldwin <jhb@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all <svn-src-all@freebsd.org>,  svn-src-head <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r354899 - head/usr.sbin/jail
Message-ID:  <CAKBkRUyzxhFDv1Yugaz=%2BK-RsfHYvutuNBdQP4iMeTKt=teH9Q@mail.gmail.com>
In-Reply-To: <b313e534-b1d2-88fd-0830-1509c983d5b3@FreeBSD.org>
References:  <201911201635.xAKGZwT4082365@repo.freebsd.org> <b313e534-b1d2-88fd-0830-1509c983d5b3@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Nov 21, 2019 at 12:42 AM John Baldwin <jhb@freebsd.org> wrote:
>
> On 11/20/19 8:35 AM, Li-Wen Hsu wrote:
> > Author: lwhsu
> > Date: Wed Nov 20 16:35:58 2019
> > New Revision: 354899
> > URL: https://svnweb.freebsd.org/changeset/base/354899
> >
> > Log:
> >   Limit the workaround to riscv only
> >
> >   PR:         242109
> >   Sponsored by:       The FreeBSD Foundation
> >
> > Modified:
> >   head/usr.sbin/jail/Makefile
> >
> > Modified: head/usr.sbin/jail/Makefile
> > ==============================================================================
> > --- head/usr.sbin/jail/Makefile       Wed Nov 20 16:32:13 2019        (r354898)
> > +++ head/usr.sbin/jail/Makefile       Wed Nov 20 16:35:58 2019        (r354899)
> > @@ -18,7 +18,9 @@ CFLAGS+=-I. -I${.CURDIR}
> >  # workaround for GNU ld (GNU Binutils) 2.33.1:
> >  #   relocation truncated to fit: R_RISCV_GPREL_I against `.LANCHOR2'
> >  # https://bugs.freebsd.org/242109
> > +.if ${MACHINE_ARCH} == "riscv"
> >  CFLAGS+=-Wl,--no-relax
> > +.endif
>
> Eh, will that work?  I think MACHINE and MACHINE_CPUARCH are riscv,
> but MACHINE_ARCH is riscv64 and riscv64sf.

No it doesn't. I was too lazy to do scp from test machine to commit
machine and caused a copy-n-paste error, but ${MACHINE} should be
better since it covers more.

> Also, it would be good to wrap this in .if ${LINKER_TYPE} == "bfd" I think.

Also added in r354900. Thanks for the suggestion.

> (I was able to build a world + kernel with lld earlier this week, though it
>  doesn't yet boot)

Looking forward to building world & kernel with llvm toolchain.

Thanks,
Li-Wen



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAKBkRUyzxhFDv1Yugaz=%2BK-RsfHYvutuNBdQP4iMeTKt=teH9Q>