Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:8 as permitted sender) client-ip=2620:137:e000::3:8;
MIME-Version: 1.0
References: <20231211172504.058ad6b6@canb.auug.org.au> <9f0763c0-8175-4160-b72e-8e9e3aee42f8@infradead.org>
 <CAHVXubjbVxEvL01uu_Cm-_R8oLddwc6fUaQYE_Bc+sBThTmvWA@mail.gmail.com> <20231213-excretory-ashen-03a34f3a4c90@spud>
In-Reply-To: <20231213-excretory-ashen-03a34f3a4c90@spud>
From:   Alexandre Ghiti <alexghiti@rivosinc.com>
Date:   Wed, 13 Dec 2023 14:33:50 +0100
Message-ID: <CAHVXubga7Q6X-zW_qztS2V_Z8TBp0gm2UV_GAj2O-5nn-_yCyA@mail.gmail.com>
Subject: Re: linux-next: Tree for Dec 11 (riscv32: patch.c)
To:     Conor Dooley <conor@kernel.org>
Cc:     Randy Dunlap <rdunlap@infradead.org>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        Linux Next Mailing List <linux-next@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-riscv <linux-riscv@lists.infradead.org>,
        Palmer Dabbelt <palmer@dabbelt.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Wed, Dec 13, 2023 at 1:57=E2=80=AFPM Conor Dooley <conor@kernel.org> wro=
te:
>
> On Tue, Dec 12, 2023 at 09:01:17AM +0100, Alexandre Ghiti wrote:
> > Hi Randy,
> >
> > On Mon, Dec 11, 2023 at 10:17=E2=80=AFPM Randy Dunlap <rdunlap@infradea=
d.org> wrote:
> > >
> > >
> > >
> > > On 12/10/23 22:25, Stephen Rothwell wrote:
> > > > Hi all,
> > > >
> > > > Changes since 20231208:
> > > >
> > >
> > > on riscv32:
> > >
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L4':
> > > patch.c:(.text+0xbe): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0xc2): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L12':
> > > patch.c:(.text+0xd2): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0xd6): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L15':
> > > patch.c:(.text+0x1ea): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L7':
> > > patch.c:(.text+0x1ee): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0x1fe): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L22':
> > > patch.c:(.text+0x202): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `__patch_insn_write':
> > > patch.c:(.text+0x410): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0x414): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0x424): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0x428): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L46':
> > > patch.c:(.text+0x53c): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0x540): undefined reference to `__exittext_begin'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
arch/riscv/kernel/patch.o: in function `.L48':
> > > patch.c:(.text+0x550): undefined reference to `__exittext_end'
> > > /opt/crosstool/gcc-13.2.0-nolibc/riscv32-linux/bin/riscv32-linux-ld: =
patch.c:(.text+0x554): undefined reference to `__exittext_end'
> > >
> > >
> > > Full randconfig file is attached.
> >
> > We can ignore this one as CONFIG_XIP is enabled and we agreed at LPC
> > that I'll remove this soon.
>
> I don't think we can (even if people hadn't sent patches yesterday for
> XIP) act as if XIP being removed is a foregone conclusion.

Ok, the fix is easy enough for me to send it right now, so I'll fix that :)