From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Ghof3-0000L0-Ja for qemu-devel@nongnu.org; Wed, 08 Nov 2006 09:44:17 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Ghof0-0000JO-Jz for qemu-devel@nongnu.org; Wed, 08 Nov 2006 09:44:17 -0500 Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Ghof0-0000JF-Gn for qemu-devel@nongnu.org; Wed, 08 Nov 2006 09:44:14 -0500 Received: from [66.92.75.137] (helo=kline-station.thibault.org) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA:32) (Exim 4.52) id 1Ghof0-0007so-Ej for qemu-devel@nongnu.org; Wed, 08 Nov 2006 09:44:14 -0500 Received: from kline-station.local.thibault.org (kline-station.local.thibault.org [127.0.0.1]) by kline-station.thibault.org (8.12.8/8.12.8) with ESMTP id kA8DgDfQ019376 for ; Wed, 8 Nov 2006 08:42:13 -0500 Subject: Re: [Qemu-devel] Thanks for QEMU! Content-Type: text/plain Date: Wed, 08 Nov 2006 09:44:08 -0500 Message-Id: <1162997048.4168.10.camel@client-30.office.endeca.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit From: John Stracke Reply-To: qemu-devel@nongnu.org List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Hetz Ben Hamo wrote: > Perhaps you could submit your printf's so other people can learn from > them please? Well, most of them I no longer have; they slowed things down, so I deleted them once I had the problems fixed. Besides, they were probably too idiosyncratic to help other people much. However, there is one change that I'd recommend adding into the QEMU distribution: in target-i386/helper2.c, in cpu_x86_handle_mmu_fault(), right before the line starting "pde_addr = ((env->cr[3] & ~0xfff)", I added a sanity check: if (env->cr[3] & 0xfff) { fprintf(logfile,"Warning: page directory is not page-aligned.\n"); } This gives a warning if the OS is using an invalid address for the page directory. You might want to put it inside #ifdef DEBUG_MMU, or something. -- /============================================================\ |John Stracke | http://www.thibault.org |HTML OK | |Francois Thibault |=======================================| |East Kingdom |Rope is rope, and string is string, and| |francis@thibault.org|never the twine shall meet. | \============================================================/