Date: Wed, 5 Sep 2001 09:19:45 +0930 From: Greg Lehey <grog@FreeBSD.org> To: John Baldwin <jhb@FreeBSD.org> Cc: Peter Wemm <peter@wemm.org>, cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/kern kern_shutdown.c Message-ID: <20010905091945.P85816@wantadilla.lemis.com> In-Reply-To: <XFMail.010904094046.jhb@FreeBSD.org>; from jhb@FreeBSD.org on Tue, Sep 04, 2001 at 09:40:46AM -0700 References: <20010904055510.050A4380C@overcee.netplex.com.au> <XFMail.010904094046.jhb@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tuesday, 4 September 2001 at 9:40:46 -0700, John Baldwin wrote: > > On 04-Sep-01 Peter Wemm wrote: >> John Baldwin wrote: >>> >>> On 02-Sep-01 Peter Wemm wrote: >>>> peter 2001/09/02 05:24:38 PDT >>>> >>>> Modified files: >>>> sys/kern kern_shutdown.c >>>> Log: >>>> The !RESTARTABLE_PANICS code has some loose ends. >>> >>> What is holding_giant for? Oh, and you do know that you are truly evil >>> incarnate now, right? >> >> I'm not sure. Matt added it in rev 1.107, but the code that sets it is >> #if 0'ed out. It is a real mess. > > Except the code to release it isn't #if 0'd out. > >> The other part of the diff was an accident that leaked from a local change. >> I've been meaning to back it out but just couldn't quite do it. Ian Dowse >> flagged it yesterday evening I think. I am amused at how few people had >> noticed it, and of those most had read the diff. > > Heh. We could leave it on in -current for all I care, except that it does > make all the useful messages scroll off the screen during a panic. :) > > \\|/ ____ \\|/ > \"@'/ .. \\`@\ > /_| \\__/ |_\\ > \\__U_/ > > Oh, and you forgot the > > Obtained from: Linux > > in the commit log. :-P Isn't this GPL'd code? Greg -- 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?20010905091945.P85816>