Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 14 May 2003 13:37:59 +0200
From:      "Poul-Henning Kamp" <phk@phk.freebsd.dk>
To:        "Greg 'groggy' Lehey" <grog@FreeBSD.org>
Cc:        cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/geom geom_disk.c 
Message-ID:  <30440.1052912279@critter.freebsd.dk>
In-Reply-To: Your message of "Wed, 14 May 2003 17:40:41 %2B0930." <20030514081041.GD4390@wantadilla.lemis.com> 

next in thread | previous in thread | raw e-mail | index | archive | help
In message <20030514081041.GD4390@wantadilla.lemis.com>, "Greg 'groggy' Lehey" 
writes:

>malloc() of "16" with the following non-sleepablelocks held:
>exclusive sleep mutex g_xup r = 0 (0xc8612ca8) locked @ /src/FreeBSD/5-CURRENT-ZAPHOD/src/sys/geom/geom_io.c:363

This is a problem, you cannot sleep in the primary I/O path (ie: strategy()
and done()), if you sleep in the I/O path, you stop all I/O processing
and if the system needs to page something out to make the ram available
to you, you have a deadlock.

>panic: final g_dev_close() with outstanding bios

This means that close was called before all the I/O requests had been
delivered, this is not allowed.

You have to stop sending I/Os, wait for the ones you have sent to come
back, then you can close.

Otherwise the I/O's arriving after the close will be very likely to
access stuff which is no longer there, and result in really weird
random memory corruption.

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk@FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.



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