Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 19 Jan 2016 20:46:30 +0000 (UTC)
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r294339 - head/sys/kern
Message-ID:  <201601192046.u0JKkUNv028765@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jhb
Date: Tue Jan 19 20:46:30 2016
New Revision: 294339
URL: https://svnweb.freebsd.org/changeset/base/294339

Log:
  Don't create a dedicated session for each AIO kernel process.
  
  This code dates back to the initial AIO support and the commit log does
  not explain why it is needed.  However, I cannot find anything in the
  AIO code or the various file methods (fo_read/fo_write) that would change
  behavior due to using a private session instead of proc0's session.
  
  Reviewed by:	kib
  Sponsored by:	Chelsio Communications
  Differential Revision:	https://reviews.freebsd.org/D4988

Modified:
  head/sys/kern/vfs_aio.c

Modified: head/sys/kern/vfs_aio.c
==============================================================================
--- head/sys/kern/vfs_aio.c	Tue Jan 19 19:04:56 2016	(r294338)
+++ head/sys/kern/vfs_aio.c	Tue Jan 19 20:46:30 2016	(r294339)
@@ -1079,9 +1079,6 @@ aio_daemon(void *_id)
 	aiop->aiothread = td;
 	aiop->aiothreadflags = 0;
 
-	/* The daemon resides in its own pgrp. */
-	sys_setsid(td, NULL);
-
 	/*
 	 * Wakeup parent process.  (Parent sleeps to keep from blasting away
 	 * and creating too many daemons.)



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201601192046.u0JKkUNv028765>