unofficial mirror of libc-alpha@sourceware.org
 help / color / mirror / Atom feed
From: Carlos O'Donell via Libc-alpha <libc-alpha@sourceware.org>
To: Adhemerval Zanella <adhemerval.zanella@linaro.org>,
	libc-alpha@sourceware.org
Subject: Re: [PATCH v2 0/5] Remove Linux generic sysdep
Date: Sun, 19 Feb 2023 16:29:00 -0500	[thread overview]
Message-ID: <2f160bc1-d240-17a8-c90f-ac601b0807e9@redhat.com> (raw)
In-Reply-To: <20221207212246.2384051-1-adhemerval.zanella@linaro.org>

On 12/7/22 16:22, Adhemerval Zanella wrote:
> With recent BZ#29657, where struct stat were not adapted for 64 time
> because I forgot that generic folder is not always used for ports with
> default 64 bit time_t; it seems better to finish the generic
> integration and make the Linux default implementation the de-facto
> "generic" implementation.

I reviewed the oustanding patches and I think this is ready to go in.

I suggest:
- Post v3 with my rb+tb carried forward (requires a rebase of one patch).
- Push v3.
- Mark v2 as superseded.
 
> Changes for v2:
> - Fixed sparc32 getsockname and getpeername kernel definition.
> - Added epoll_create and inotify_init Makefile rules.
> - Expanded commits messages.
> 
> Adhemerval Zanella (5):
>   Linux: Move wordsize-32 Version to default
>   Linux: Assume and consolidate getsockname wire-up syscall
>   Linux: Assume and consolidate getpeername wire-up syscall
>   Linux: Remove unused generic Makefile
>   Linux: Remove generic Implies
> 
>  sysdeps/unix/sysv/linux/Makefile                     |  2 ++
>  sysdeps/unix/sysv/linux/Versions                     |  3 +++
>  sysdeps/unix/sysv/linux/aarch64/Implies              |  1 -
>  sysdeps/unix/sysv/linux/arc/Implies                  |  2 --
>  sysdeps/unix/sysv/linux/arm/Versions                 |  3 ---
>  sysdeps/unix/sysv/linux/csky/Implies                 |  2 --
>  sysdeps/unix/sysv/linux/generic/Makefile             |  3 ---
>  sysdeps/unix/sysv/linux/generic/README               | 11 -----------
>  sysdeps/unix/sysv/linux/generic/syscalls.list        |  5 -----
>  sysdeps/unix/sysv/linux/generic/wordsize-32/Versions |  5 -----
>  sysdeps/unix/sysv/linux/getpeername.c                |  7 +------
>  sysdeps/unix/sysv/linux/getsockname.c                |  7 +------
>  sysdeps/unix/sysv/linux/hppa/Versions                |  3 ---
>  sysdeps/unix/sysv/linux/i386/Versions                |  4 ----
>  sysdeps/unix/sysv/linux/i386/kernel-features.h       |  8 ++------
>  sysdeps/unix/sysv/linux/kernel-features.h            |  2 ++
>  sysdeps/unix/sysv/linux/loongarch/lp64/Implies       |  1 -
>  sysdeps/unix/sysv/linux/m68k/Versions                |  3 ---
>  sysdeps/unix/sysv/linux/m68k/kernel-features.h       |  8 ++------
>  sysdeps/unix/sysv/linux/microblaze/Versions          |  5 -----
>  sysdeps/unix/sysv/linux/microblaze/kernel-features.h |  2 --
>  sysdeps/unix/sysv/linux/mips/Versions                |  3 ---
>  sysdeps/unix/sysv/linux/nios2/Implies                |  2 --
>  sysdeps/unix/sysv/linux/or1k/Implies                 |  2 --
>  sysdeps/unix/sysv/linux/powerpc/kernel-features.h    |  2 --
>  sysdeps/unix/sysv/linux/powerpc/powerpc32/Versions   |  3 ---
>  sysdeps/unix/sysv/linux/riscv/rv32/Implies           |  2 --
>  sysdeps/unix/sysv/linux/riscv/rv64/Implies           |  1 -
>  sysdeps/unix/sysv/linux/s390/kernel-features.h       |  8 ++------
>  sysdeps/unix/sysv/linux/s390/s390-32/Versions        |  3 ---
>  sysdeps/unix/sysv/linux/sh/Versions                  |  3 ---
>  sysdeps/unix/sysv/linux/sh/kernel-features.h         |  2 --
>  sysdeps/unix/sysv/linux/sparc/kernel-features.h      |  7 +++++++
>  sysdeps/unix/sysv/linux/sparc/sparc32/Versions       |  3 ---
>  sysdeps/unix/sysv/linux/syscalls.list                |  2 --
>  35 files changed, 22 insertions(+), 108 deletions(-)
>  delete mode 100644 sysdeps/unix/sysv/linux/generic/Makefile
>  delete mode 100644 sysdeps/unix/sysv/linux/generic/README
>  delete mode 100644 sysdeps/unix/sysv/linux/generic/syscalls.list
>  delete mode 100644 sysdeps/unix/sysv/linux/generic/wordsize-32/Versions
>  delete mode 100644 sysdeps/unix/sysv/linux/microblaze/Versions
> 

-- 
Cheers,
Carlos.


      parent reply	other threads:[~2023-02-20 13:34 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-07 21:22 [PATCH v2 0/5] Remove Linux generic sysdep Adhemerval Zanella via Libc-alpha
2022-12-07 21:22 ` [PATCH v2 1/5] Linux: Move wordsize-32 Version to default Adhemerval Zanella via Libc-alpha
2023-02-19 21:11   ` Carlos O'Donell via Libc-alpha
2022-12-07 21:22 ` [PATCH v2 2/5] Linux: Assume and consolidate getsockname wire-up syscall Adhemerval Zanella via Libc-alpha
2023-02-19 21:20   ` Carlos O'Donell via Libc-alpha
2022-12-07 21:22 ` [PATCH v2 3/5] Linux: Assume and consolidate getpeername " Adhemerval Zanella via Libc-alpha
2023-02-19 21:23   ` Carlos O'Donell via Libc-alpha
2022-12-07 21:22 ` [PATCH v2 4/5] Linux: Remove unused generic Makefile Adhemerval Zanella via Libc-alpha
2023-02-19 21:27   ` Carlos O'Donell via Libc-alpha
2022-12-08 12:46 ` [PATCH v2 5/5] Linux: Remove generic Implies Adhemerval Zanella via Libc-alpha
2023-02-19 21:29 ` Carlos O'Donell via Libc-alpha [this message]

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

  List information: https://www.gnu.org/software/libc/involved.html

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

  git send-email \
    --in-reply-to=2f160bc1-d240-17a8-c90f-ac601b0807e9@redhat.com \
    --to=libc-alpha@sourceware.org \
    --cc=adhemerval.zanella@linaro.org \
    --cc=carlos@redhat.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).