Date: Mon, 18 Jul 2011 14:42:04 +0300 From: Andriy Gapon <avg@FreeBSD.org> To: freebsd-current@FreeBSD.org Subject: Re: Fwd: stop scheduler in panic context Message-ID: <4E241C0C.1000604@FreeBSD.org> In-Reply-To: <4E05F2A4.6060908@FreeBSD.org> References: <4E05F2A4.6060908@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
This is a reminder that if you would like to see the code for sane panic(9) context in 9.0, then I still need at least one independent reviewer and tester for the code. Thank you. on 25/06/2011 17:37 Andriy Gapon said the following: > > I would like to ask for testing of the following patch. > Since the patch affects panic(9) context, then obviously its testing requires > getting some sort of panic, and preferably some sort of "post-panic" activity too: > doing something in kdb, dumping a core (via debugger command or in unattended > mode), resetting a machine finally. > > At minimum I hope that no regressions are introduced. > At maximum I hope that some things are improved like, e.g., crash dump succeeding > where it failed before (PR amd64/139614). > > The patch is for recent head/CURRENT. It mostly affects SMP systems, but also has > a smaller impact on UP systems. > I can try to adapt it to stable/8, if sufficient interest arises. > Please see the following message for the patch and some more details. > > -------- Original Message -------- > > I would like to present the following diff for review and discussion: > http://people.freebsd.org/~avg/stop_scheduler_on_panic.diff > > The idea is to stop scheduler in a panic context and to provide a special > environment for the only running thread, the one that called panic(9). > > I tried to make this diff as minimal as possible, it doesn't include changes that > I consider to be useful improvements and [even] bug fixes, but which generated > controversy in non-public discussions. > > If there is no negative feedback within next few days, then I plan to post the > patch to current@ to solicit some testing. I will definitely wait for positive > feedback before committing this change. I hope that I will be able to sneak it > into the 9 release (unless there are objections to this). > > Thank you! -- Andriy Gapon
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4E241C0C.1000604>