From owner-freebsd-current@FreeBSD.ORG Mon Jan 23 14:01:23 2012 Return-Path: Delivered-To: current@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 567E61065676; Mon, 23 Jan 2012 14:01:23 +0000 (UTC) (envelope-from avg@FreeBSD.org) Received: from citadel.icyb.net.ua (citadel.icyb.net.ua [212.40.38.140]) by mx1.freebsd.org (Postfix) with ESMTP id 7143D8FC1C; Mon, 23 Jan 2012 14:01:22 +0000 (UTC) Received: from odyssey.starpoint.kiev.ua (alpha-e.starpoint.kiev.ua [212.40.38.101]) by citadel.icyb.net.ua (8.8.8p3/ICyb-2.3exp) with ESMTP id QAA05013; Mon, 23 Jan 2012 16:01:20 +0200 (EET) (envelope-from avg@FreeBSD.org) Message-ID: <4F1D6830.60602@FreeBSD.org> Date: Mon, 23 Jan 2012 16:01:20 +0200 From: Andriy Gapon User-Agent: Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120111 Thunderbird/9.0 MIME-Version: 1.0 To: Gleb Smirnoff References: <20120117110242.GD12760@glebius.int.ru> <20120120154158.GD16676@FreeBSD.org> <4F1ABFF3.9090305@FreeBSD.org> <20120122163539.GF16676@glebius.int.ru> <4F1D18A5.8010006@FreeBSD.org> <20120123130743.GI16676@glebius.int.ru> In-Reply-To: <20120123130743.GI16676@glebius.int.ru> X-Enigmail-Version: undefined Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: current@FreeBSD.org Subject: Re: new panic in cpu_reset() with WITNESS X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 23 Jan 2012 14:01:23 -0000 on 23/01/2012 15:07 Gleb Smirnoff said the following: > On Mon, Jan 23, 2012 at 10:21:57AM +0200, Andriy Gapon wrote: > A> > I can confirm that I can reboot succesfully with a new kernel > A> > and kern.stop_scheduler_on_panic=0. > A> > A> This is very puzzling. You observation means that stop_scheduler_on_panic has > A> an effect on the code outside the panic path. I reviewed the > A> SCHEDULER_STOPPED() changes and I still can not see how that could be possible > A> (modulo compiler bugs). > A> > A> Unfortunately, I also can not reproduce the problem here - without > A> WITNESS_SKIPSPIN I am running into a panic[*] during boot. > > I suppose your patch (the one in other post) should help against > this panic on boot. Yes, it does. It also masks/fixes the panic at reboot. > A> BTW, do you get any other LOR reports _before_ you do a reboot? > > I have some, but I don't think they are related. OK. > A> Can you try to change printfs in witness to db_printfs? Perhaps this will allow > A> to get the details of the LOR in uart_cnputc. Maybe that will reveal some > A> important additional details. > > Should I do s/printf/db_printf/ throughout entire subr_witness.c, or only > in special places? I think that "replace all" should work for this test. > A> Finally, can you try the patch from my other post? > > Yes, it works. > Thank you again. -- Andriy Gapon