On 01/25/2012 06:09 PM, Eric Wong wrote: > As the replacement cannot be made thread-safe, we need to > document this to prevent users from having a false sense > of safety. > > * doc/glibc-functions/pipe2.texi (pipe2): mention lack of thread-safety > > Signed-off-by: Eric Wong <normalper...@yhbt.net> > --- > I took the line off the existing documentation for the > similar-in-purpose *at() functions.
We need the same for accept4() and other places where we have non-atomic replacements for FD_CLOEXEC (and someday, I need to get around to implementing open(O_CLOEXEC), which would also be non-atomic). I'm pushing this instead. From 41423a6ca6273e751ca6fd2376641f57b8e40ff8 Mon Sep 17 00:00:00 2001 From: Eric Blake <ebl...@redhat.com> Date: Thu, 26 Jan 2012 08:31:48 -0700 Subject: [PATCH] pipe2, assign4: document lack of thread-safety in replacement * doc/glibc-functions/pipe2.texi (pipe2): Mention thread safety issue in replacement. * doc/glibc-functions/accept4.texi (accept4): Likewise. Based on a report by Eric Wong. --- ChangeLog | 8 ++++++++ doc/glibc-functions/accept4.texi | 1 + doc/glibc-functions/pipe2.texi | 1 + 3 files changed, 10 insertions(+), 0 deletions(-) diff --git a/ChangeLog b/ChangeLog index 2aa1e83..a1c8a29 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2012-01-26 Eric Blake <ebl...@redhat.com> + + pipe2: document lack of thread-safety in replacement + * doc/glibc-functions/pipe2.texi (pipe2): Mention thread safety + issue in replacement. + * doc/glibc-functions/accept4.texi (accept4): Likewise. + Based on a report by Eric Wong. + 2012-01-24 Marc-André Lureau <marcandre.lur...@redhat.com> (tiny change) Bruno Haible <br...@clisp.org> diff --git a/doc/glibc-functions/accept4.texi b/doc/glibc-functions/accept4.texi index f21d62b..cd0486c 100644 --- a/doc/glibc-functions/accept4.texi +++ b/doc/glibc-functions/accept4.texi @@ -10,6 +10,7 @@ accept4 This function is missing on many non-glibc platforms: MacOS X 10.5, FreeBSD 6.0, NetBSD 5.0, OpenBSD 3.8, Minix 3.1.8, AIX 5.1, HP-UX 11, IRIX 6.5, OSF/1 5.1, Solaris 11 2011-11, Cygwin 1.7.1, mingw, MSVC 9, Interix 3.5, BeOS. +But the replacement function is not atomic in multi-threaded programs. @end itemize Portability problems not fixed by Gnulib: diff --git a/doc/glibc-functions/pipe2.texi b/doc/glibc-functions/pipe2.texi index 2fef538..e632d3d 100644 --- a/doc/glibc-functions/pipe2.texi +++ b/doc/glibc-functions/pipe2.texi @@ -10,6 +10,7 @@ pipe2 This function is missing on many non-glibc platforms: MacOS X 10.5, FreeBSD 6.0, NetBSD 5.0, OpenBSD 3.8, Minix 3.1.8, AIX 5.1, HP-UX 11, IRIX 6.5, OSF/1 5.1, Solaris 11 2011-11, Cygwin 1.7.1, mingw, MSVC 9, Interix 3.5, BeOS. +But the replacement function is not atomic in multi-threaded programs. @end itemize Portability problems not fixed by Gnulib: -- 1.7.7.6 -- Eric Blake ebl...@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature