From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NWbA1-0003I3-IT for qemu-devel@nongnu.org; Sun, 17 Jan 2010 14:51:45 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1NWb9x-0003HW-1k for qemu-devel@nongnu.org; Sun, 17 Jan 2010 14:51:45 -0500 Received: from [199.232.76.173] (port=46740 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NWb9w-0003HQ-Rv for qemu-devel@nongnu.org; Sun, 17 Jan 2010 14:51:40 -0500 Received: from mail-pw0-f43.google.com ([209.85.160.43]:59809) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NWb9v-0003Ye-Ut for qemu-devel@nongnu.org; Sun, 17 Jan 2010 14:51:40 -0500 Received: by pwj11 with SMTP id 11so1474376pwj.2 for ; Sun, 17 Jan 2010 11:51:35 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <1263595050-17791-1-git-send-email-atar4qemu@google.com> From: Blue Swirl Date: Sun, 17 Jan 2010 19:51:15 +0000 Message-ID: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] Re: sparc32 do not clear interrupts when masking List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Artyom Tarasenko Cc: qemu-devel@nongnu.org On Sun, Jan 17, 2010 at 6:42 PM, Artyom Tarasenko wrote: > 2010/1/16 Blue Swirl : >> On Sat, Jan 16, 2010 at 8:47 AM, Blue Swirl wrote= : >>> On Sat, Jan 16, 2010 at 8:16 AM, Blue Swirl wrot= e: >>>> On Fri, Jan 15, 2010 at 10:37 PM, Artyom Tarasenko >>>> wrote: >>>>> Don't clear interrupts on disabling, because >>>>> * Sun4M_SystemArchitecture_edited2.pdf doesn't describe >>>>> =C2=A0that masking or un-masking IRQ shall clear pending ones. >>>>> >>>>> * Field tests also show that SPARCstation-20 doesn't >>>>> =C2=A0clear them. >>>> >>>> Awesome work! >>>> >>>>> * The patch makes Solaris 2.5.1/2.6 boot ~1500 times >>>>> =C2=A0faster (~20 seconds instead of ~8 hours) >>>> >>>> Unfortunately there is some problem with the patch (or more likely >>>> there is some other bug that this uncovers), because all my Linux test >>>> now panic. NetBSD and OpenBSD are unaffected. >>>> >>>> For example, sparc-test: >>>> eth0: LANCE 52:54:00:12:34:56 >>>> esp0: no command in esp_handle() >>>> Kernel panic - not syncing: esp_handle: current_SC =3D=3D penguin with= in interrupt! >>>> =C2=A0<0>Press L1-A to return to the boot prom >>>> >>>> The bug may be in ESP interrupt handling (or in the interrupt chain >>>> between ESP, DMA controller and interrupt controller) as the panic >>>> always happens in ESP probe. >>> >>> In fact the bug was that OpenBIOS didn't clear ESP interrupts after >>> issuing commands, so the IRQ line was left raised which confused >>> Linux. >>> >> >> Thanks, applied. I fixed the OpenBIOS bug and updated images in pc-bios. > > Looks like this version of the OpenBIOS has a sort of a regression: it > doesn't recognize > NetBSD 1.3.3 miniroot anymore: > > qemu-system-sparc -nographic =C2=A0-hda ../images/miniroot-133.fs -m 64 -= M SS-20 > > Welcome to OpenBIOS v1.0 built on Jan 16 2010 08:54 > =C2=A0Type 'help' for detailed information > > [sparc] Booting file 'disk' with parameters '' > Trying disk (disk) > Trying disk:d (disk:d) > Unsupported image format > > I say "sort of" because it may be the correct behavior: due to the bad > disk labels OBP doesn't boot miniroots either. > > On the other hand it doesn't recognize Solaris cdroms too: > > 0 > boot cdrom > [sparc] Booting file 'cdrom' with parameters '' > Trying cdrom (cdrom) > Trying cdrom:d (cdrom:d) > Unsupported image format > =C2=A0ok Yes, that's what I meant with committing the wrong fix, 694b9309462f07307d16f492961f01271f10c245 doesn't boot anything. That will cause bisection problems, sorry. OpenBIOS r666 should work as before. I'm considering whether to update the ROMs, but it seems lame to update them so soon.