diff options
author | Arnaldo Carvalho de Melo <acme@redhat.com> | 2016-04-08 11:53:02 -0300 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2016-05-12 11:26:58 -0300 |
commit | 22a9f41b555673e7499b97acf3ffb07bf0af31ad (patch) | |
tree | d15ff34cc873566d170aa845b178d612376c3d2f /kernel/auditsc.c | |
parent | 7839b9f32e45075d9eb48da8480faef3dbd019f0 (diff) | |
download | lwn-22a9f41b555673e7499b97acf3ffb07bf0af31ad.tar.gz lwn-22a9f41b555673e7499b97acf3ffb07bf0af31ad.zip |
perf tools: Use readdir() instead of deprecated readdir_r()
The readdir() function is thread safe as long as just one thread uses a
DIR, which is the case when parsing tracepoint event definitions, to
avoid breaking the build with glibc-2.23.90 (upcoming 2.24), use it
instead of readdir_r().
See: http://man7.org/linux/man-pages/man3/readdir.3.html
"However, in modern implementations (including the glibc implementation),
concurrent calls to readdir() that specify different directory streams
are thread-safe. In cases where multiple threads must read from the
same directory stream, using readdir() with external synchronization is
still preferable to the use of the deprecated readdir_r(3) function."
Noticed while building on a Fedora Rawhide docker container.
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-wddn49r6bz6wq4ee3dxbl7lo@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'kernel/auditsc.c')
0 files changed, 0 insertions, 0 deletions