All of lore.kernel.org
 help / color / mirror / Atom feed
From: Richard Henderson <rth@twiddle.net>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: qemu-devel <qemu-devel@nongnu.org>,
	Aurelien Jarno <aurelien@aurel32.net>
Subject: Re: [Qemu-devel] [RFC] Streamlining endian handling in TCG
Date: Wed, 28 Aug 2013 10:45:43 -0700	[thread overview]
Message-ID: <521E3747.4070709@twiddle.net> (raw)
In-Reply-To: <CAFEAcA8Un3bLywmFoa4cFhigEtL7NGGOti2g9pA=AFmRzxSk5Q@mail.gmail.com>

On 08/28/2013 10:28 AM, Peter Maydell wrote:
> Well, what do ldl_p or ldl_phys or any of the other functions
> without an le/be qualifier mean any more if "is this CPU in
> big endian mode?" now requires you to have a CPUState
> to ask? I guess we can tackle that separately from getting
> the assumptions on endianness out of the tcg backends
> though.

ldl_p is the host native endian unaligned load primitive.

ldl_phys is the one that'll need to be adjusted or eliminated.

>From a quick glance, this doesn't look too bad.  Most of the uses are in the
target-* directories already, and mostly for emulating page tables.  Those
should be trivial to adjust as needed.

The troublesome offender appears to be virtio.  I know some discussion of
cleaning that up has happened in the context of LE PowerPC, but I didn't really
follow what went on there.


r~

  reply	other threads:[~2013-08-28 17:46 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-28  4:39 [Qemu-trivial] [PATCH] target-arm: Report unimplemented opcodes (LOG_UNIMP) Stefan Weil
2013-08-28  4:39 ` [Qemu-devel] " Stefan Weil
2013-08-28  8:15 ` [Qemu-trivial] " Peter Maydell
2013-08-28  8:15   ` [Qemu-devel] " Peter Maydell
2013-08-28 14:31   ` [Qemu-trivial] " Richard Henderson
2013-08-28 14:31     ` Richard Henderson
2013-08-28 14:34     ` [Qemu-trivial] " Peter Maydell
2013-08-28 14:34       ` Peter Maydell
2013-08-28 15:26       ` [Qemu-devel] [RFC] Streamlining endian handling in TCG Richard Henderson
2013-08-28 16:38         ` Peter Maydell
2013-08-28 17:16           ` Richard Henderson
2013-08-28 17:28             ` Peter Maydell
2013-08-28 17:45               ` Richard Henderson [this message]
2013-08-28 17:41           ` Stefan Weil
2013-08-28 20:42         ` Edgar E. Iglesias
2013-08-28 21:06           ` Peter Maydell
2013-08-28 21:23           ` Richard Henderson
2013-09-02 23:42         ` Aurelien Jarno
2013-09-03 15:11           ` Richard Henderson
2013-09-01 15:35 ` [Qemu-trivial] [PATCH] target-arm: Report unimplemented opcodes (LOG_UNIMP) Michael Tokarev
2013-09-01 15:35   ` [Qemu-devel] " Michael Tokarev

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=521E3747.4070709@twiddle.net \
    --to=rth@twiddle.net \
    --cc=aurelien@aurel32.net \
    --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.