From owner-cvs-src@FreeBSD.ORG Thu Mar 9 21:36:30 2006 Return-Path: X-Original-To: cvs-src@freebsd.org Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 98B1A16A420; Thu, 9 Mar 2006 21:36:30 +0000 (GMT) (envelope-from ru@ip.net.ua) Received: from tigra.ip.net.ua (tigra.ip.net.ua [82.193.96.10]) by mx1.FreeBSD.org (Postfix) with ESMTP id EDB9743D45; Thu, 9 Mar 2006 21:36:29 +0000 (GMT) (envelope-from ru@ip.net.ua) Received: from localhost (rocky.ip.net.ua [82.193.96.2]) by tigra.ip.net.ua (8.12.11/8.12.11) with ESMTP id k29LaSPn041393; Thu, 9 Mar 2006 23:36:28 +0200 (EET) (envelope-from ru@ip.net.ua) Received: from tigra.ip.net.ua ([82.193.96.10]) by localhost (rocky.ip.net.ua [82.193.96.2]) (amavisd-new, port 10024) with LMTP id 33337-05-3; Thu, 9 Mar 2006 23:35:59 +0200 (EET) Received: from heffalump.ip.net.ua (heffalump.ip.net.ua [82.193.96.213]) by tigra.ip.net.ua (8.12.11/8.12.11) with ESMTP id k29LZAm1041326 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 9 Mar 2006 23:35:10 +0200 (EET) (envelope-from ru@ip.net.ua) Received: (from ru@localhost) by heffalump.ip.net.ua (8.13.4/8.13.4) id k29LZPaJ084849; Thu, 9 Mar 2006 23:35:25 +0200 (EET) (envelope-from ru) Date: Thu, 9 Mar 2006 23:35:25 +0200 From: Ruslan Ermilov To: Scott Long Message-ID: <20060309213525.GG48078@ip.net.ua> References: <200602221021.k1MAL5qF031937@repoman.freebsd.org> <20060307003911.GB7835@green.homeunix.org> <20060307133200.GA60850@garage.freebsd.pl> <20060309185721.GE7835@green.homeunix.org> <20060309203818.GD48078@ip.net.ua> <44109657.2090405@samsco.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="RMedoP2+Pr6Rq0N2" Content-Disposition: inline In-Reply-To: <44109657.2090405@samsco.org> User-Agent: Mutt/1.5.11 X-Virus-Scanned: amavisd-new at ip.net.ua Cc: Brian Fundakowski Feldman , src-committers@freebsd.org, Pawel Jakub Dawidek , cvs-all@freebsd.org, cvs-src@freebsd.org Subject: Re: cvs commit: src/sys/geom/raid3 g_raid3.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 09 Mar 2006 21:36:30 -0000 --RMedoP2+Pr6Rq0N2 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Mar 09, 2006 at 01:55:51PM -0700, Scott Long wrote: > Ruslan Ermilov wrote: > >On Thu, Mar 09, 2006 at 01:57:21PM -0500, Brian Fundakowski Feldman wrot= e: > > > >>On Tue, Mar 07, 2006 at 02:32:00PM +0100, Pawel Jakub Dawidek wrote: > >> > >>>On Mon, Mar 06, 2006 at 07:39:11PM -0500, Brian Fundakowski Feldman=20 > >>>wrote: > >>>+> On Wed, Feb 22, 2006 at 10:21:05AM +0000, Pawel Jakub Dawidek wrote: > >>>+> > pjd 2006-02-22 10:21:05 UTC > >>>+> >=20 > >>>+> > FreeBSD src repository > >>>+> >=20 > >>>+> > Modified files: > >>>+> > sys/geom/raid3 g_raid3.c=20 > >>>+> > Log: > >>>+> > Do not use bio structure after g_io_deliver(), it may not longe= r=20 > >>>by valid. > >>>+> > =20 > >>>+> > Found and fixed by: Vsevolod Lobko > >>>+> > MFC after: 3 days > >>>+>=20 > >>>+> I actually found and fixed it half a year ago... could you please > >>>+> integrate the rest of the fixes from my changes back then? A > >>>+> short-term low-memory deadlock is still possible (observed in > >>>+> practice). I think the changes also improve readability -- see > >>>+> for example the reason r1.46 existed. > >>> > >>>Heh. I own you apology. I haven't had time to work on graid3 back then > >>>and I also overlooked fix of this very problem. > >>> > >>>I integrated you fixes to my last patch which I'm planning to commit > >>>after receiving some feedback: > >>> > >>> http://people.freebsd.org/~pjd/patches/graid3.patch > >> > >>Everything seems good to go, running -CURRENT's graid3 + these changes > >>on 6-STABLE; are there particular areas where you think I should try > >>testing for improved performance? > >> > > > >I was fighting a problem today when our geom_cache attempts to fill > >256K of bio_data coming from graid3, and this obviously fails because > >graid3 only allocates memory for first 64k from its 64k UMA zone. > >This patch doesn't fix this problem. I have MAXPHYS bumped to 512K, > >which is important to reproduce the problem. Can you please fix this? > >Temporarily, I've added 128/256/512 zones to satisfy my needs. > > > > > >Cheers, >=20 > MAXPHYS was never meant to go above 128K. You'll run a very real risk=20 > of exhausting KVA under load. And, all block drivers for the past 10=20 > years have been written with the assumption that MAXPHYS will never be > above 128K. In other words, don't shoot yourself in the foot and then > cry that it hurts =3D-) >=20 We've fixed some defines that abused MAXPHYS to mean 64k. I'm compiling the kernel with default MAXPHYS now, but still, there's no 128k zone in graid3 to allocate from. Cheers, --=20 Ruslan Ermilov ru@FreeBSD.org FreeBSD committer --RMedoP2+Pr6Rq0N2 Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (FreeBSD) iD8DBQFEEJ+dqRfpzJluFF4RAuYCAKCEsJHl6RQjzCPIH1eG7dReFEQOFwCgg9Tb rmsf92K0w6gnUsFsHyYiPLU= =+bJA -----END PGP SIGNATURE----- --RMedoP2+Pr6Rq0N2--