From owner-cvs-src@FreeBSD.ORG Tue Nov 18 13:20:34 2003 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id CC4F516A4CE; Tue, 18 Nov 2003 13:20:33 -0800 (PST) Received: from newtrinity.zeist.de (newtrinity.zeist.de [217.24.217.8]) by mx1.FreeBSD.org (Postfix) with ESMTP id 09C3943FDD; Tue, 18 Nov 2003 13:20:29 -0800 (PST) (envelope-from marius@newtrinity.zeist.de) Received: from newtrinity.zeist.de (localhost [127.0.0.1]) hAILKRK0007409; Tue, 18 Nov 2003 22:20:27 +0100 (CET) (envelope-from marius@newtrinity.zeist.de) Received: (from marius@localhost) by newtrinity.zeist.de (8.12.10/8.12.10/Submit) id hAILKMsO007408; Tue, 18 Nov 2003 22:20:22 +0100 (CET) (envelope-from marius) Date: Tue, 18 Nov 2003 22:20:22 +0100 From: Marius Strobl To: John Baldwin Message-ID: <20031118222022.R10910@newtrinity.zeist.de> References: <20031118161335.P10910@newtrinity.zeist.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.2.5.1i In-Reply-To: ; from jhb@FreeBSD.org on Tue, Nov 18, 2003 at 03:54:49PM -0500 X-AntiVirus: checked by AntiVir Milter 1.0.6; AVE 6.22.0.1; VDF 6.22.0.39 cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/i386/isa atpic.c X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Nov 2003 21:20:34 -0000 On Tue, Nov 18, 2003 at 03:54:49PM -0500, John Baldwin wrote: > > On 18-Nov-2003 Marius Strobl wrote: > > On Tue, Nov 18, 2003 at 09:21:18AM -0500, John Baldwin wrote: > >> > >> On 18-Nov-2003 Marius Strobl wrote: > >> > On Mon, Nov 17, 2003 at 05:53:14PM -0500, John Baldwin wrote: > >> >> > >> >> On 17-Nov-2003 Marius Strobl wrote: > >> >> > On Fri, Nov 14, 2003 at 04:21:44PM -0500, John Baldwin wrote: > >> >> >> > >> >> >> On 14-Nov-2003 John Baldwin wrote: > >> >> >> > jhb 2003/11/14 13:02:49 PST > >> >> >> > > >> >> >> > FreeBSD src repository > >> >> >> > > >> >> >> > Modified files: > >> >> >> > sys/i386/isa atpic.c > >> >> >> > Log: > >> >> >> > Always install IDT entries for ATPIC interrupt sources. The APIC no > >> >> >> > longer uses these interrupt vectors for its ISA interrupt pins, so these > >> >> >> > entries will not be overwritten. If we get a spurious interrupt from the > >> >> >> > ATPIC when using the APIC, it will be treated as a stray interrupt instead > >> >> >> > of causing a panic. > >> >> >> > >> >> >> This should fix all of the "panic: Couldn't get vector from ISR" and > >> >> >> trap 30 (reserved) panics. If you were using older versions of either > >> >> >> spurious.patch or atpic.patch, please try reverting them and using > >> >> >> plain CVS to see if that fixes your problem. If not, please let me know. > >> >> >> If you get several stray IRQ 7 messages and would like to shut them up, > >> >> >> you can try http://www.FreeBSD.org/~jhb/patches/spurious.patch > >> >> >> > >> >> > > >> >> > Unfortunately it doesn't fix the type 30 trap triggered by a spurious > >> > > >> > Oops, meant stray interrupt... > >> > > >> >> > interrupt here (need atpic.patch). > >> >> > >> >> Do you get the trap very early? > >> >> > >> > > >> > Yes, please see <20031110172540.A29948@newtrinity.zeist.de> on current@ > >> > or http://quad.zeist.de/newintr.log (this was with older sources but the > >> > problem is still the same after the commit above). > >> > >> Can you try www.freebsd.org/~jhb/patches/vm86.patch > >> > > > > Looks worse: http://quad.zeist.de/newintr2.log > > Well, vm86 is broken but that patch seems to have helped fix that breakage. > There is another bug in the handling of stray interrupts. Can you revert > vm86.patch and try this instead: > > http://www.FreeBSD.org/~jhb/patches/atpic.patch > Works. > Assuming that works, you should be able to use > > http://www.FreeBSD.org/~jhb/patches/spurious.patch > > to quiet the 'stray irq 7' messages. > Also works. However, as you may have noticed in my boot logs I get 'stray irq 13' with the new interrupt code when npx gets probed. I'm aware that your spurious.patch only addresses stray irqs 7 and 15, I'm just wondering a bit why the co-processor mucks up with the new code...