From: Paul Mundt <lethal@linux-sh.org>
To: qemu-devel@nongnu.org
Cc: Vladimir Prus <vladimir@codesourcery.com>
Subject: Re: [Qemu-devel] [PATCH] SH: Add prefi, icbi, synco
Date: Mon, 20 Oct 2008 13:01:24 +0900 [thread overview]
Message-ID: <20081020040124.GA19601@linux-sh.org> (raw)
In-Reply-To: <200810172056.53076.paul@codesourcery.com>
On Fri, Oct 17, 2008 at 08:56:52PM +0100, Paul Brook wrote:
> On Friday 17 October 2008, Vladimir Prus wrote:
> > This patch makes qemu recognize (and ignore), three instructions from SH4A.
>
> Shouldn't these generate an illegal instruction exception on SH4 cpus?
>
Note that all of these are SH-5 instructions, which gradually found their
way in to the SH-4A ISA. There are also other SH-5 instructions that have
found their way in to the SH-2A ISA which are not as of yet reflected in
SH-4A. The dependency tracking gets to be pretty ugly, as is evident in
binutils. In order to get this right, it would be necessary to throw ISA
versioning in to the CPU definition and test to figure out what to
support, as per the table in binutils. While this would be a good idea in
general, it is something that should be done as a larger follow-up to
these patches.
next prev parent reply other threads:[~2008-10-20 4:01 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-10-17 12:52 [Qemu-devel] [PATCH] SH: Add prefi, icbi, synco Vladimir Prus
2008-10-17 19:56 ` Paul Brook
2008-10-20 4:01 ` Paul Mundt [this message]
2008-10-20 10:54 ` Paul Brook
2008-10-20 13:00 ` Paul Mundt
2008-10-20 16:27 ` Paul Brook
2008-10-20 16:31 ` Vladimir Prus
2008-10-20 16:46 ` Paul Brook
2008-10-20 16:49 ` Paul Mundt
2008-10-20 17:07 ` Paul Brook
2008-10-20 17:15 ` Laurent Desnogues
2008-10-20 17:19 ` Paul Brook
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20081020040124.GA19601@linux-sh.org \
--to=lethal@linux-sh.org \
--cc=qemu-devel@nongnu.org \
--cc=vladimir@codesourcery.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).