Re: Bug in latest rev kern_acct.c: panic: Trying sleep, but thread marked as sleeping prohibited

From: Wojciech A. Koszek <dunstan_at_zsno.ids.czest.pl>
Date: Mon, 23 Jan 2006 00:39:06 +0100
On Sun, Jan 01, 2006 at 06:22:27PM +0000, Wojciech A. Koszek wrote:
> On Sat, Dec 31, 2005 at 06:39:31PM -0600, Richard Todd wrote:
> > "Wojciech A. Koszek" <dunstan_at_freebsd.czest.pl> writes:
> > 
> > > On Wed, Dec 21, 2005 at 11:11:43AM -0500, John Baldwin wrote:
> > >> acctwatch() probably isn't a good thing to do from a callout since it wants to 
> > >> do VOPs and such.  Probably the easiest fix is to stick acctwatch() in its 
> > >> own kthread.
> > >
> > > I think I didn't break current behaviour. Patch is here:
> > >
> > > http://freebsd.czest.pl/dunstan/FreeBSD/kern_acct.0.patch
> > >
> > > Richard, could you try to repeat your problem with my patch?
> > 
> > Got a chance to try a new kernel today with your patch. I tried to reproduce 
> > the panic by repeatedly copying a bunch of stuff onto /usr and deleting it
> > in one window, and in another repeatedly doing "make configure; make clean"
> > in one rather large port.  Lots of "Accounting suspended/resumed" messages
> > in the logs and no panics.  So it's looking good from here. 
> 
> Thanks for testing this. I couldn't crash my box either. I belive that
> KASSERT() before suspend/resume handling will always be true, just like
> ACCT_ENABLED state. In the final version of the patch I'll include some
> cosmetic changes suggested by Robert Watson (thanks!).

(Sorry for late response. I'm having terrible problems with DNS server
for freebsd.czest.pl. I'll try to get them fixed this week).

Updated patch for accouting is here:

	http://zsno.ids.czest.pl/~dunstan/FreeBSD/kern_acct.2.patch

-- 
* Wojciech A. Koszek && <dunstan%zsno.ids.czest.pl>
Received on Sun Jan 22 2006 - 20:59:20 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:51 UTC