From: Richard Henderson <rth@twiddle.net>
To: Peter Crosthwaite <peter.crosthwaite@xilinx.com>
Cc: "Peter Maydell" <peter.maydell@linaro.org>,
"Peter Crosthwaite" <crosthwaite.peter@gmail.com>,
"Peter Crosthwaite" <crosthwaitepeter@gmail.com>,
"qemu-devel@nongnu.org Developers" <qemu-devel@nongnu.org>,
"Edgar Iglesias" <edgari@xilinx.com>,
"Andreas Färber" <afaerber@suse.de>
Subject: Re: [Qemu-devel] [PATCH] microblaze: Remove uses of TCGv and target_ulong
Date: Fri, 15 May 2015 10:17:32 -0700 [thread overview]
Message-ID: <55562A2C.9080005@twiddle.net> (raw)
In-Reply-To: <CAEgOgz7iTKs-3Q4JWRpJrqqn49JxrQka0CMs1bUk4-aD9+N-zA@mail.gmail.com>
On 05/15/2015 09:48 AM, Peter Crosthwaite wrote:
> On Fri, May 15, 2015 at 8:41 AM, Richard Henderson <rth@twiddle.net> wrote:
>> On 05/14/2015 09:49 PM, Peter Crosthwaite wrote:
>>> To prepare support for conversion of Microblaze TARGET_LONG to 64 bits.
>>> This in turn will then allow support for multi-arch QEMU including both
>>> Microblaze and 64-bit CPU targets (notably AArch64).
>>
>> I don't understand why multi-arch requires all of the arches
>> to have the same width. This seems like a major failure to me.
>>
>> I'm not particularly keen on this at all.
>>
>
> What is the alternative? What is the def of the global symbols TCGv
> and TARGET_LONG in the multi-arch cases?
Different for every file? Not relevant for "multi-arch" itself?
I dunno. Where does stuff break down first?
I would expect 80% of it to be private to target-foo, and /mostly/ encapsulated
in the tcg ops that are produced.
I realize there's a problem of how addresses are treated inside the tcg
backend, but that should be surmountable. Perhaps all we need are 4 new
opcodes so that both 32-bit and 64-bit addresses can be represented within the
opcode stream simultaneously.
I assume we're not still talking about multi-arch linux-user, but are really
only talking about softmmu here...
r~
next prev parent reply other threads:[~2015-05-15 17:17 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-15 4:49 [Qemu-devel] [PATCH] microblaze: Remove uses of TCGv and target_ulong Peter Crosthwaite
2015-05-15 5:56 ` Edgar E. Iglesias
2015-05-15 15:41 ` Richard Henderson
2015-05-15 16:48 ` Peter Crosthwaite
2015-05-15 17:17 ` Richard Henderson [this message]
2015-05-15 20:52 ` Peter Crosthwaite
2015-05-18 18:37 ` Richard Henderson
2015-05-18 18:43 ` Peter Maydell
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=55562A2C.9080005@twiddle.net \
--to=rth@twiddle.net \
--cc=afaerber@suse.de \
--cc=crosthwaite.peter@gmail.com \
--cc=crosthwaitepeter@gmail.com \
--cc=edgari@xilinx.com \
--cc=peter.crosthwaite@xilinx.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.