From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NVMQZ-0003E0-Ec for qemu-devel@nongnu.org; Thu, 14 Jan 2010 04:55:43 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1NVMQU-0003Bf-02 for qemu-devel@nongnu.org; Thu, 14 Jan 2010 04:55:42 -0500 Received: from [199.232.76.173] (port=43023 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NVMQT-0003Bb-SW for qemu-devel@nongnu.org; Thu, 14 Jan 2010 04:55:37 -0500 Received: from mail-bw0-f212.google.com ([209.85.218.212]:64430) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NVMQT-0003IF-9m for qemu-devel@nongnu.org; Thu, 14 Jan 2010 04:55:37 -0500 Received: by bwz4 with SMTP id 4so16390923bwz.2 for ; Thu, 14 Jan 2010 01:55:35 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <20091218220522.GM24729@hall.aurel32.net> References: <20091218213237.GA13509@debbook.brain-dump.org> <20091218220522.GM24729@hall.aurel32.net> Date: Thu, 14 Jan 2010 10:55:34 +0100 Message-ID: <5b31733c1001140155l66943824g4bfeeaa6b7164398@mail.gmail.com> Subject: Re: [Qemu-devel] [PATCH] target-arm: ARMv4 emulation From: Filip Navara Content-Type: multipart/alternative; boundary=0015175df034648d34047d1ce3d6 List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aurelien Jarno Cc: vince@kyllikki.org, qemu-devel@nongnu.org, paul@codesourcery.com --0015175df034648d34047d1ce3d6 Content-Type: text/plain; charset=ISO-8859-1 On Fri, Dec 18, 2009 at 11:05 PM, Aurelien Jarno wrote: > On Fri, Dec 18, 2009 at 10:32:37PM +0100, Marc Andre Tanner wrote: > > On Tue, Dec 01, 2009 at 09:41:43PM +0100, Filip Navara wrote: > > > While most of the ARMv5 instructions are backward compatible with > ARMv4, there > > > are few important differences. Most notably the stack pop and load > instructions > > > ignore the lowest bit, which is used by ARMv5 to switch to Thumb mode. > A > > > base-updated data-abort model is used on ARM7TDMI, CP15 coprocessor is > not > > > present and several instructions of later architectures are not > implemented. > > > > > > This patch introduces flags for the V5, CP15 and ABORT_BU (base-updated > abort > > > model) features. When V5 feature is not set the bit 0 on POP, LD and > LDM of PC > > > register is ignored and doesn't swith to/from Thumb mode and several > > > instructions are treated as unimplemented (BLX, PLD, BKPT, LDRD, STRD). > > > > > > Added are processor definitions for ARM7TDMI and ARM920T. > > > > > > Based on patches by Ulrich Hecht and Vincent Sanders < > vince@kyllikki.org>. > > > > Could someone please review + apply this patch, bonus points if it ends > > up in 0.12. > > > > ARM stuff is usually reviewed by Paul Brook. Paul, do you plan to review > it, or should I do it? > > I would appreciate at least some progress on this patch. A "no, because..." would be fine with me too. Best regards, Filip Navara --0015175df034648d34047d1ce3d6 Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable
On Fri, Dec 18, 2009 at 11:05 PM, Aurelien Jarno= <aurelien@aur= el32.net> wrote:
On Fri, Dec 18, 2009 at 10:32:37PM +0100,= Marc Andre Tanner wrote:
> On Tue, Dec 01, 2009 at 09:41:43PM +0100, Filip Navara wrote:
> > While most of the ARMv5 instructions are backward compatible with= ARMv4, there
> > are few important differences. Most notably the stack pop and loa= d instructions
> > ignore the lowest bit, which is used by ARMv5 to switch to Thumb = mode. A
> > base-updated data-abort model is used on ARM7TDMI, CP15 coprocess= or is not
> > present and several instructions of later architectures are not i= mplemented.
> >
> > This patch introduces flags for the V5, CP15 and ABORT_BU (base-u= pdated abort
> > model) features. When V5 feature is not set the bit 0 on POP, LD = and LDM of PC
> > register is ignored and doesn't swith to/from Thumb mode and = several
> > instructions are treated as unimplemented (BLX, PLD, BKPT, LDRD, = STRD).
> >
> > Added are processor definitions for ARM7TDMI and ARM920T.
> >
> > Based on patches by Ulrich Hecht <uli@suse.de> and Vincent Sanders <vince@kyllikki.org>.
>
> Could someone please review + apply this patch, bonus points if it end= s
> up in 0.12.
>

ARM stuff is usually reviewed by Paul Brook. Paul, do you plan = to review
it, or should I do it?

I would appreciate at least some progress on= this patch. A "no, because..." would be fine with me too.

Best regards,
Filip Navara
--0015175df034648d34047d1ce3d6--