From owner-svn-src-all@FreeBSD.ORG Wed Nov 2 13:43:17 2011 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 649291065670; Wed, 2 Nov 2011 13:43:17 +0000 (UTC) (envelope-from pawel@dawidek.net) Received: from mail.dawidek.net (60.wheelsystems.com [83.12.187.60]) by mx1.freebsd.org (Postfix) with ESMTP id 040FD8FC0C; Wed, 2 Nov 2011 13:43:17 +0000 (UTC) Received: from localhost (58.wheelsystems.com [83.12.187.58]) by mail.dawidek.net (Postfix) with ESMTPSA id AED71F04; Wed, 2 Nov 2011 14:43:14 +0100 (CET) Date: Wed, 2 Nov 2011 14:42:26 +0100 From: Pawel Jakub Dawidek To: Alexander Motin Message-ID: <20111102134226.GA1656@garage.freebsd.pl> References: <201111020924.pA29OxUV009135@svn.freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="1yeeQ81UyVL57Vl7" Content-Disposition: inline In-Reply-To: <201111020924.pA29OxUV009135@svn.freebsd.org> X-OS: FreeBSD 9.0-CURRENT amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r227015 - head/sys/geom X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Nov 2011 13:43:17 -0000 --1yeeQ81UyVL57Vl7 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Nov 02, 2011 at 09:24:59AM +0000, Alexander Motin wrote: > Author: mav > Date: Wed Nov 2 09:24:59 2011 > New Revision: 227015 > URL: http://svn.freebsd.org/changeset/base/227015 >=20 > Log: > Add mutex and two flags to make orphan() call properly asynchronous: > - delay consumer closing and detaching on orphan() until all I/Os comp= lete; > - prevent new I/Os submission after orphan() called. > Previous implementation could destroy consumers still having active > requests and worked only because of global workaround made on GEOM leve= l. Alexander, I'm not sure I agree with your recent changes to address this. The checks in GEOM were there to avoid the need for counting outstanding I/O requests in every single GEOM class. Why do you think the checks in GEOM are not good enough? Can we design solution that can be implemented in the framework itself, so simple GEOM classes can stay simple? --=20 Pawel Jakub Dawidek http://www.wheelsystems.com FreeBSD committer http://www.FreeBSD.org Am I Evil? Yes, I Am! http://yomoli.com --1yeeQ81UyVL57Vl7 Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.14 (FreeBSD) iEYEARECAAYFAk6xSMIACgkQForvXbEpPzR76ACfcnD2VIq+gZP/9wz8VrlaZrwE McUAnj2DgGFLNV5j6g068uyWkM0C3qs8 =eTX9 -----END PGP SIGNATURE----- --1yeeQ81UyVL57Vl7--