public inbox for binutils@sourceware.org
 help / color / mirror / Atom feed
From: Xi Ruoyao <xry111@xry111.site>
To: Carlos O'Donell <carlos@redhat.com>,
	libc-alpha <libc-alpha@sourceware.org>
Cc: caiyinyu <caiyinyu@loongson.cn>,
	liuzhensong <liuzhensong@loongson.cn>,
	 binutils@sourceware.org, Wang Xuerui <i@xen0n.name>,
	Fangrui Song <maskray@google.com>
Subject: Re: glibc 2.36 - Slushy freeze (3 weeks to release)
Date: Tue, 12 Jul 2022 00:06:45 +0800	[thread overview]
Message-ID: <f4c1aee6565263608fd758d3a657b649525eea91.camel@xry111.site> (raw)
In-Reply-To: <fef8c3c7-fd30-d8b5-e539-f0f21d562c51@redhat.com>

+binutils because we'll have to discuss binutils-related issues.

On Mon, 2022-07-11 at 11:20 -0400, Carlos O'Donell via Libc-alpha wrote:

> Desirable:
> 
> * GLIBC LoongArch PATCHES
> 
> The LoongArch patches are currently under review, but there looks to be
> some unresolved binutils issues. Just for clarity we expect a glibc port
> to have committed patches for the linux kernel, gcc, and binutils before
> inclusion in glibc.

GCC is mostly fine.  There are some "outstanding" bugs in 12.1 but AFAIK
they don't cause issues building glibc.  You can use releases/gcc-12
branch if you have any doubt.

Kernel userspace API is fine in 5.19-rc.  There are issues about boot
protocol and some drivers but these issues are completely unrelated to
glibc.

For binutils, ld is generating strange R_LARCH_NONE relocations (caused
by an over-allocate of .rel.* sections and the usage of 0 as padding),
and ld is generating R_LARCH_IRELATIVE for .rel.plt section (Fangrui
says .rel.plt should not contain R_LARCH_IRELATIVE).

Loongson engineers seems preparing a large patch series containing
*both* the bug fix removing buggy R_LARCH_NONE and R_LARCH_IRELATIVE
relocations, *and* the implementation of many new relocation types
(superseding the current stack-based relocs which are disliked by many
people, including me).

Unfortunately, binutils 2.39 release branch is already created and I
don't think such a large change set can be reviewed and landed into
binutils soon.  So I'll repeat my suggestion again: it's better to
separate the bug fix and the new feature into two patch series, and get
the bug fix landed and backported for binutils-2.39 branch ASAP.  The
new relocs (and/or other new features) can be reviewed later for
binutils 2.40, after 2.39 release.

I don't like the stack-based relocs, maybe even more than you guys - I
remember I'd shout loudly with "colorful metaphors" when I had to use
these relocs in LLVM.  However another binutils release "supporting"
LoongArch but completely unusable in practice will be worse.  (Remind:
2.38 is already such a release.)

       reply	other threads:[~2022-07-11 16:06 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <fef8c3c7-fd30-d8b5-e539-f0f21d562c51@redhat.com>
2022-07-11 16:06 ` Xi Ruoyao [this message]
2022-07-11 19:10   ` Adhemerval Zanella Netto
2022-07-12  0:48     ` Xi Ruoyao
2022-07-12  1:02       ` Xi Ruoyao
2022-07-12  2:32       ` Alan Modra
2022-07-12  4:24         ` Fangrui Song
2022-07-12  6:19           ` Jan Beulich
2022-07-12  6:42           ` WANG Xuerui
2022-07-12  7:33             ` Florian Weimer
2022-07-12  8:49               ` Xi Ruoyao
2022-07-12  8:58                 ` Florian Weimer
2022-07-12  9:24             ` Xi Ruoyao
2022-07-12 10:21               ` Adhemerval Zanella Netto
2022-07-12 11:01                 ` Adhemerval Zanella Netto
2022-07-12 12:15                   ` Michael Matz
2022-07-12 13:17                     ` Florian Weimer
2022-07-12 13:28                       ` Michael Matz
2022-07-15 22:34                         ` Hans-Peter Nilsson
2022-07-12 12:48                   ` caiyinyu
2022-07-12 13:00                     ` Xi Ruoyao
2022-07-12  7:33           ` Andrew Waterman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f4c1aee6565263608fd758d3a657b649525eea91.camel@xry111.site \
    --to=xry111@xry111.site \
    --cc=binutils@sourceware.org \
    --cc=caiyinyu@loongson.cn \
    --cc=carlos@redhat.com \
    --cc=i@xen0n.name \
    --cc=libc-alpha@sourceware.org \
    --cc=liuzhensong@loongson.cn \
    --cc=maskray@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).