From owner-cvs-src-old@FreeBSD.ORG  Sun May 16 11:56:51 2010
Return-Path: <owner-cvs-src-old@FreeBSD.ORG>
Delivered-To: cvs-src-old@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id D1924106564A
	for <cvs-src-old@freebsd.org>; Sun, 16 May 2010 11:56:51 +0000 (UTC)
	(envelope-from pjd@FreeBSD.org)
Received: from repoman.freebsd.org (unknown [IPv6:2001:4f8:fff6::29])
	by mx1.freebsd.org (Postfix) with ESMTP id BF80A8FC18
	for <cvs-src-old@freebsd.org>; Sun, 16 May 2010 11:56:51 +0000 (UTC)
Received: from repoman.freebsd.org (localhost [127.0.0.1])
	by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id o4GBup9N008078
	for <cvs-src-old@freebsd.org>; Sun, 16 May 2010 11:56:51 GMT
	(envelope-from pjd@repoman.freebsd.org)
Received: (from svn2cvs@localhost)
	by repoman.freebsd.org (8.14.3/8.14.3/Submit) id o4GBupmi008077
	for cvs-src-old@freebsd.org; Sun, 16 May 2010 11:56:51 GMT
	(envelope-from pjd@repoman.freebsd.org)
Message-Id: <201005161156.o4GBupmi008077@repoman.freebsd.org>
X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to
	pjd@repoman.freebsd.org using -f
From: Pawel Jakub Dawidek <pjd@FreeBSD.org>
Date: Sun, 16 May 2010 11:56:42 +0000 (UTC)
To: cvs-src-old@freebsd.org
X-FreeBSD-CVS-Branch: HEAD
Subject: cvs commit: src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs
	vdev_geom.c
X-BeenThere: cvs-src-old@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: **OBSOLETE** CVS commit messages for the src tree
	<cvs-src-old.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src-old>,
	<mailto:cvs-src-old-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/cvs-src-old>
List-Post: <mailto:cvs-src-old@freebsd.org>
List-Help: <mailto:cvs-src-old-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/cvs-src-old>,
	<mailto:cvs-src-old-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 16 May 2010 11:56:51 -0000

pjd         2010-05-16 11:56:42 UTC

  FreeBSD src repository

  Modified files:
    sys/cddl/contrib/opensolaris/uts/common/fs/zfs vdev_geom.c 
  Log:
  SVN rev 208142 on 2010-05-16 11:56:42Z by pjd
  
  The whole point of having dedicated worker thread for each leaf VDEV was to
  avoid calling zio_interrupt() from geom_up thread context. It turns out that
  when provider is forcibly removed from the system and we kill worker thread
  there can still be some ZIOs pending. To complete pending ZIOs when there is
  no worker thread anymore we still have to call zio_interrupt() from geom_up
  context. To avoid this race just remove use of worker threads altogether.
  This should be more or less fine, because I also thought that zio_interrupt()
  does more work, but it only makes small UMA allocation with M_WAITOK.
  It also saves one context switch per I/O request.
  
  PR:             kern/145339
  Reported by:    Alex Bakhtin <Alex.Bakhtin@gmail.com>
  MFC after:      1 week
  
  Revision  Changes    Path
  1.21      +21 -104   src/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_geom.c