qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Alex Bennée" <alex.bennee@linaro.org>
To: qemu-devel@nongnu.org
Cc: edgar.iglesias@xilinx.com, peter.maydell@linaro.org,
	"Alex Bennée" <alex.bennee@linaro.org>,
	greg.bellows@linaro.org
Subject: [Qemu-devel] [RCF PATCH 0/8] aarch64 migration fixes and psr cleanup
Date: Mon,  2 Jun 2014 17:21:54 +0100	[thread overview]
Message-ID: <1401726122-11132-1-git-send-email-alex.bennee@linaro.org> (raw)

Hi,

This started with a hack-patch Peter had written to get migration
working for aarch64 system images. It was noted that cpu.h was
starting to proliferate program state functions. This series attempts
to fix that by:

  - creating a "universal" save/restore_to/from_spsr function
  - using that for migration purposes
  - removing existing use of cpsr/pstate_read/write functions
  - fixing some places to manipulate integer flags directly

Hanging questions:

  - should xpsr_read/write get merged or left as it's own special
    snowflake?
  - should env->uncached_cpsr and env->pstate be totally hidden behind
    access functions so other flags can get split (i.e. env->el_mode?)

I've tested this with ARMv7 and ARMv8 with aarch32 user-space system
images. I've not given it a good linux-user test yet so I'm less sure
about the sigcontext manipulation code.

Comments?

Cheers,

--
Alex Bennée
QEMU/KVM Hacker for Linaro


Alex Bennée (8):
  target-arm/cpu.h: document various program state functions
  target-arm/cpu.h: implement common state save/restore
  target-arm: Support save/load for 64 bit CPUs
  target-arm: replace cpsr_read/pstate_read calls
  arm/nwfps: remove use of cpsr_write() and set flags directly
  linux-user/main.c: __kernel_cmpxchg set env->CF directly
  target-arm: remove last users of cpsr_write
  target-arm: remove final users of pstate_write

 linux-user/arm/nwfpe/fpa11.h |   5 +-
 linux-user/elfload.c         |   4 +-
 linux-user/main.c            |  16 ++--
 linux-user/signal.c          |  64 ++++++++--------
 target-arm/cpu.h             | 173 ++++++++++++++++++++++++++++++++++---------
 target-arm/gdbstub.c         |  10 ++-
 target-arm/gdbstub64.c       |   6 +-
 target-arm/helper-a64.c      |  11 +--
 target-arm/helper.c          |  76 +------------------
 target-arm/kvm32.c           |   4 +-
 target-arm/kvm64.c           |   4 +-
 target-arm/machine.c         |  27 ++++---
 target-arm/op_helper.c       |  55 +++++++++++---
 target-arm/translate-a64.c   |   2 +-
 target-arm/translate.c       |   2 +-
 15 files changed, 266 insertions(+), 193 deletions(-)

-- 
2.0.0

             reply	other threads:[~2014-06-02 16:22 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-02 16:21 Alex Bennée [this message]
2014-06-02 16:21 ` [Qemu-devel] [RCF PATCH 1/8] target-arm/cpu.h: document various program state functions Alex Bennée
2014-06-02 16:40   ` Greg Bellows
2014-06-02 16:21 ` [Qemu-devel] [RCF PATCH 2/8] target-arm/cpu.h: implement common state save/restore Alex Bennée
2014-06-02 16:21 ` [Qemu-devel] [RCF PATCH 3/8] target-arm: Support save/load for 64 bit CPUs Alex Bennée
2014-06-02 16:21 ` [Qemu-devel] [RCF PATCH 4/8] target-arm: replace cpsr_read/pstate_read calls Alex Bennée
2014-06-02 16:21 ` [Qemu-devel] [RCF PATCH 5/8] arm/nwfps: remove use of cpsr_write() and set flags directly Alex Bennée
2014-06-03 16:11   ` Peter Maydell
2014-06-04 11:11     ` Alex Bennée
2014-06-04 13:10       ` Peter Maydell
2014-06-02 16:22 ` [Qemu-devel] [RCF PATCH 6/8] linux-user/main.c: __kernel_cmpxchg set env->CF directly Alex Bennée
2014-06-02 16:22 ` [Qemu-devel] [RCF PATCH 7/8] target-arm: remove last users of cpsr_write Alex Bennée
2014-06-02 16:22 ` [Qemu-devel] [RCF PATCH 8/8] target-arm: remove final users of pstate_write Alex Bennée
2014-06-03 10:19   ` Alex Bennée

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=1401726122-11132-1-git-send-email-alex.bennee@linaro.org \
    --to=alex.bennee@linaro.org \
    --cc=edgar.iglesias@xilinx.com \
    --cc=greg.bellows@linaro.org \
    --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 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).