Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 6 Apr 2000 16:16:04 +0930
From:      Greg Lehey <grog@lemis.com>
To:        Soren Schmidt <sos@freebsd.dk>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/dev/vinum vinuminterrupt.c
Message-ID:  <20000406161604.J73669@freebie.lemis.com>
In-Reply-To: <200004060641.IAA90179@freebsd.dk>
References:  <200004060303.UAA06576@freefall.freebsd.org> <200004060641.IAA90179@freebsd.dk>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thursday,  6 April 2000 at  8:41:49 +0200, Søren Schmidt wrote:
> It seems Greg Lehey wrote:
>> grog        2000/04/05 20:03:32 PDT
>>
>>   Modified files:
>>     sys/dev/vinum        vinuminterrupt.c
>>   Log:
>>   complete_rqe: Remove a race condition in RAID-4 and RAID-5 where a
>>   	request could be deallocated before the top half had finished
>>   	issuing it.  The problem seems only to happen with IDE drives
>>   	and vn devices, but theoretically it could happen with any
>>   	drive.  This is the most important part of a possible series
>>   	of fixes designed to remove race conditions without locking
>>   	out interrupts for longer than absolutely necessary.
>>
>>   Reported-by:	       sos
>>   Fix-supplied-by:       dillon
>
>
> Cool!, I'll throw my server at it as soon as I get home today...

Good.  I'm not 100% sure that this fix alone will work, but I don't
want to overdo things, because it can have an adverse effect on
performance.  I'll do some tests on vnode devices here; that should
have the same effect.

Greg
--
Finger grog@lemis.com for PGP public key
See complete headers for address and phone numbers


To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




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