From owner-svn-src-head@FreeBSD.ORG  Mon Jan  7 21:35:26 2013
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from mx1.freebsd.org (mx1.FreeBSD.org [8.8.178.115])
 by hub.freebsd.org (Postfix) with ESMTP id 989FC646;
 Mon,  7 Jan 2013 21:35:26 +0000 (UTC)
 (envelope-from jimharris@FreeBSD.org)
Received: from svn.freebsd.org (svn.freebsd.org
 [IPv6:2001:1900:2254:2068::e6a:0])
 by mx1.freebsd.org (Postfix) with ESMTP id 8950A843;
 Mon,  7 Jan 2013 21:35:26 +0000 (UTC)
Received: from svn.freebsd.org (svn.FreeBSD.org [8.8.178.70])
 by svn.freebsd.org (8.14.5/8.14.5) with ESMTP id r07LZQNt092662;
 Mon, 7 Jan 2013 21:35:26 GMT
 (envelope-from jimharris@svn.freebsd.org)
Received: (from jimharris@localhost)
 by svn.freebsd.org (8.14.5/8.14.5/Submit) id r07LZQbc092661;
 Mon, 7 Jan 2013 21:35:26 GMT
 (envelope-from jimharris@svn.freebsd.org)
Message-Id: <201301072135.r07LZQbc092661@svn.freebsd.org>
From: Jim Harris <jimharris@FreeBSD.org>
Date: Mon, 7 Jan 2013 21:35:26 +0000 (UTC)
To: src-committers@freebsd.org, svn-src-all@freebsd.org,
 svn-src-head@freebsd.org
Subject: svn commit: r245136 - head/sys/dev/nvme
X-SVN-Group: head
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
X-BeenThere: svn-src-head@freebsd.org
X-Mailman-Version: 2.1.14
Precedence: list
List-Id: SVN commit messages for the src tree for head/-current
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Mon, 07 Jan 2013 21:35:26 -0000

Author: jimharris
Date: Mon Jan  7 21:35:25 2013
New Revision: 245136
URL: http://svnweb.freebsd.org/changeset/base/245136

Log:
  Revert r244549.
  
  This change was originally intended to account for test kthreads under
  the nvmecontrol process, but jhb indicated it may not be safe to
  associate kthreads with userland processes and this could have
  unintended consequences.
  
  I did not observe any problems with this change, but my testing didn't
  exhaust the kinds of corner cases that could cause problems.  It is not
  that important to account for these test threads under nvmecontrol, so I
  am just reverting this change for now.
  
  On a related note, the part of this patch for <= 7.x fails compilation
  so reverting this fixes that too.
  
  Suggested by: jhb

Modified:
  head/sys/dev/nvme/nvme_test.c

Modified: head/sys/dev/nvme/nvme_test.c
==============================================================================
--- head/sys/dev/nvme/nvme_test.c	Mon Jan  7 20:36:51 2013	(r245135)
+++ head/sys/dev/nvme/nvme_test.c	Mon Jan  7 21:35:25 2013	(r245136)
@@ -287,10 +287,10 @@ nvme_ns_test(struct nvme_namespace *ns, 
 	for (i = 0; i < io_test->num_threads; i++)
 #if __FreeBSD_version >= 800004
 		kthread_add(fn, io_test_internal,
-		    curproc, NULL, 0, 0, "nvme_io_test[%d]", i);
+		    NULL, NULL, 0, 0, "nvme_io_test[%d]", i);
 #else
 		kthread_create(fn, io_test_internal,
-		    curproc, 0, 0, "nvme_io_test[%d]", i);
+		    NULL, 0, 0, "nvme_io_test[%d]", i);
 #endif
 
 	tsleep(io_test_internal, 0, "nvme_test", io_test->time * 2 * hz);