From 5fb3b8e377deeb0ce362a7bcb9323542b579fdef Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Mon, 18 May 2009 13:30:16 +0200 Subject: Move down the become_root()/unbecome_root() calls into the VFS modules The aio_fork module does not need this, as it does not communicate via signals but with pipes. Watching a strace log with those become_root() calls in aio.c is absolutely awful, and it does affect performance. --- source3/modules/vfs_default.c | 24 ++++++++++++++++++++++-- 1 file changed, 22 insertions(+), 2 deletions(-) (limited to 'source3/modules') diff --git a/source3/modules/vfs_default.c b/source3/modules/vfs_default.c index 6c1946a99d..aa207056b3 100644 --- a/source3/modules/vfs_default.c +++ b/source3/modules/vfs_default.c @@ -1423,12 +1423,32 @@ static int vfswrap_fsetxattr(struct vfs_handle_struct *handle, struct files_stru static int vfswrap_aio_read(struct vfs_handle_struct *handle, struct files_struct *fsp, SMB_STRUCT_AIOCB *aiocb) { - return sys_aio_read(aiocb); + int ret; + /* + * aio_read must be done as root, because in the glibc aio + * implementation the helper thread needs to be able to send a signal + * to the main thread, even when it has done a seteuid() to a + * different user. + */ + become_root(); + ret = sys_aio_read(aiocb); + unbecome_root(); + return ret; } static int vfswrap_aio_write(struct vfs_handle_struct *handle, struct files_struct *fsp, SMB_STRUCT_AIOCB *aiocb) { - return sys_aio_write(aiocb); + int ret; + /* + * aio_write must be done as root, because in the glibc aio + * implementation the helper thread needs to be able to send a signal + * to the main thread, even when it has done a seteuid() to a + * different user. + */ + become_root(); + ret = sys_aio_write(aiocb); + unbecome_root(); + return ret; } static ssize_t vfswrap_aio_return(struct vfs_handle_struct *handle, struct files_struct *fsp, SMB_STRUCT_AIOCB *aiocb) -- cgit