devel

devel@uclibc-ng.org

1160 discussions

[PATCH 1/1] arc: fix signal handler restorer
by Sergey Matyukevich 10 Jan '22

10 Jan '22

From: Sergey Matyukevich <sergey.matyukevich(a)synopsys.com> When MD_FALLBACK_FRAME_STATE_FOR handler in libgcc unwind code checks the second instruction opcode in __default_rt_sa_restorer function, it expects to see the following values for ARC cores: - 0x7ee0781e for ARCv2 LE - 0x003f226f for ARC700 LE ARC700 value correspond to trap0 instruction. ARCv2 value corresponds to the following code: traps_0 j_s [blink] However, unlike glibc, uClibc implementation of __default_rt_sa_restorer for ARC does not have that jump. Hence libgcc unwind code is not able to recognize signal frame correctly on ARCv2 and completes too early. This change fixes libgcc unwinding over signal frame on ARCv2 adding missing jump to __default_rt_sa_restorer. Signed-off-by: Sergey Matyukevich <sergey.matyukevich(a)synopsys.com> --- libc/sysdeps/linux/arc/sigrestorer.S | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libc/sysdeps/linux/arc/sigrestorer.S b/libc/sysdeps/linux/arc/sigrestorer.S index e4deb6bd4..60d35ab3e 100644 --- a/libc/sysdeps/linux/arc/sigrestorer.S +++ b/libc/sysdeps/linux/arc/sigrestorer.S @@ -1,5 +1,5 @@ /* - * Copyright (C) 2017 Synopsys, Inc. (www.synopsys.com) + * Copyright (C) 2017-2022 Synopsys, Inc. (www.synopsys.com) * * Licensed under the LGPL v2.1 or later, see the file COPYING.LIB in this tarball. */ @@ -22,4 +22,5 @@ __default_rt_sa_restorer: mov r8, __NR_rt_sigreturn ARC_TRAP_INSN + j_s [blink] -- 2.34.1

2 1

Re: test, please ignore
by Waldemar Brodkorb 23 Dec '21

23 Dec '21

Hi Yann, the archiving of emails was broken since I setup mailman3 :( I hopefully fixed this now. best regards Waldemar Yann Sionneau wrote, > ack! Received :) > > Le 23/12/2021 à 22:43, Waldemar Brodkorb a écrit : > > _______________________________________________ > > devel mailing list -- devel(a)uclibc-ng.org > > To unsubscribe send an email to devel-leave(a)uclibc-ng.org >

2 1

Re: [PATCH 1/3] kvx: fix warning about __BITS_PER_LONG not being defined
by Waldemar Brodkorb 23 Dec '21

23 Dec '21

Hi Yann, finally pushed all three patches, seems I missed them. best regards Waldemar Yann Sionneau wrote, > Signed-off-by: Yann Sionneau <ysionneau(a)kalray.eu> > --- > libc/sysdeps/linux/kvx/bits/kernel_types.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libc/sysdeps/linux/kvx/bits/kernel_types.h b/libc/sysdeps/linux/kvx/bits/kernel_types.h > index 832b17674..8f6bda8e9 100644 > --- a/libc/sysdeps/linux/kvx/bits/kernel_types.h > +++ b/libc/sysdeps/linux/kvx/bits/kernel_types.h > @@ -7,6 +7,8 @@ > #ifndef __ASM_GENERIC_POSIX_TYPES_H > #define __ASM_GENERIC_POSIX_TYPES_H > > +#include <asm/bitsperlong.h> > + > typedef unsigned long __kernel_dev_t; > typedef unsigned long __kernel_ino_t; > typedef unsigned int __kernel_mode_t; > -- > 2.17.1 > > _______________________________________________ > devel mailing list -- devel(a)uclibc-ng.org > To unsubscribe send an email to devel-leave(a)uclibc-ng.org >

1 0

[PATCH] Fix warnings due to missing attributes for __EI_ prefixed symbols
by Yann Sionneau 25 Aug '21

25 Aug '21

With new compiler (gcc >= 9 ?) building uClibc-ng now gives this sort of warnings: ./include/libc-symbols.h:426:25: warning: '__EI_localeconv' specifies less restrictive attribute than its target 'localeconv': 'nothrow' [-Wmissing-attributes] 426 | extern __typeof (name) __EI_##name __attribute__((alias (__hidden_asmname1 (,#local)))) | ^~~~~ ./include/libc-symbols.h:429:29: note: in expansion of macro '__hidden_ver1' 429 | # define hidden_def(name) __hidden_ver1(__GI_##name, name, name); | ^~~~~~~~~~~~~ ./include/libc-symbols.h:497:32: note: in expansion of macro 'hidden_def' 497 | # define libc_hidden_def(name) hidden_def (name) | ^~~~~~~~~~ libc/misc/locale/locale.c:306:1: note: in expansion of macro 'libc_hidden_def' 306 | libc_hidden_def(localeconv) | ^~~~~~~~~~~~~~~ In file included from libc/misc/locale/localeconv.c:8: libc/misc/locale/locale.c:261:15: note: '__EI_localeconv' target declared here 261 | struct lconv *localeconv(void) | ^~~~~~~~~~ The fix is mostly being backported/adapted from glibc. --- include/libc-symbols.h | 14 ++++++++++---- include/sys/cdefs.h | 11 +++++++++++ 2 files changed, 21 insertions(+), 4 deletions(-) diff --git a/include/libc-symbols.h b/include/libc-symbols.h index 4dc05c76e..2254375ad 100644 --- a/include/libc-symbols.h +++ b/include/libc-symbols.h @@ -156,6 +156,11 @@ # define ASM_LINE_SEP ; #endif +#ifndef __attribute_copy__ +/* Provide an empty definition when cdefs.h is not included. */ +# define __attribute_copy__(arg) +#endif + #ifndef __ASSEMBLER__ /* GCC understands weak symbols and aliases; use its interface where possible, instead of embedded assembly language. */ @@ -163,13 +168,13 @@ /* Define ALIASNAME as a strong alias for NAME. */ # define strong_alias(name, aliasname) _strong_alias(name, aliasname) # define _strong_alias(name, aliasname) \ - extern __typeof (name) aliasname __attribute__ ((alias (#name))); + extern __typeof (name) aliasname __attribute__ ((alias (#name))) __attribute_copy__ (name); /* Same, but does not check for type match. Use sparingly. Example: strong_alias(stat,stat64) may fail, this one works: */ # define strong_alias_untyped(name, aliasname) \ _strong_alias_untyped(name, aliasname) # define _strong_alias_untyped(name, aliasname) \ - extern __typeof (aliasname) aliasname __attribute__ ((alias (#name))); + extern __typeof (aliasname) aliasname __attribute__ ((alias (#name))) __attribute_copy__ (name); # ifdef HAVE_WEAK_SYMBOLS @@ -182,7 +187,7 @@ If weak aliases are not available, this defines a strong alias. */ # define weak_alias(name, aliasname) _weak_alias (name, aliasname) # define _weak_alias(name, aliasname) \ - extern __typeof (name) aliasname __attribute__ ((weak, alias (#name))); + extern __typeof (name) aliasname __attribute__ ((weak, alias (#name))) __attribute_copy__ (name); /* Declare SYMBOL as weak undefined symbol (resolved to 0 if not defined). */ # define weak_extern(symbol) _weak_extern (weak symbol) @@ -423,7 +428,8 @@ FIXME! - ? # define __hidden_asmname2(prefix, name) #prefix name # define __hidden_ver1(local, internal, name) \ extern __typeof (name) __EI_##name __asm__(__hidden_asmname (#internal)); \ - extern __typeof (name) __EI_##name __attribute__((alias (__hidden_asmname1 (,#local)))) + extern __typeof (name) __EI_##name __attribute__((alias (__hidden_asmname1 (,#local)))) \ + __attribute_copy__ (name) # define hidden_ver(local, name) __hidden_ver1(local, __GI_##name, name); # define hidden_data_ver(local, name) hidden_ver(local, name) # define hidden_def(name) __hidden_ver1(__GI_##name, name, name); diff --git a/include/sys/cdefs.h b/include/sys/cdefs.h index a4359a05c..5c4daebcd 100644 --- a/include/sys/cdefs.h +++ b/include/sys/cdefs.h @@ -330,6 +330,17 @@ # endif #endif +/* Undefine (also defined in libc-symbols.h). */ +#undef __attribute_copy__ +#if __GNUC_PREREQ (9, 0) +/* Copies attributes from the declaration or type referenced by + the argument. */ +# define __attribute_copy__(arg) __attribute__ ((__copy__ (arg))) +#else +# define __attribute_copy__(arg) +#endif + + /* GCC 4.3 and above allow passing all anonymous arguments of an __extern_always_inline function to some other vararg function. */ #if __GNUC_PREREQ (4,3) -- 2.17.1

2 1

[sparc] revert of gcc patch for sparc for build with fPIE
by Romain Naour 06 Jun '21

06 Jun '21

Hello, Since Buildroot 2021.02 [1] an upstream gcc commit is reverted to allow booting the qemu_sparc_ss10_defconfig. The revert must be applied for gcc 8.4, 9.3 and 10.2 and any newer version. upstream gcc commit: https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=0a83f1a441d7aaadecb368c237b6e… The commit has been introduced to fix the following bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92095 The regression has been reported to gcc [2] but the gcc change may affect the ldso code or how the GOT register is handled with fPIC in uClibc-ng. This part of the code seems fPIC friendly, it can be worse with the new default BR2_PIC_PIE option. So I would drop the sparc support while adding gcc 11 to avoid carrying non upstream patch forever in gcc package. There is no problem for the sparc64 with a glibc toolchain (sparc64 for uClibc-ng is not currently available in Buildroot) Help welcome. Best regards, Romain [1] https://git.buildroot.net/buildroot/commit/?id=4d16e6f5324f0285f51bfbb5a350… [2] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98784

1 0

[PATCH 0/1] V4 fix powerpc PIE/PIC + secure-plt builds
by Yann Sionneau 04 Jun '21

04 Jun '21

This is the 4th iteration of the patch. Changes wrt v3: * add #ifdef __PIC__ test in ldso startup code, to be sure, even though I'm pretty sure this piece of code is always built as PIC. Changes wrt v2: * more precise description of the problem. * only set r30 in PIC+secureplt case as it seems indeed to be the only case which causes problem. Secureplt without PIC does not need r30. * don't rely on uclibc being already built: now we check for secure-plt by doing $(CC) --verbose and searching for the ./configure parameters of GCC for "--enable-secureplt". Yann Sionneau (1): powerpc: fix PIE/PIC builds with newer gcc/binutils which use secureplt by default Rules.mak | 3 ++- ldso/ldso/powerpc/dl-startup.h | 3 +++ libc/sysdeps/linux/powerpc/crt1.S | 4 ++++ 3 files changed, 9 insertions(+), 1 deletion(-) -- 2.25.1

2 2

[PATCH 0/1] V3 fix powerpc PIE/PIC + secure-plt builds
by Yann Sionneau 31 May '21

31 May '21

This is the 3rd iteration of the patch, here are the changes with regard to previous one: * more precise description of the problem. * only set r30 in PIC+secureplt case as it seems indeed to be the only case which causes problem. Secureplt without PIC does not need r30. * don't rely on uclibc being already built: now we check for secure-plt by doing $(CC) --verbose and searching for the ./configure parameters of GCC for "--enable-secureplt". Yann Sionneau (1): powerpc: fix PIE/PIC builds with newer gcc/binutils which use secureplt by default Rules.mak | 3 ++- ldso/ldso/powerpc/dl-startup.h | 3 +++ libc/sysdeps/linux/powerpc/crt1.S | 4 ++++ 3 files changed, 9 insertions(+), 1 deletion(-) -- 2.25.1

2 2

Race condition in LinuxThread's sigaction wrapper
by Vladimir Murzin 25 May '21

25 May '21

Hi I got report about race condition in LinuxThread's sigaction wrapper located in libpthread/linuxthreads/signals.c In sigaction wrapper A.1) override handler for new action to point at pthread_sighandler A.2) __libc_sigaction A.3) update global array (sighandler) used by pthread_sighandler with real action (from new) pthread_sighandler B.1) find real action (indexed by signal number) B.2) unconditionally execute action IIUC, if we receive signal in a window between A.2 and A.3 we are missing executing real action (likely we try to execute NULL). Does that make sense? Cheers Vladimir

2 3

[PATCH] powerpc: fix builds with newer gcc/binutils which use secureplt by default
by yann＠sionneau.net 25 May '21

25 May '21

From: Yann Sionneau <yann(a)sionneau.net> This patch fixes segfault of all user space processes (including init, which caused a panic) on recent buildroot powerpc32 builds. Recent buildroot toolchain enables secure PLT in powerpc gcc. The latter will then supply -msecure-plt to gas invocations by default. For the secure PLT to work, the r30 register needs to point to the GOT. Old "bss plt" was just a one-instruction-wide PLT slot, pointed-to by a R_PPC_JMP_SLOT relocation, which was written on-the-fly to contain a branch instruction to the correct address. It therefore had to stay writable. New secure PLT only contains read-only code which loads the branch address from the writable GOT. Signed-off-by: Yann Sionneau <yann(a)sionneau.net> --- Rules.mak | 4 +++- ldso/ldso/powerpc/dl-startup.h | 3 +++ libc/sysdeps/linux/powerpc/crt1.S | 5 ++++- 3 files changed, 10 insertions(+), 2 deletions(-) diff --git a/Rules.mak b/Rules.mak index 1fa09be23..0ab41e800 100644 --- a/Rules.mak +++ b/Rules.mak @@ -58,6 +58,7 @@ LD = $(CROSS_COMPILE)ld NM = $(CROSS_COMPILE)nm OBJDUMP = $(CROSS_COMPILE)objdump STRIPTOOL = $(CROSS_COMPILE)strip +READELF = $(CROSS_COMPILE)readelf INSTALL = install LN = ln @@ -483,9 +484,10 @@ ifeq ($(TARGET_ARCH),powerpc) PICFLAG:=-fpic PIEFLAG_NAME:=-fpie PPC_HAS_REL16:=$(shell printf "\t.text\n\taddis 11,30,_GLOBAL_OFFSET_TABLE_-.@ha\n" | $(CC) -c -x assembler -o /dev/null - 2> /dev/null && echo -n y || echo -n n) + PPC_HAS_SECUREPLT:=$(shell tmpfile=$$(mktemp); printf '\#include <stdio.h>\nint main(void) { puts("hello"); return 0;}' > $${tmpfile}.c; $(CC) -x c $${tmpfile}.c -o $${tmpfile}; $(READELF) -d $${tmpfile} | grep PPC_GOT > /dev/null && echo -n y || echo -n n; rm $${tmpfile}.c $${tmpfile}) + CPU_CFLAGS-$(PPC_HAS_SECUREPLT) += -DPPC_HAS_SECUREPLT CPU_CFLAGS-$(PPC_HAS_REL16)+= -DHAVE_ASM_PPC_REL16 CPU_CFLAGS-$(CONFIG_E500) += "-D__NO_MATH_INLINES" - endif ifeq ($(TARGET_ARCH),bfin) diff --git a/ldso/ldso/powerpc/dl-startup.h b/ldso/ldso/powerpc/dl-startup.h index 8b2a517e2..7749395eb 100644 --- a/ldso/ldso/powerpc/dl-startup.h +++ b/ldso/ldso/powerpc/dl-startup.h @@ -25,6 +25,9 @@ __asm__( #else " bl _GLOBAL_OFFSET_TABLE_-4@local\n" /* Put our GOT pointer in r31, */ " mflr 31\n" +#endif +#ifdef PPC_HAS_SECUREPLT + " mr 30,31\n" #endif " addi 1,1,16\n" /* Restore SP */ " lwz 7,_dl_skip_args@got(31)\n" /* load EA of _dl_skip_args */ diff --git a/libc/sysdeps/linux/powerpc/crt1.S b/libc/sysdeps/linux/powerpc/crt1.S index 27bfc5a5a..78b946ad6 100644 --- a/libc/sysdeps/linux/powerpc/crt1.S +++ b/libc/sysdeps/linux/powerpc/crt1.S @@ -47,7 +47,7 @@ _start: mr r9,r1 /* Save the stack pointer and pass it to __uClibc_main */ clrrwi r1,r1,4 /* Align stack ptr to 16 bytes */ -#ifdef __PIC__ +#if defined(__PIC__) || defined(PPC_HAS_SECUREPLT) # ifdef HAVE_ASM_PPC_REL16 bcl 20,31,1f 1: mflr r31 @@ -57,6 +57,9 @@ _start: bl _GLOBAL_OFFSET_TABLE_-4@local mflr r31 # endif +#endif +#ifdef PPC_HAS_SECUREPLT + mr 30,31 #endif /* Set up the small data pointer in r13. */ #ifdef __PIC__ -- 2.25.1

3 2

[Powerpc] HAVE_ASM_PPC_REL16 and BR2_PIC_PIE
by Romain Naour 22 May '21

22 May '21

Hello, Recently in Buildroot the option BR2_PIC_PIE has been enabled by default along with other hardening features [1]. Since then some ppc defconfig such qemu_ppc_e500mc_defconfig are failing to boot due to a segfault in init program. The segfault appear very early in __uClibc_main while starting any binaries, an issue located in crt1.S (powerpc)[2]. After some trial and error, removing HAVE_ASM_PPC_REL16 from CFLAGS [3] allow to generate a working system again. But this is actually wrong since instead we should consider HAVE_ASM_PPC_REL16 always true nowadays. Indeed, Glibc removed HAVE_ASM_PPC_REL16 since version 2.22 [4] since "the minimum binutils supports rel16 relocs". Binutils 2.22 supports R_PPC_REL16 as default. uClibc-ng should remove HAVE_ASM_PPC_REL16 but keep the code as it was defined. But this doesn't fix the initial issue. Any idea ? [1] https://git.buildroot.net/buildroot/commit/?id=810ba387bec3c5b6904e8893fb4c… [2] https://cgit.uclibc-ng.org/cgi/cgit/uclibc-ng.git/tree/libc/sysdeps/linux/p… [3] https://cgit.uclibc-ng.org/cgi/cgit/uclibc-ng.git/tree/Rules.mak?id=2bf4991… [4] https://sourceware.org/git/?p=glibc.git;a=commit;h=59261ad3eb345e0d7b9f5c73… Best regards, Romain

6 14

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

devel