Simplify sytem call renaming. Instead of _foo() <-- _libc_foo <-- foo(),

just use _foo() <-- foo().  In the case of a libpthread that doesn't do
call conversion (such as linuxthreads and our upcoming libpthread), this
is adequate.  In the case of libc_r, we still need three names, which are
now _thread_sys_foo() <-- _foo() <-- foo().

Convert all internal libc usage of: aio_suspend(), close(), fsync(), msync(),
nanosleep(), open(), fcntl(), read(), and write() to _foo() instead of foo().

Remove all internal libc usage of: creat(), pause(), sleep(), system(),
tcdrain(), wait(), and waitpid().

Make thread cancellation fully POSIX-compliant.

Suggested by:	deischen

1 files changed, 12 insertions, 7 deletions

diff --git a/lib/libkse/thread/thr_read.c b/lib/libkse/thread/thr_read.c
index ad66df3..b4d9694 100644
--- a/lib/libkse/thread/thr_read.c
+++ b/lib/libkse/thread/thr_read.c
@@ -42,16 +42,13 @@
 #include "pthread_private.h"
 
 ssize_t
-_libc_read(int fd, void *buf, size_t nbytes)
+_read(int fd, void *buf, size_t nbytes)
 {
 	int	ret;
 	int	type;
 
-	_thread_enter_cancellation_point();
-
 	/* POSIX says to do just this: */
 	if (nbytes == 0) {
-		_thread_leave_cancellation_point();
 		return (0);
 	}
 
@@ -65,7 +62,6 @@ _libc_read(int fd, void *buf, size_t nbytes)
 			/* File is not open for read: */
 			errno = EBADF;
 			_FD_UNLOCK(fd, FD_READ);
-			_thread_leave_cancellation_point();
 			return (-1);
 		}
 
@@ -97,9 +93,18 @@ _libc_read(int fd, void *buf, size_t nbytes)
 		}
 		_FD_UNLOCK(fd, FD_READ);
 	}
-	_thread_leave_cancellation_point();
 	return (ret);
 }
 
-__weak_reference(_libc_read, read);
+ssize_t
+read(int fd, void *buf, size_t nbytes)
+{
+	ssize_t	ret;
+
+	_thread_enter_cancellation_point();
+	ret = _read(fd, buf, nbytes);
+	_thread_leave_cancellation_point();
+
+	return ret;
+}
 #endif