From: Adhemerval Zanella <adhemerval.zanella@linaro.org>
To: libc-alpha@sourceware.org
Subject: Re: [PATCH 5/7] Consolidate scandir{at}{64} implementation
Date: Fri, 20 Apr 2018 13:50:00 -0000 [thread overview]
Message-ID: <0c28c0d7-7bb5-20e8-8773-b43302aa1535@linaro.org> (raw)
In-Reply-To: <1520017165-15830-5-git-send-email-adhemerval.zanella@linaro.org>
I will commit this shortly if no one opposes it.
On 02/03/2018 15:59, Adhemerval Zanella wrote:
> This patch consolidates scandir{at}{64} implementation on just
> the default dirent/scandir{at}{64}{_r}.c ones. It changes the logic
> to follow the conventions used on other code consolidation:
>
> * scandir{at} is only built for _DIRENT_MATCHES_DIRENT64 being 0.
>
> * scandir{at}{64} is always built and aliased to getdents for ABIs
> that define _DIRENT_MATCHES_DIRENT64 to 1.
>
> Also on Linux the compat symbol for old non-LFS dirent64 definition
> requires a platform-specific scandir64.c.
>
> Checked on aarch64-linux-gnu, x86_64-linux-gnu, i686-linux-gnu,
> sparcv9-linux-gnu, sparc64-linux-gnu, powerpc-linux-gnu, and
> powerpc64le-linux-gnu.
>
> * dirent/scandir-tail-common.c: New file.
> * dirent/scandir-tail.c: Use scandir-tail-common.c.
> (__scandir_tail): Build iff _DIRENT_MATCHES_DIRENT64 is not defined.
> * dirent/scandir.c: Use scandir-tail-common.c.
> * dirent/scandirat.c: Likewise.
> * dirent/scandir64-tail.c: Use scandir-tail-common.c.
> * dirent/scandir64.c (scandir64): Always build and alias to scandir
> if _DIRENT_MATCHES_DIRENT64 is defined.
> * dirent/scandirat64.c (scandirat64): Likewise.
> * include/dirent.h (__scandir_tail): Only define iff
> _DIRENT_MATCHES_DIRENT64 is not defined.
> (__scandir64_tail): Define regardless.
> (__scandirat, scandirat64): Remove libc_hidden_proto.
> * sysdeps/unix/sysv/linux/arm/scandir64.c: Remove file.
> * sysdeps/unix/sysv/linux/m68k/scandir64.c: Likewise.
> * sysdeps/unix/sysv/linux/powerpc/powerpc32/scandir64.c: Likewise.
> * sysdeps/unix/sysv/linux/s390/s390-32/scandir64.c: Likewise.
> * sysdeps/unix/sysv/linux/i386/scandir64.c: Likewise.
> * sysdeps/unix/sysv/linux/sparc/sparc32/scandir64.c: Likewise.
> * sysdeps/unix/sysv/linux/scandir64.c: New file.
> ---
> ChangeLog | 21 +++++
> dirent/scandir-tail-common.c | 103 +++++++++++++++++++++
> dirent/scandir-tail.c | 95 ++-----------------
> dirent/scandir.c | 27 +-----
> dirent/scandir64-tail.c | 10 +-
> dirent/scandir64.c | 23 +++--
> dirent/scandirat.c | 30 +-----
> dirent/scandirat64.c | 23 +++--
> include/dirent.h | 18 ++--
> sysdeps/unix/sysv/linux/arm/scandir64.c | 1 -
> sysdeps/unix/sysv/linux/m68k/scandir64.c | 1 -
> .../unix/sysv/linux/powerpc/powerpc32/scandir64.c | 1 -
> sysdeps/unix/sysv/linux/s390/s390-32/scandir64.c | 1 -
> sysdeps/unix/sysv/linux/{i386 => }/scandir64.c | 37 ++++----
> sysdeps/unix/sysv/linux/sparc/sparc32/scandir64.c | 1 -
> 15 files changed, 198 insertions(+), 194 deletions(-)
> create mode 100644 dirent/scandir-tail-common.c
> delete mode 100644 sysdeps/unix/sysv/linux/arm/scandir64.c
> delete mode 100644 sysdeps/unix/sysv/linux/m68k/scandir64.c
> delete mode 100644 sysdeps/unix/sysv/linux/powerpc/powerpc32/scandir64.c
> delete mode 100644 sysdeps/unix/sysv/linux/s390/s390-32/scandir64.c
> rename sysdeps/unix/sysv/linux/{i386 => }/scandir64.c (81%)
> delete mode 100644 sysdeps/unix/sysv/linux/sparc/sparc32/scandir64.c
>
> diff --git a/dirent/scandir-tail-common.c b/dirent/scandir-tail-common.c
> new file mode 100644
> index 0000000..f89cf77
> --- /dev/null
> +++ b/dirent/scandir-tail-common.c
> @@ -0,0 +1,103 @@
> +/* Common implementation for scandir{at}.
> + Copyright (C) 2018 Free Software Foundation, Inc.
> + This file is part of the GNU C Library.
> +
> + The GNU C Library is free software; you can redistribute it and/or
> + modify it under the terms of the GNU Lesser General Public
> + License as published by the Free Software Foundation; either
> + version 2.1 of the License, or (at your option) any later version.
> +
> + The GNU C Library is distributed in the hope that it will be useful,
> + but WITHOUT ANY WARRANTY; without even the implied warranty of
> + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> + Lesser General Public License for more details.
> +
> + You should have received a copy of the GNU Lesser General Public
> + License along with the GNU C Library; if not, see
> + <http://www.gnu.org/licenses/>. */
> +
> +#include <string.h>
> +#include <errno.h>
> +
> +int
> +SCANDIR_TAIL (DIR *dp,
> + DIRENT_TYPE ***namelist,
> + int (*select) (const DIRENT_TYPE *),
> + int (*cmp) (const DIRENT_TYPE **, const DIRENT_TYPE **))
> +{
> + if (dp == NULL)
> + return -1;
> +
> + int save = errno;
> + __set_errno (0);
> +
> + int result;
> + struct scandir_cancel_struct c = { .dp = dp };
> + __libc_cleanup_push (&__scandir_cancel_handler, &c);
> +
> + DIRENT_TYPE **v = NULL;
> + size_t vsize = 0;
> + DIRENT_TYPE *d;
> + while ((d = READDIR (dp)) != NULL)
> + {
> + if (select != NULL)
> + {
> + int selected = (*select) (d);
> +
> + /* The SELECT function might have set errno to non-zero on
> + success. It was zero before and it needs to be again to
> + make the later tests work. */
> + __set_errno (0);
> +
> + if (!selected)
> + continue;
> + }
> +
> + if (__glibc_unlikely (c.cnt == vsize))
> + {
> + if (vsize == 0)
> + vsize = 10;
> + else
> + vsize *= 2;
> + DIRENT_TYPE **new = realloc (v, vsize * sizeof *v);
> + if (new == NULL)
> + break;
> + c.v = v = new;
> + }
> +
> + size_t dsize = &d->d_name[_D_ALLOC_NAMLEN (d)] - (char *) d;
> + DIRENT_TYPE *vnew = malloc (dsize);
> + if (vnew == NULL)
> + break;
> + v[c.cnt++] = (DIRENT_TYPE *) memcpy (vnew, d, dsize);
> +
> + /* Ignore errors from readdir, malloc or realloc. These functions
> + might have set errno to non-zero on success. It was zero before
> + and it needs to be again to make the latter tests work. */
> + __set_errno (0);
> + }
> +
> + if (__glibc_likely (errno == 0))
> + {
> + __closedir (dp);
> +
> + /* Sort the list if we have a comparison function to sort with. */
> + if (cmp != NULL)
> + qsort (v, c.cnt, sizeof *v, (__compar_fn_t) cmp);
> +
> + *namelist = v;
> + result = c.cnt;
> + }
> + else
> + {
> + /* This frees everything and calls closedir. */
> + __scandir_cancel_handler (&c);
> + result = -1;
> + }
> +
> + __libc_cleanup_pop (0);
> +
> + if (result >= 0)
> + __set_errno (save);
> + return result;
> +}
> diff --git a/dirent/scandir-tail.c b/dirent/scandir-tail.c
> index 67c9c92..7395dc4 100644
> --- a/dirent/scandir-tail.c
> +++ b/dirent/scandir-tail.c
> @@ -17,96 +17,13 @@
> <http://www.gnu.org/licenses/>. */
>
> #include <dirent.h>
> -#include <errno.h>
> -#include <stdlib.h>
> -#include <string.h>
> -#include <libc-lock.h>
>
> -#ifndef SCANDIR_TAIL
> -# define SCANDIR_TAIL __scandir_tail
> -# define READDIR __readdir
> -# define DIRENT_TYPE struct dirent
> -#endif
> -
> -int
> -SCANDIR_TAIL (DIR *dp,
> - DIRENT_TYPE ***namelist,
> - int (*select) (const DIRENT_TYPE *),
> - int (*cmp) (const DIRENT_TYPE **, const DIRENT_TYPE **))
> -{
> - if (dp == NULL)
> - return -1;
> -
> - int save = errno;
> - __set_errno (0);
> -
> - int result;
> - struct scandir_cancel_struct c = { .dp = dp };
> - __libc_cleanup_push (&__scandir_cancel_handler, &c);
> -
> - DIRENT_TYPE **v = NULL;
> - size_t vsize = 0;
> - DIRENT_TYPE *d;
> - while ((d = READDIR (dp)) != NULL)
> - {
> - if (select != NULL)
> - {
> - int selected = (*select) (d);
> -
> - /* The SELECT function might have set errno to non-zero on
> - success. It was zero before and it needs to be again to
> - make the later tests work. */
> - __set_errno (0);
> -
> - if (!selected)
> - continue;
> - }
> +#if !_DIRENT_MATCHES_DIRENT64
>
> - if (__glibc_unlikely (c.cnt == vsize))
> - {
> - if (vsize == 0)
> - vsize = 10;
> - else
> - vsize *= 2;
> - DIRENT_TYPE **new = realloc (v, vsize * sizeof *v);
> - if (new == NULL)
> - break;
> - c.v = v = new;
> - }
> +# define SCANDIR_TAIL __scandir_tail
> +# define READDIR __readdir
> +# define DIRENT_TYPE struct dirent
>
> - size_t dsize = &d->d_name[_D_ALLOC_NAMLEN (d)] - (char *) d;
> - DIRENT_TYPE *vnew = malloc (dsize);
> - if (vnew == NULL)
> - break;
> - v[c.cnt++] = (DIRENT_TYPE *) memcpy (vnew, d, dsize);
> +# include <dirent/scandir-tail-common.c>
>
> - /* Ignore errors from readdir, malloc or realloc. These functions
> - might have set errno to non-zero on success. It was zero before
> - and it needs to be again to make the latter tests work. */
> - __set_errno (0);
> - }
> -
> - if (__glibc_likely (errno == 0))
> - {
> - __closedir (dp);
> -
> - /* Sort the list if we have a comparison function to sort with. */
> - if (cmp != NULL)
> - qsort (v, c.cnt, sizeof *v, (__compar_fn_t) cmp);
> -
> - *namelist = v;
> - result = c.cnt;
> - }
> - else
> - {
> - /* This frees everything and calls closedir. */
> - __scandir_cancel_handler (&c);
> - result = -1;
> - }
> -
> - __libc_cleanup_pop (0);
> -
> - if (result >= 0)
> - __set_errno (save);
> - return result;
> -}
> +#endif
> diff --git a/dirent/scandir.c b/dirent/scandir.c
> index b24e157..6d8352d 100644
> --- a/dirent/scandir.c
> +++ b/dirent/scandir.c
> @@ -15,31 +15,14 @@
> License along with the GNU C Library; if not, see
> <http://www.gnu.org/licenses/>. */
>
> -/* We need to avoid the header declaration of scandir64, because
> - the types don't match scandir and then the compiler will
> - complain about the mismatch when we do the alias below. */
> -#define scandir64 __renamed_scandir64
> -
> #include <dirent.h>
>
> -#undef scandir64
> -
> -#ifndef SCANDIR
> -# define SCANDIR scandir
> -# define SCANDIR_TAIL __scandir_tail
> -# define DIRENT_TYPE struct dirent
> -#endif
> -
> -
> +#if !_DIRENT_MATCHES_DIRENT64
> int
> -SCANDIR (const char *dir,
> - DIRENT_TYPE ***namelist,
> - int (*select) (const DIRENT_TYPE *),
> - int (*cmp) (const DIRENT_TYPE **, const DIRENT_TYPE **))
> +scandir (const char *dir, struct dirent ***namelist,
> + int (*select) (const struct dirent *),
> + int (*cmp) (const struct dirent **, const struct dirent **))
> {
> - return SCANDIR_TAIL (__opendir (dir), namelist, select, cmp);
> + return __scandir_tail (__opendir (dir), namelist, select, cmp);
> }
> -
> -#if _DIRENT_MATCHES_DIRENT64
> -weak_alias (scandir, scandir64)
> #endif
> diff --git a/dirent/scandir64-tail.c b/dirent/scandir64-tail.c
> index 8d5cc07..4e873d4 100644
> --- a/dirent/scandir64-tail.c
> +++ b/dirent/scandir64-tail.c
> @@ -18,9 +18,7 @@
>
> #include <dirent.h>
>
> -#if !_DIRENT_MATCHES_DIRENT64
> -# define SCANDIR_TAIL __scandir64_tail
> -# define READDIR __readdir64
> -# define DIRENT_TYPE struct dirent64
> -# include <scandir-tail.c>
> -#endif
> +#define SCANDIR_TAIL __scandir64_tail
> +#define READDIR __readdir64
> +#define DIRENT_TYPE struct dirent64
> +#include <scandir-tail-common.c>
> diff --git a/dirent/scandir64.c b/dirent/scandir64.c
> index 0c63fa9..6bdd462 100644
> --- a/dirent/scandir64.c
> +++ b/dirent/scandir64.c
> @@ -15,15 +15,18 @@
> License along with the GNU C Library; if not, see
> <http://www.gnu.org/licenses/>. */
>
> +#define scandir __no_scandir_decl
> #include <dirent.h>
> -
> -/* scandir.c defines scandir64 as an alias if _DIRENT_MATCHES_DIRENT64. */
> -#if !_DIRENT_MATCHES_DIRENT64
> -
> -# define SCANDIR scandir64
> -# define SCANDIR_TAIL __scandir64_tail
> -# define DIRENT_TYPE struct dirent64
> -
> -# include <dirent/scandir.c>
> -
> +#undef scandir
> +
> +int
> +scandir64 (const char *dir, struct dirent64 ***namelist,
> + int (*select) (const struct dirent64 *),
> + int (*cmp) (const struct dirent64 **, const struct dirent64 **))
> +{
> + return __scandir64_tail (__opendir (dir), namelist, select, cmp);
> +}
> +
> +#if _DIRENT_MATCHES_DIRENT64
> +weak_alias (scandir64, scandir)
> #endif
> diff --git a/dirent/scandirat.c b/dirent/scandirat.c
> index 96a1b0e..8dad1e8 100644
> --- a/dirent/scandirat.c
> +++ b/dirent/scandirat.c
> @@ -15,35 +15,15 @@
> License along with the GNU C Library; if not, see
> <http://www.gnu.org/licenses/>. */
>
> -/* We need to avoid the header declaration of scandir64, because
> - the types don't match scandir and then the compiler will
> - complain about the mismatch when we do the alias below. */
> -#define scandirat64 __renamed_scandirat64
> -
> #include <dirent.h>
>
> -#undef scandirat64
> -
> -#ifndef SCANDIRAT
> -# define SCANDIRAT __scandirat
> -# define SCANDIR_TAIL __scandir_tail
> -# define DIRENT_TYPE struct dirent
> -# define SCANDIRAT_WEAK_ALIAS
> -#endif
> -
> +#if !_DIRENT_MATCHES_DIRENT64
> int
> -SCANDIRAT (int dfd, const char *dir,
> - DIRENT_TYPE ***namelist,
> - int (*select) (const DIRENT_TYPE *),
> - int (*cmp) (const DIRENT_TYPE **, const DIRENT_TYPE **))
> +__scandirat (int dfd, const char *dir, struct dirent ***namelist,
> + int (*select) (const struct dirent *),
> + int (*cmp) (const struct dirent **, const struct dirent **))
> {
> - return SCANDIR_TAIL (__opendirat (dfd, dir), namelist, select, cmp);
> + return __scandir_tail (__opendirat (dfd, dir), namelist, select, cmp);
> }
> -libc_hidden_def (SCANDIRAT)
> -#ifdef SCANDIRAT_WEAK_ALIAS
> weak_alias (__scandirat, scandirat)
> #endif
> -
> -#if _DIRENT_MATCHES_DIRENT64
> -weak_alias (scandirat, scandirat64)
> -#endif
> diff --git a/dirent/scandirat64.c b/dirent/scandirat64.c
> index 520ae7c..7e5e209 100644
> --- a/dirent/scandirat64.c
> +++ b/dirent/scandirat64.c
> @@ -15,15 +15,18 @@
> License along with the GNU C Library; if not, see
> <http://www.gnu.org/licenses/>. */
>
> +#define scandirat __no_scandirat_decl
> #include <dirent.h>
> -
> -/* scandirat.c defines scandirat64 as an alias if _DIRENT_MATCHES_DIRENT64. */
> -#if !_DIRENT_MATCHES_DIRENT64
> -
> -# define SCANDIRAT scandirat64
> -# define SCANDIR_TAIL __scandir64_tail
> -# define DIRENT_TYPE struct dirent64
> -
> -# include <scandirat.c>
> -
> +#undef scandirat
> +
> +int
> +scandirat64 (int dfd, const char *dir, struct dirent64 ***namelist,
> + int (*select) (const struct dirent64 *),
> + int (*cmp) (const struct dirent64 **, const struct dirent64 **))
> +{
> + return __scandir64_tail (__opendirat (dfd, dir), namelist, select, cmp);
> +}
> +
> +#if _DIRENT_MATCHES_DIRENT64
> +weak_alias (scandirat64, scandirat)
> #endif
> diff --git a/include/dirent.h b/include/dirent.h
> index cc8f189..f8f1942 100644
> --- a/include/dirent.h
> +++ b/include/dirent.h
> @@ -57,25 +57,23 @@ extern int __scandir_tail (DIR *dp,
> int (*cmp) (const struct dirent **,
> const struct dirent **))
> attribute_hidden;
> -# if _DIRENT_MATCHES_DIRENT64
> -# define __scandir64_tail (dp, namelist, select, cmp) \
> - __scandir_tail (dp, (struct dirent ***) (namelist), \
> - (int (*) (const struct dirent *)) (select), \
> - (int (*) (const struct dirent **, \
> - const struct dirent **)) (cmp))
> -# else
> +# if !_DIRENT_MATCHES_DIRENT64
> +extern int __scandir_tail (DIR *dp,
> + struct dirent ***namelist,
> + int (*select) (const struct dirent *),
> + int (*cmp) (const struct dirent **,
> + const struct dirent **))
> + attribute_hidden;
> +# endif
> extern int __scandir64_tail (DIR *dp,
> struct dirent64 ***namelist,
> int (*select) (const struct dirent64 *),
> int (*cmp) (const struct dirent64 **,
> const struct dirent64 **))
> attribute_hidden;
> -# endif
>
> libc_hidden_proto (__rewinddir)
> extern __typeof (scandirat) __scandirat;
> -libc_hidden_proto (__scandirat)
> -libc_hidden_proto (scandirat64)
>
> # if IS_IN (rtld) && !defined NO_RTLD_HIDDEN
> extern __typeof (__rewinddir) __rewinddir attribute_hidden;
> diff --git a/sysdeps/unix/sysv/linux/arm/scandir64.c b/sysdeps/unix/sysv/linux/arm/scandir64.c
> deleted file mode 100644
> index 506fd88..0000000
> --- a/sysdeps/unix/sysv/linux/arm/scandir64.c
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <sysdeps/unix/sysv/linux/i386/scandir64.c>
> diff --git a/sysdeps/unix/sysv/linux/m68k/scandir64.c b/sysdeps/unix/sysv/linux/m68k/scandir64.c
> deleted file mode 100644
> index 506fd88..0000000
> --- a/sysdeps/unix/sysv/linux/m68k/scandir64.c
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <sysdeps/unix/sysv/linux/i386/scandir64.c>
> diff --git a/sysdeps/unix/sysv/linux/powerpc/powerpc32/scandir64.c b/sysdeps/unix/sysv/linux/powerpc/powerpc32/scandir64.c
> deleted file mode 100644
> index 506fd88..0000000
> --- a/sysdeps/unix/sysv/linux/powerpc/powerpc32/scandir64.c
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <sysdeps/unix/sysv/linux/i386/scandir64.c>
> diff --git a/sysdeps/unix/sysv/linux/s390/s390-32/scandir64.c b/sysdeps/unix/sysv/linux/s390/s390-32/scandir64.c
> deleted file mode 100644
> index 506fd88..0000000
> --- a/sysdeps/unix/sysv/linux/s390/s390-32/scandir64.c
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <sysdeps/unix/sysv/linux/i386/scandir64.c>
> diff --git a/sysdeps/unix/sysv/linux/i386/scandir64.c b/sysdeps/unix/sysv/linux/scandir64.c
> similarity index 81%
> rename from sysdeps/unix/sysv/linux/i386/scandir64.c
> rename to sysdeps/unix/sysv/linux/scandir64.c
> index 50590c3..efb502f 100644
> --- a/sysdeps/unix/sysv/linux/i386/scandir64.c
> +++ b/sysdeps/unix/sysv/linux/scandir64.c
> @@ -15,26 +15,28 @@
> License along with the GNU C Library; if not, see
> <http://www.gnu.org/licenses/>. */
>
> +#define scandir __no_scandir_decl
> #include <dirent.h>
> +#undef scandir
>
> -#define SCANDIR __scandir64
> -#define SCANDIR_TAIL __scandir64_tail
> -#define DIRENT_TYPE struct dirent64
> -
> -#include <dirent/scandir.c>
> -
> -#undef SCANDIR
> -#undef SCANDIR_TAIL
> -#undef DIRENT_TYPE
> -
> -#include <shlib-compat.h>
> +int
> +__scandir64 (const char *dir, struct dirent64 ***namelist,
> + int (*select) (const struct dirent64 *),
> + int (*cmp) (const struct dirent64 **, const struct dirent64 **))
> +{
> + return __scandir64_tail (__opendir (dir), namelist, select, cmp);
> +}
>
> +#if _DIRENT_MATCHES_DIRENT64
> +weak_alias (__scandir64, scandir64)
> +weak_alias (__scandir64, scandir)
> +#else
> +# include <shlib-compat.h>
> versioned_symbol (libc, __scandir64, scandir64, GLIBC_2_2);
> -
> -#if SHLIB_COMPAT (libc, GLIBC_2_1, GLIBC_2_2)
> -# include <string.h>
> -# include <errno.h>
> -# include "olddirent.h"
> +# if SHLIB_COMPAT (libc, GLIBC_2_1, GLIBC_2_2)
> +# include <string.h>
> +# include <errno.h>
> +# include "olddirent.h"
>
> int
> __old_scandir64 (const char *dir, struct __old_dirent64 ***namelist,
> @@ -133,4 +135,5 @@ __old_scandir64 (const char *dir, struct __old_dirent64 ***namelist,
> }
> compat_symbol (libc, __old_scandir64, scandir64, GLIBC_2_1);
>
> -#endif
> +# endif /* SHLIB_COMPAT (libc, GLIBC_2_1, GLIBC_2_2) */
> +#endif /* _DIRENT_MATCHES_DIRENT64 */
> diff --git a/sysdeps/unix/sysv/linux/sparc/sparc32/scandir64.c b/sysdeps/unix/sysv/linux/sparc/sparc32/scandir64.c
> deleted file mode 100644
> index 506fd88..0000000
> --- a/sysdeps/unix/sysv/linux/sparc/sparc32/scandir64.c
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <sysdeps/unix/sysv/linux/i386/scandir64.c>
>
next prev parent reply other threads:[~2018-04-20 13:50 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-02 18:59 [PATCH 1/7] Assume O_DIRECTORY for opendir Adhemerval Zanella
2018-03-02 18:59 ` [PATCH 4/7] Consolidate Linux getdents{64} implementation Adhemerval Zanella
2018-03-09 18:10 ` Adhemerval Zanella
2018-04-17 19:42 ` Adhemerval Zanella
2018-04-19 17:29 ` Joseph Myers
2018-04-19 18:23 ` Adhemerval Zanella
2018-03-02 18:59 ` [PATCH 7/7] Consolidate getdirentries{64} implementation Adhemerval Zanella
2018-04-25 19:49 ` Adhemerval Zanella
2018-03-02 18:59 ` [PATCH 6/7] Consolidate alphasort{64} and versionsort{64} implementation Adhemerval Zanella
2018-03-02 20:33 ` Joseph Myers
2018-03-06 11:28 ` Adhemerval Zanella
2018-04-23 18:01 ` Adhemerval Zanella
2018-03-02 18:59 ` [PATCH 5/7] Consolidate scandir{at}{64} implementation Adhemerval Zanella
2018-04-20 13:50 ` Adhemerval Zanella [this message]
2018-04-20 15:30 ` Florian Weimer
2018-04-20 15:56 ` Adhemerval Zanella
2018-03-02 18:59 ` [PATCH 2/7] Define _DIRENT_MATCHES_DIRENT64 regardless Adhemerval Zanella
2018-03-02 19:30 ` Andreas Schwab
2018-03-02 18:59 ` [PATCH 3/7] Consolidate Linux readdir{64}{_r} implementation Adhemerval Zanella
2018-04-06 19:49 ` Adhemerval Zanella
2018-04-02 13:59 ` [PATCH 1/7] Assume O_DIRECTORY for opendir Adhemerval Zanella
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=0c28c0d7-7bb5-20e8-8773-b43302aa1535@linaro.org \
--to=adhemerval.zanella@linaro.org \
--cc=libc-alpha@sourceware.org \
/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).