* [PATCH 0/2] Fix incorrect stub warnings after generic refactor
@ 2022-12-19 12:03 Florian Weimer
2022-12-19 12:03 ` [PATCH 1/2] Linux: Reflow and sort some Makefile variables Florian Weimer
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Florian Weimer @ 2022-12-19 12:03 UTC (permalink / raw)
To: libc-alpha; +Cc: Adhemerval Zanella, Carlos O'Donell
They cause link failures with -Wl,--fatal-warnings on aarch64.
Tested on aarch64-linux-gnu, i686-linux-gnu, x86_64-linux-gnu. Manually
verified that the warnings are gone on aarch64-linux-gnu.
Florian Weimer (2):
Linux: Reflow and sort some Makefile variables
Linux: Remove epoll_create, inotify_init from syscalls.list
sysdeps/unix/sysv/linux/Makefile | 220 ++++++++++++++++++--------
sysdeps/unix/sysv/linux/syscalls.list | 2 -
2 files changed, 157 insertions(+), 65 deletions(-)
base-commit: 0fb10e49ad169374650534509767ec1d4232e230
--
2.38.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] Linux: Reflow and sort some Makefile variables
2022-12-19 12:03 [PATCH 0/2] Fix incorrect stub warnings after generic refactor Florian Weimer
@ 2022-12-19 12:03 ` Florian Weimer
2022-12-19 12:15 ` Adhemerval Zanella Netto
2022-12-19 12:03 ` [PATCH 2/2] Linux: Remove epoll_create, inotify_init from syscalls.list Florian Weimer
2022-12-19 12:16 ` [PATCH 0/2] Fix incorrect stub warnings after generic refactor Adhemerval Zanella Netto
2 siblings, 1 reply; 7+ messages in thread
From: Florian Weimer @ 2022-12-19 12:03 UTC (permalink / raw)
To: libc-alpha; +Cc: Adhemerval Zanella, Carlos O'Donell
---
sysdeps/unix/sysv/linux/Makefile | 218 ++++++++++++++++++++++---------
1 file changed, 155 insertions(+), 63 deletions(-)
diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
index 60e4cb1521..42e51625ab 100644
--- a/sysdeps/unix/sysv/linux/Makefile
+++ b/sysdeps/unix/sysv/linux/Makefile
@@ -54,21 +54,57 @@ CFLAGS-malloc.c += -DMORECORE_CLEARS=2
endif
ifeq ($(subdir),misc)
-sysdep_routines += adjtimex clone umount umount2 readahead sysctl \
- setfsuid setfsgid epoll_pwait epoll_pwait2 signalfd \
- eventfd eventfd_read eventfd_write prlimit prlimit64 \
- personality epoll_wait tee vmsplice splice \
- open_by_handle_at mlock2 pkey_mprotect pkey_set pkey_get \
- timerfd_gettime timerfd_settime prctl \
- process_vm_readv process_vm_writev clock_adjtime \
- pselect32 \
- xstat fxstat lxstat xstat64 fxstat64 lxstat64 \
- fxstatat fxstatat64 \
- xmknod xmknodat convert_scm_timestamps \
- closefrom_fallback \
- clone3 clone-internal \
- fanotify_mark \
- mremap \
+sysdep_routines += \
+ adjtimex \
+ clock_adjtime \
+ clone \
+ clone-internal \
+ clone3 \
+ closefrom_fallback \
+ convert_scm_timestamps \
+ epoll_pwait \
+ epoll_pwait2 \
+ epoll_wait \
+ eventfd \
+ eventfd_read \
+ eventfd_write \
+ fanotify_mark \
+ fxstat \
+ fxstat64 \
+ fxstatat \
+ fxstatat64 \
+ lxstat \
+ lxstat64 \
+ mlock2 \
+ mremap \
+ open_by_handle_at \
+ personality \
+ pkey_get \
+ pkey_mprotect \
+ pkey_set \
+ prctl \
+ prlimit \
+ prlimit64 \
+ process_vm_readv \
+ process_vm_writev \
+ pselect32 \
+ readahead \
+ setfsgid \
+ setfsuid \
+ signalfd \
+ splice \
+ sysctl \
+ tee \
+ timerfd_gettime \
+ timerfd_settime \
+ umount \
+ umount2 \
+ vmsplice \
+ xmknod \
+ xmknodat \
+ xstat \
+ xstat64 \
+ # sysdeps-routines
CFLAGS-gethostid.c = -fexceptions
CFLAGS-tee.c = -fexceptions -fasynchronous-unwind-tables
@@ -79,58 +115,114 @@ CFLAGS-sync_file_range.c = -fexceptions -fasynchronous-unwind-tables
CFLAGS-pselect32.c = -fexceptions -fasynchronous-unwind-tables
CFLAGS-tst-writev.c += "-DARTIFICIAL_LIMIT=(0x80000000-sysconf(_SC_PAGESIZE))"
-sysdep_headers += sys/mount.h sys/acct.h \
- sys/klog.h \
- sys/user.h sys/prctl.h \
- sys/kd.h sys/soundcard.h sys/vt.h \
- sys/quota.h sys/fsuid.h \
- scsi/sg.h scsi/scsi.h scsi/scsi_ioctl.h sys/pci.h \
- sys/raw.h sys/personality.h sys/epoll.h \
- bits/a.out.h sys/inotify.h sys/signalfd.h sys/eventfd.h \
- sys/timerfd.h sys/fanotify.h bits/eventfd.h bits/inotify.h \
- bits/signalfd.h bits/timerfd.h bits/epoll.h \
- bits/socket_type.h bits/syscall.h \
- bits/mman-linux.h bits/mman-shared.h bits/ptrace-shared.h \
- bits/siginfo-arch.h bits/siginfo-consts-arch.h \
- bits/procfs.h bits/procfs-id.h bits/procfs-extra.h \
- bits/procfs-prregset.h bits/mman-map-flags-generic.h \
- bits/shmlba.h \
- bits/termios-struct.h bits/termios-c_cc.h \
- bits/termios-c_iflag.h bits/termios-c_oflag.h \
- bits/termios-baud.h bits/termios-c_cflag.h \
- bits/termios-c_lflag.h bits/termios-tcflow.h \
- bits/termios-misc.h \
- bits/types/struct_semid_ds.h \
- bits/types/struct_msqid_ds.h \
- bits/types/struct_shmid_ds.h \
- bits/ipc-perm.h \
- bits/struct_stat.h \
- bits/struct_stat_time64_helper.h \
- bits/types/struct_msqid64_ds.h \
- bits/types/struct_msqid64_ds_helper.h \
- bits/types/struct_semid64_ds.h \
- bits/types/struct_semid64_ds_helper.h \
- bits/types/struct_shmid64_ds.h \
- bits/types/struct_shmid64_ds_helper.h \
- bits/pthread_stack_min.h bits/pthread_stack_min-dynamic.h \
- sys/rseq.h bits/rseq.h \
- sys/pidfd.h
-
-tests += tst-clone tst-clone2 tst-clone3 tst-fanotify tst-personality \
- tst-quota tst-sync_file_range tst-sysconf-iov_max tst-ttyname \
- test-errno-linux tst-memfd_create tst-mlock2 tst-pkey \
- tst-rlimit-infinity tst-ofdlocks tst-gettid tst-gettid-kill \
- tst-tgkill tst-sysvsem-linux tst-sysvmsg-linux tst-sysvshm-linux \
- tst-timerfd tst-ppoll \
- tst-clock_adjtime tst-adjtimex tst-ntp_adjtime tst-ntp_gettime \
- tst-ntp_gettimex tst-sigtimedwait tst-misalign-clone \
- tst-prctl \
- tst-scm_rights \
+sysdep_headers += \
+ bits/a.out.h \
+ bits/epoll.h \
+ bits/eventfd.h \
+ bits/inotify.h \
+ bits/ipc-perm.h \
+ bits/mman-linux.h \
+ bits/mman-map-flags-generic.h \
+ bits/mman-shared.h \
+ bits/procfs-extra.h \
+ bits/procfs-id.h \
+ bits/procfs-prregset.h \
+ bits/procfs.h \
+ bits/pthread_stack_min-dynamic.h \
+ bits/pthread_stack_min.h \
+ bits/ptrace-shared.h \
+ bits/rseq.h \
+ bits/shmlba.h \
+ bits/siginfo-arch.h \
+ bits/siginfo-consts-arch.h \
+ bits/signalfd.h \
+ bits/socket_type.h \
+ bits/struct_stat.h \
+ bits/struct_stat_time64_helper.h \
+ bits/syscall.h \
+ bits/termios-baud.h \
+ bits/termios-c_cc.h \
+ bits/termios-c_cflag.h \
+ bits/termios-c_iflag.h \
+ bits/termios-c_lflag.h \
+ bits/termios-c_oflag.h \
+ bits/termios-misc.h \
+ bits/termios-struct.h \
+ bits/termios-tcflow.h \
+ bits/timerfd.h \
+ bits/types/struct_msqid64_ds.h \
+ bits/types/struct_msqid64_ds_helper.h \
+ bits/types/struct_msqid_ds.h \
+ bits/types/struct_semid64_ds.h \
+ bits/types/struct_semid64_ds_helper.h \
+ bits/types/struct_semid_ds.h \
+ bits/types/struct_shmid64_ds.h \
+ bits/types/struct_shmid64_ds_helper.h \
+ bits/types/struct_shmid_ds.h \
+ scsi/scsi.h \
+ scsi/scsi_ioctl.h \
+ scsi/sg.h \
+ sys/acct.h \
+ sys/epoll.h \
+ sys/eventfd.h \
+ sys/fanotify.h \
+ sys/fsuid.h \
+ sys/inotify.h \
+ sys/kd.h \
+ sys/klog.h \
+ sys/mount.h \
+ sys/pci.h \
+ sys/personality.h \
+ sys/pidfd.h \
+ sys/prctl.h \
+ sys/quota.h \
+ sys/raw.h \
+ sys/rseq.h \
+ sys/signalfd.h \
+ sys/soundcard.h \
+ sys/timerfd.h \
+ sys/user.h \
+ sys/vt.h \
+ # sysdep-headers
+
+tests += \
+ test-errno-linux \
+ tst-adjtimex \
+ tst-clock_adjtime \
+ tst-clone \
+ tst-clone2 \
+ tst-clone3 \
tst-epoll \
+ tst-fanotify \
tst-getauxval \
+ tst-gettid \
+ tst-gettid-kill \
+ tst-memfd_create \
+ tst-misalign-clone \
+ tst-mlock2 \
+ tst-mount \
+ tst-ntp_adjtime \
+ tst-ntp_gettime \
+ tst-ntp_gettimex \
+ tst-ofdlocks \
+ tst-personality \
tst-pidfd \
+ tst-pkey \
+ tst-ppoll \
+ tst-prctl \
tst-process_mrelease \
- tst-mount \
+ tst-quota \
+ tst-rlimit-infinity \
+ tst-scm_rights \
+ tst-sigtimedwait \
+ tst-sync_file_range \
+ tst-sysconf-iov_max \
+ tst-sysvmsg-linux \
+ tst-sysvsem-linux \
+ tst-sysvshm-linux \
+ tst-tgkill \
+ tst-timerfd \
+ tst-ttyname \
# tests
# process_madvise requires CAP_SYS_ADMIN.
--
2.38.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] Linux: Reflow and sort some Makefile variables
2022-12-19 12:03 ` [PATCH 1/2] Linux: Reflow and sort some Makefile variables Florian Weimer
@ 2022-12-19 12:15 ` Adhemerval Zanella Netto
0 siblings, 0 replies; 7+ messages in thread
From: Adhemerval Zanella Netto @ 2022-12-19 12:15 UTC (permalink / raw)
To: Florian Weimer, libc-alpha; +Cc: Carlos O'Donell
LGTM, thanks.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
On 19/12/22 09:03, Florian Weimer wrote:
> ---
> sysdeps/unix/sysv/linux/Makefile | 218 ++++++++++++++++++++++---------
> 1 file changed, 155 insertions(+), 63 deletions(-)
>
> diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
> index 60e4cb1521..42e51625ab 100644
> --- a/sysdeps/unix/sysv/linux/Makefile
> +++ b/sysdeps/unix/sysv/linux/Makefile
> @@ -54,21 +54,57 @@ CFLAGS-malloc.c += -DMORECORE_CLEARS=2
> endif
>
> ifeq ($(subdir),misc)
> -sysdep_routines += adjtimex clone umount umount2 readahead sysctl \
> - setfsuid setfsgid epoll_pwait epoll_pwait2 signalfd \
> - eventfd eventfd_read eventfd_write prlimit prlimit64 \
> - personality epoll_wait tee vmsplice splice \
> - open_by_handle_at mlock2 pkey_mprotect pkey_set pkey_get \
> - timerfd_gettime timerfd_settime prctl \
> - process_vm_readv process_vm_writev clock_adjtime \
> - pselect32 \
> - xstat fxstat lxstat xstat64 fxstat64 lxstat64 \
> - fxstatat fxstatat64 \
> - xmknod xmknodat convert_scm_timestamps \
> - closefrom_fallback \
> - clone3 clone-internal \
> - fanotify_mark \
> - mremap \
> +sysdep_routines += \
> + adjtimex \
> + clock_adjtime \
> + clone \
> + clone-internal \
> + clone3 \
> + closefrom_fallback \
> + convert_scm_timestamps \
> + epoll_pwait \
> + epoll_pwait2 \
> + epoll_wait \
> + eventfd \
> + eventfd_read \
> + eventfd_write \
> + fanotify_mark \
> + fxstat \
> + fxstat64 \
> + fxstatat \
> + fxstatat64 \
> + lxstat \
> + lxstat64 \
> + mlock2 \
> + mremap \
> + open_by_handle_at \
> + personality \
> + pkey_get \
> + pkey_mprotect \
> + pkey_set \
> + prctl \
> + prlimit \
> + prlimit64 \
> + process_vm_readv \
> + process_vm_writev \
> + pselect32 \
> + readahead \
> + setfsgid \
> + setfsuid \
> + signalfd \
> + splice \
> + sysctl \
> + tee \
> + timerfd_gettime \
> + timerfd_settime \
> + umount \
> + umount2 \
> + vmsplice \
> + xmknod \
> + xmknodat \
> + xstat \
> + xstat64 \
> + # sysdeps-routines
>
> CFLAGS-gethostid.c = -fexceptions
> CFLAGS-tee.c = -fexceptions -fasynchronous-unwind-tables
> @@ -79,58 +115,114 @@ CFLAGS-sync_file_range.c = -fexceptions -fasynchronous-unwind-tables
> CFLAGS-pselect32.c = -fexceptions -fasynchronous-unwind-tables
> CFLAGS-tst-writev.c += "-DARTIFICIAL_LIMIT=(0x80000000-sysconf(_SC_PAGESIZE))"
>
> -sysdep_headers += sys/mount.h sys/acct.h \
> - sys/klog.h \
> - sys/user.h sys/prctl.h \
> - sys/kd.h sys/soundcard.h sys/vt.h \
> - sys/quota.h sys/fsuid.h \
> - scsi/sg.h scsi/scsi.h scsi/scsi_ioctl.h sys/pci.h \
> - sys/raw.h sys/personality.h sys/epoll.h \
> - bits/a.out.h sys/inotify.h sys/signalfd.h sys/eventfd.h \
> - sys/timerfd.h sys/fanotify.h bits/eventfd.h bits/inotify.h \
> - bits/signalfd.h bits/timerfd.h bits/epoll.h \
> - bits/socket_type.h bits/syscall.h \
> - bits/mman-linux.h bits/mman-shared.h bits/ptrace-shared.h \
> - bits/siginfo-arch.h bits/siginfo-consts-arch.h \
> - bits/procfs.h bits/procfs-id.h bits/procfs-extra.h \
> - bits/procfs-prregset.h bits/mman-map-flags-generic.h \
> - bits/shmlba.h \
> - bits/termios-struct.h bits/termios-c_cc.h \
> - bits/termios-c_iflag.h bits/termios-c_oflag.h \
> - bits/termios-baud.h bits/termios-c_cflag.h \
> - bits/termios-c_lflag.h bits/termios-tcflow.h \
> - bits/termios-misc.h \
> - bits/types/struct_semid_ds.h \
> - bits/types/struct_msqid_ds.h \
> - bits/types/struct_shmid_ds.h \
> - bits/ipc-perm.h \
> - bits/struct_stat.h \
> - bits/struct_stat_time64_helper.h \
> - bits/types/struct_msqid64_ds.h \
> - bits/types/struct_msqid64_ds_helper.h \
> - bits/types/struct_semid64_ds.h \
> - bits/types/struct_semid64_ds_helper.h \
> - bits/types/struct_shmid64_ds.h \
> - bits/types/struct_shmid64_ds_helper.h \
> - bits/pthread_stack_min.h bits/pthread_stack_min-dynamic.h \
> - sys/rseq.h bits/rseq.h \
> - sys/pidfd.h
> -
> -tests += tst-clone tst-clone2 tst-clone3 tst-fanotify tst-personality \
> - tst-quota tst-sync_file_range tst-sysconf-iov_max tst-ttyname \
> - test-errno-linux tst-memfd_create tst-mlock2 tst-pkey \
> - tst-rlimit-infinity tst-ofdlocks tst-gettid tst-gettid-kill \
> - tst-tgkill tst-sysvsem-linux tst-sysvmsg-linux tst-sysvshm-linux \
> - tst-timerfd tst-ppoll \
> - tst-clock_adjtime tst-adjtimex tst-ntp_adjtime tst-ntp_gettime \
> - tst-ntp_gettimex tst-sigtimedwait tst-misalign-clone \
> - tst-prctl \
> - tst-scm_rights \
> +sysdep_headers += \
> + bits/a.out.h \
> + bits/epoll.h \
> + bits/eventfd.h \
> + bits/inotify.h \
> + bits/ipc-perm.h \
> + bits/mman-linux.h \
> + bits/mman-map-flags-generic.h \
> + bits/mman-shared.h \
> + bits/procfs-extra.h \
> + bits/procfs-id.h \
> + bits/procfs-prregset.h \
> + bits/procfs.h \
> + bits/pthread_stack_min-dynamic.h \
> + bits/pthread_stack_min.h \
> + bits/ptrace-shared.h \
> + bits/rseq.h \
> + bits/shmlba.h \
> + bits/siginfo-arch.h \
> + bits/siginfo-consts-arch.h \
> + bits/signalfd.h \
> + bits/socket_type.h \
> + bits/struct_stat.h \
> + bits/struct_stat_time64_helper.h \
> + bits/syscall.h \
> + bits/termios-baud.h \
> + bits/termios-c_cc.h \
> + bits/termios-c_cflag.h \
> + bits/termios-c_iflag.h \
> + bits/termios-c_lflag.h \
> + bits/termios-c_oflag.h \
> + bits/termios-misc.h \
> + bits/termios-struct.h \
> + bits/termios-tcflow.h \
> + bits/timerfd.h \
> + bits/types/struct_msqid64_ds.h \
> + bits/types/struct_msqid64_ds_helper.h \
> + bits/types/struct_msqid_ds.h \
> + bits/types/struct_semid64_ds.h \
> + bits/types/struct_semid64_ds_helper.h \
> + bits/types/struct_semid_ds.h \
> + bits/types/struct_shmid64_ds.h \
> + bits/types/struct_shmid64_ds_helper.h \
> + bits/types/struct_shmid_ds.h \
> + scsi/scsi.h \
> + scsi/scsi_ioctl.h \
> + scsi/sg.h \
> + sys/acct.h \
> + sys/epoll.h \
> + sys/eventfd.h \
> + sys/fanotify.h \
> + sys/fsuid.h \
> + sys/inotify.h \
> + sys/kd.h \
> + sys/klog.h \
> + sys/mount.h \
> + sys/pci.h \
> + sys/personality.h \
> + sys/pidfd.h \
> + sys/prctl.h \
> + sys/quota.h \
> + sys/raw.h \
> + sys/rseq.h \
> + sys/signalfd.h \
> + sys/soundcard.h \
> + sys/timerfd.h \
> + sys/user.h \
> + sys/vt.h \
> + # sysdep-headers
> +
> +tests += \
> + test-errno-linux \
> + tst-adjtimex \
> + tst-clock_adjtime \
> + tst-clone \
> + tst-clone2 \
> + tst-clone3 \
> tst-epoll \
> + tst-fanotify \
> tst-getauxval \
> + tst-gettid \
> + tst-gettid-kill \
> + tst-memfd_create \
> + tst-misalign-clone \
> + tst-mlock2 \
> + tst-mount \
> + tst-ntp_adjtime \
> + tst-ntp_gettime \
> + tst-ntp_gettimex \
> + tst-ofdlocks \
> + tst-personality \
> tst-pidfd \
> + tst-pkey \
> + tst-ppoll \
> + tst-prctl \
> tst-process_mrelease \
> - tst-mount \
> + tst-quota \
> + tst-rlimit-infinity \
> + tst-scm_rights \
> + tst-sigtimedwait \
> + tst-sync_file_range \
> + tst-sysconf-iov_max \
> + tst-sysvmsg-linux \
> + tst-sysvsem-linux \
> + tst-sysvshm-linux \
> + tst-tgkill \
> + tst-timerfd \
> + tst-ttyname \
> # tests
>
> # process_madvise requires CAP_SYS_ADMIN.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] Linux: Remove epoll_create, inotify_init from syscalls.list
2022-12-19 12:03 [PATCH 0/2] Fix incorrect stub warnings after generic refactor Florian Weimer
2022-12-19 12:03 ` [PATCH 1/2] Linux: Reflow and sort some Makefile variables Florian Weimer
@ 2022-12-19 12:03 ` Florian Weimer
2022-12-19 12:16 ` Adhemerval Zanella Netto
2022-12-19 12:16 ` [PATCH 0/2] Fix incorrect stub warnings after generic refactor Adhemerval Zanella Netto
2 siblings, 1 reply; 7+ messages in thread
From: Florian Weimer @ 2022-12-19 12:03 UTC (permalink / raw)
To: libc-alpha; +Cc: Adhemerval Zanella, Carlos O'Donell
Their presence causes stub warnings to be created on architectures
which do not implement them.
Fixes commit d1d23b134244d59c4d6ef2295 ("Lninux: consolidate
epoll_create implementation") and commit 842128f160a48e5545900ea3b
("Linux: consolidate inotify_init implementation").
---
sysdeps/unix/sysv/linux/Makefile | 2 ++
sysdeps/unix/sysv/linux/syscalls.list | 2 --
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
index 42e51625ab..f298878e8f 100644
--- a/sysdeps/unix/sysv/linux/Makefile
+++ b/sysdeps/unix/sysv/linux/Makefile
@@ -62,6 +62,7 @@ sysdep_routines += \
clone3 \
closefrom_fallback \
convert_scm_timestamps \
+ epoll_create \
epoll_pwait \
epoll_pwait2 \
epoll_wait \
@@ -73,6 +74,7 @@ sysdep_routines += \
fxstat64 \
fxstatat \
fxstatat64 \
+ inotify_init \
lxstat \
lxstat64 \
mlock2 \
diff --git a/sysdeps/unix/sysv/linux/syscalls.list b/sysdeps/unix/sysv/linux/syscalls.list
index b9f45463c5..73e941ef89 100644
--- a/sysdeps/unix/sysv/linux/syscalls.list
+++ b/sysdeps/unix/sysv/linux/syscalls.list
@@ -6,7 +6,6 @@ capget EXTRA capget i:pp capget
capset EXTRA capset i:pp capset
create_module EXTRA create_module 3 __compat_create_module create_module@GLIBC_2.0:GLIBC_2.23
delete_module EXTRA delete_module 3 delete_module
-epoll_create EXTRA epoll_create i:i epoll_create
epoll_create1 EXTRA epoll_create1 i:i epoll_create1
epoll_ctl EXTRA epoll_ctl i:iiip epoll_ctl
eventfd EXTRA eventfd2 i:ii eventfd
@@ -28,7 +27,6 @@ getresgid - getresgid i:ppp getresgid
getsid - getsid i:i getsid
init_module EXTRA init_module 5 init_module
inotify_add_watch EXTRA inotify_add_watch i:isi inotify_add_watch
-inotify_init EXTRA inotify_init i: inotify_init
inotify_init1 EXTRA inotify_init1 i:I inotify_init1
inotify_rm_watch EXTRA inotify_rm_watch i:ii inotify_rm_watch
ioperm - ioperm i:UUi ioperm
--
2.38.1
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] Linux: Remove epoll_create, inotify_init from syscalls.list
2022-12-19 12:03 ` [PATCH 2/2] Linux: Remove epoll_create, inotify_init from syscalls.list Florian Weimer
@ 2022-12-19 12:16 ` Adhemerval Zanella Netto
0 siblings, 0 replies; 7+ messages in thread
From: Adhemerval Zanella Netto @ 2022-12-19 12:16 UTC (permalink / raw)
To: Florian Weimer, libc-alpha; +Cc: Carlos O'Donell
On 19/12/22 09:03, Florian Weimer wrote:
> Their presence causes stub warnings to be created on architectures
> which do not implement them.
>
> Fixes commit d1d23b134244d59c4d6ef2295 ("Lninux: consolidate
> epoll_create implementation") and commit 842128f160a48e5545900ea3b
> ("Linux: consolidate inotify_init implementation").
LGTM, thanks.
Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
> ---
> sysdeps/unix/sysv/linux/Makefile | 2 ++
> sysdeps/unix/sysv/linux/syscalls.list | 2 --
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/sysdeps/unix/sysv/linux/Makefile b/sysdeps/unix/sysv/linux/Makefile
> index 42e51625ab..f298878e8f 100644
> --- a/sysdeps/unix/sysv/linux/Makefile
> +++ b/sysdeps/unix/sysv/linux/Makefile
> @@ -62,6 +62,7 @@ sysdep_routines += \
> clone3 \
> closefrom_fallback \
> convert_scm_timestamps \
> + epoll_create \
> epoll_pwait \
> epoll_pwait2 \
> epoll_wait \
> @@ -73,6 +74,7 @@ sysdep_routines += \
> fxstat64 \
> fxstatat \
> fxstatat64 \
> + inotify_init \
> lxstat \
> lxstat64 \
> mlock2 \
> diff --git a/sysdeps/unix/sysv/linux/syscalls.list b/sysdeps/unix/sysv/linux/syscalls.list
> index b9f45463c5..73e941ef89 100644
> --- a/sysdeps/unix/sysv/linux/syscalls.list
> +++ b/sysdeps/unix/sysv/linux/syscalls.list
> @@ -6,7 +6,6 @@ capget EXTRA capget i:pp capget
> capset EXTRA capset i:pp capset
> create_module EXTRA create_module 3 __compat_create_module create_module@GLIBC_2.0:GLIBC_2.23
> delete_module EXTRA delete_module 3 delete_module
> -epoll_create EXTRA epoll_create i:i epoll_create
> epoll_create1 EXTRA epoll_create1 i:i epoll_create1
> epoll_ctl EXTRA epoll_ctl i:iiip epoll_ctl
> eventfd EXTRA eventfd2 i:ii eventfd
> @@ -28,7 +27,6 @@ getresgid - getresgid i:ppp getresgid
> getsid - getsid i:i getsid
> init_module EXTRA init_module 5 init_module
> inotify_add_watch EXTRA inotify_add_watch i:isi inotify_add_watch
> -inotify_init EXTRA inotify_init i: inotify_init
> inotify_init1 EXTRA inotify_init1 i:I inotify_init1
> inotify_rm_watch EXTRA inotify_rm_watch i:ii inotify_rm_watch
> ioperm - ioperm i:UUi ioperm
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] Fix incorrect stub warnings after generic refactor
2022-12-19 12:03 [PATCH 0/2] Fix incorrect stub warnings after generic refactor Florian Weimer
2022-12-19 12:03 ` [PATCH 1/2] Linux: Reflow and sort some Makefile variables Florian Weimer
2022-12-19 12:03 ` [PATCH 2/2] Linux: Remove epoll_create, inotify_init from syscalls.list Florian Weimer
@ 2022-12-19 12:16 ` Adhemerval Zanella Netto
2022-12-19 12:27 ` Florian Weimer
2 siblings, 1 reply; 7+ messages in thread
From: Adhemerval Zanella Netto @ 2022-12-19 12:16 UTC (permalink / raw)
To: Florian Weimer, libc-alpha; +Cc: Carlos O'Donell
On 19/12/22 09:03, Florian Weimer wrote:
> They cause link failures with -Wl,--fatal-warnings on aarch64.
>
Would be possible to use this option as default to catch such errors?
> Tested on aarch64-linux-gnu, i686-linux-gnu, x86_64-linux-gnu. Manually
> verified that the warnings are gone on aarch64-linux-gnu.
>
> Florian Weimer (2):
> Linux: Reflow and sort some Makefile variables
> Linux: Remove epoll_create, inotify_init from syscalls.list
>
> sysdeps/unix/sysv/linux/Makefile | 220 ++++++++++++++++++--------
> sysdeps/unix/sysv/linux/syscalls.list | 2 -
> 2 files changed, 157 insertions(+), 65 deletions(-)
>
>
> base-commit: 0fb10e49ad169374650534509767ec1d4232e230
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 0/2] Fix incorrect stub warnings after generic refactor
2022-12-19 12:16 ` [PATCH 0/2] Fix incorrect stub warnings after generic refactor Adhemerval Zanella Netto
@ 2022-12-19 12:27 ` Florian Weimer
0 siblings, 0 replies; 7+ messages in thread
From: Florian Weimer @ 2022-12-19 12:27 UTC (permalink / raw)
To: Adhemerval Zanella Netto; +Cc: libc-alpha, Carlos O'Donell
* Adhemerval Zanella Netto:
> On 19/12/22 09:03, Florian Weimer wrote:
>> They cause link failures with -Wl,--fatal-warnings on aarch64.
>>
>
> Would be possible to use this option as default to catch such errors?
We'd have to make it conditional because tests for mktemp and the like
do result in warnings. We also do not have an inotify test, but in this
particular case, maybe the nscd build would have caught it.
Thanks,
Florian
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-12-19 12:27 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-12-19 12:03 [PATCH 0/2] Fix incorrect stub warnings after generic refactor Florian Weimer
2022-12-19 12:03 ` [PATCH 1/2] Linux: Reflow and sort some Makefile variables Florian Weimer
2022-12-19 12:15 ` Adhemerval Zanella Netto
2022-12-19 12:03 ` [PATCH 2/2] Linux: Remove epoll_create, inotify_init from syscalls.list Florian Weimer
2022-12-19 12:16 ` Adhemerval Zanella Netto
2022-12-19 12:16 ` [PATCH 0/2] Fix incorrect stub warnings after generic refactor Adhemerval Zanella Netto
2022-12-19 12:27 ` Florian Weimer
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).