Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 9 Mar 2006 13:57:21 -0500
From:      Brian Fundakowski Feldman <green@FreeBSD.org>
To:        Pawel Jakub Dawidek <pjd@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/geom/raid3 g_raid3.c
Message-ID:  <20060309185721.GE7835@green.homeunix.org>
In-Reply-To: <20060307133200.GA60850@garage.freebsd.pl>
References:  <200602221021.k1MAL5qF031937@repoman.freebsd.org> <20060307003911.GB7835@green.homeunix.org> <20060307133200.GA60850@garage.freebsd.pl>

next in thread | previous in thread | raw e-mail | index | archive | help
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 wrote:
> +> On Wed, Feb 22, 2006 at 10:21:05AM +0000, Pawel Jakub Dawidek wrote:
> +> > pjd         2006-02-22 10:21:05 UTC
> +> > 
> +> >   FreeBSD src repository
> +> > 
> +> >   Modified files:
> +> >     sys/geom/raid3       g_raid3.c 
> +> >   Log:
> +> >   Do not use bio structure after g_io_deliver(), it may not longer by valid.
> +> >   
> +> >   Found and fixed by:     Vsevolod Lobko <seva@ip.net.ua>
> +> >   MFC after:              3 days
> +> 
> +> 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?

-- 
Brian Fundakowski Feldman                           \'[ FreeBSD ]''''''''''\
  <> green@FreeBSD.org                               \  The Power to Serve! \
 Opinions expressed are my own.                       \,,,,,,,,,,,,,,,,,,,,,,\



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