Received-SPF: pass (google.com: domain of linux-kernel+bounces-73915-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240216235203.229256-1-irogers@google.com> <20240216235203.229256-3-irogers@google.com>
In-Reply-To: <20240216235203.229256-3-irogers@google.com>
From: Namhyung Kim <namhyung@kernel.org>
Date: Tue, 20 Feb 2024 17:53:03 -0800
Message-ID: <CAM9d7cgqpvXLgq35B5MoysFzLeUBg7bJzX+pTK_ER=-hM0D=AQ@mail.gmail.com>
Subject: Re: [PATCH v5 2/8] perf list: Add scandirat compatibility function
To: Ian Rogers <irogers@google.com>
Cc: Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>, 
	Arnaldo Carvalho de Melo <acme@kernel.org>, Mark Rutland <mark.rutland@arm.com>, 
	Alexander Shishkin <alexander.shishkin@linux.intel.com>, Jiri Olsa <jolsa@kernel.org>, 
	Adrian Hunter <adrian.hunter@intel.com>, Nathan Chancellor <nathan@kernel.org>, 
	Nick Desaulniers <ndesaulniers@google.com>, Bill Wendling <morbo@google.com>, 
	Justin Stitt <justinstitt@google.com>, Athira Jajeev <atrajeev@linux.vnet.ibm.com>, 
	James Clark <james.clark@arm.com>, Kan Liang <kan.liang@linux.intel.com>, 
	Yang Jihong <yangjihong1@huawei.com>, linux-kernel@vger.kernel.org, 
	linux-perf-users@vger.kernel.org, llvm@lists.linux.dev
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Ian,

On Fri, Feb 16, 2024 at 3:55=E2=80=AFPM Ian Rogers <irogers@google.com> wro=
te:
>
> scandirat is used during the printing of tracepoint events but may be
> missing from certain libcs. Add a compatibility implementation that
> uses the symlink of an fd in /proc as a path for the reliably present
> scandir.
>
> Signed-off-by: Ian Rogers <irogers@google.com>

This fails to build in alpine:

  util/print-events.c: In function 'print_tracepoint_events':
  util/print-events.c:92:29: error: implicit declaration of function
'scandirat'; did you mean 'scandir'?
[-Werror=3Dimplicit-function-declaration]
  92 |                 evt_items =3D scandirat(events_fd,
sys_dirent->d_name, &evt_namelist, NULL, alphasort);
  |                             ^~~~~~~~~
  |                             scandir

Thanks,
Namhyung


> ---
>  tools/perf/util/print-events.c | 12 +++---------
>  tools/perf/util/util.c         | 19 +++++++++++++++++++
>  tools/perf/util/util.h         |  8 ++++++++
>  3 files changed, 30 insertions(+), 9 deletions(-)
>
> diff --git a/tools/perf/util/print-events.c b/tools/perf/util/print-event=
s.c
> index 9e47712507cc..bf79dd366e2e 100644
> --- a/tools/perf/util/print-events.c
> +++ b/tools/perf/util/print-events.c
> @@ -63,6 +63,8 @@ void print_tracepoint_events(const struct print_callbac=
ks *print_cb __maybe_unus
>  {
>         char *events_path =3D get_tracing_file("events");
>         int events_fd =3D open(events_path, O_PATH);
> +       struct dirent **sys_namelist =3D NULL;
> +       int sys_items;
>
>         put_tracing_file(events_path);
>         if (events_fd < 0) {
> @@ -70,10 +72,7 @@ void print_tracepoint_events(const struct print_callba=
cks *print_cb __maybe_unus
>                 return;
>         }
>
> -#ifdef HAVE_SCANDIRAT_SUPPORT
> -{
> -       struct dirent **sys_namelist =3D NULL;
> -       int sys_items =3D tracing_events__scandir_alphasort(&sys_namelist=
);
> +       sys_items =3D tracing_events__scandir_alphasort(&sys_namelist);
>
>         for (int i =3D 0; i < sys_items; i++) {
>                 struct dirent *sys_dirent =3D sys_namelist[i];
> @@ -130,11 +129,6 @@ void print_tracepoint_events(const struct print_call=
backs *print_cb __maybe_unus
>         }
>
>         free(sys_namelist);
> -}
> -#else
> -       printf("\nWARNING: Your libc doesn't have the scandirat function,=
 please ask its maintainers to implement it.\n"
> -              "         As a rough fallback, please do 'ls %s' to see th=
e available tracepoint events.\n", events_path);
> -#endif
>         close(events_fd);
>  }
>
> diff --git a/tools/perf/util/util.c b/tools/perf/util/util.c
> index c1fd9ba6d697..4f561e5e4162 100644
> --- a/tools/perf/util/util.c
> +++ b/tools/perf/util/util.c
> @@ -552,3 +552,22 @@ int sched_getcpu(void)
>         return -1;
>  }
>  #endif
> +
> +#ifndef HAVE_SCANDIRAT_SUPPORT
> +int scandirat(int dirfd, const char *dirp,
> +             struct dirent ***namelist,
> +             int (*filter)(const struct dirent *),
> +             int (*compar)(const struct dirent **, const struct dirent *=
*))
> +{
> +       char path[PATH_MAX];
> +       int err, fd =3D openat(dirfd, dirp, O_PATH);
> +
> +       if (fd < 0)
> +               return fd;
> +
> +       snprintf(path, sizeof(path), "/proc/%d/fd/%d", getpid(), fd);
> +       err =3D scandir(path, namelist, filter, compar);
> +       close(fd);
> +       return err;
> +}
> +#endif
> diff --git a/tools/perf/util/util.h b/tools/perf/util/util.h
> index 7c8915d92dca..9966c21aaf04 100644
> --- a/tools/perf/util/util.h
> +++ b/tools/perf/util/util.h
> @@ -6,6 +6,7 @@
>  /* glibc 2.20 deprecates _BSD_SOURCE in favour of _DEFAULT_SOURCE */
>  #define _DEFAULT_SOURCE 1
>
> +#include <dirent.h>
>  #include <fcntl.h>
>  #include <stdbool.h>
>  #include <stddef.h>
> @@ -56,6 +57,13 @@ int perf_tip(char **strp, const char *dirpath);
>  int sched_getcpu(void);
>  #endif
>
> +#ifndef HAVE_SCANDIRAT_SUPPORT
> +int scandirat(int dirfd, const char *dirp,
> +             struct dirent ***namelist,
> +             int (*filter)(const struct dirent *),
> +             int (*compar)(const struct dirent **, const struct dirent *=
*));
> +#endif
> +
>  extern bool perf_singlethreaded;
>
>  void perf_set_singlethreaded(void);
> --
> 2.44.0.rc0.258.g7320e95886-goog
>