From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1M8t8R-0005rw-W7 for qemu-devel@nongnu.org; Tue, 26 May 2009 05:39:52 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1M8t8P-0005rN-SW for qemu-devel@nongnu.org; Tue, 26 May 2009 05:39:51 -0400 Received: from [199.232.76.173] (port=36972 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1M8t8P-0005rH-L2 for qemu-devel@nongnu.org; Tue, 26 May 2009 05:39:49 -0400 Received: from flounder.pepperfish.net ([87.237.62.181]:53601) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1M8t8P-0003FZ-B6 for qemu-devel@nongnu.org; Tue, 26 May 2009 05:39:49 -0400 Date: Tue, 26 May 2009 10:39:44 +0100 From: Vincent Sanders Subject: Re: [Qemu-devel] [PATCH 1/16] ARM Add ARM 920T identifiers Message-ID: <20090526093944.GB4784@derik> References: <20090423171503.GC4629@derik> <200904301708.01385.paul@codesourcery.com> <20090523165033.GC8037@derik> <200905241931.19368.paul@codesourcery.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200905241931.19368.paul@codesourcery.com> List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Paul Brook On Sun, May 24, 2009 at 07:31:19PM +0100, Paul Brook wrote: > On Saturday 23 May 2009, Vincent Sanders wrote: > > On Thu, Apr 30, 2009 at 05:08:01PM +0100, Paul Brook wrote: > > > > env->cp15.c0_cpuid = id; > > > > switch (id) { > > > > + case ARM_CPUID_ARM920T: > > > > + break; > > > > > > This is woefully incomplete. At minimum it needs a comment describing how > > > this is a lie. > > > > I have worked out what you meant was that the copro 15 registers > > needed configuring appropriately in this case. > > More fundamental than that. We emulate armv5te. The arm920t is armv4t. > i.e. even usermode code may not work right. > Thankyou, right, now I understand. Perhaps if you had been a little more explicit in your first reply I would have been able to grasp what you wanted sooner. I will post the updated patch to the list shortly. Please be assured that every program we have tried (boot loaders, Linux, BSD etc.) appears to function as expected, to the point it had not been apparient the 920 was a different ISA version to the default emulation. So while this is indeed a technical issue, in practice it doesnt seem to be a major problem. -- Regards Vincent