[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 06/10] linux-user/main.c: __kernel_cmpxchg se
From: |
Riku Voipio |
Subject: |
Re: [Qemu-devel] [PATCH v2 06/10] linux-user/main.c: __kernel_cmpxchg set env->CF directly |
Date: |
Tue, 15 Jul 2014 16:39:23 +0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Jul 10, 2014 at 04:50:03PM +0100, Alex Bennée wrote:
> As we only need to manipulate the single flag do it directly though env.
Acked-by: Riku Voipio <address@hidden>
> Signed-off-by: Alex Bennée <address@hidden>
>
> ---
>
> v2:
> - remove unused cpsr
> - the direct flag setting seems a little hacky?
>
> diff --git a/linux-user/main.c b/linux-user/main.c
> index 8848e15..9101541 100644
> --- a/linux-user/main.c
> +++ b/linux-user/main.c
> @@ -468,7 +468,7 @@ void cpu_loop(CPUX86State *env)
> static void arm_kernel_cmpxchg64_helper(CPUARMState *env)
> {
> uint64_t oldval, newval, val;
> - uint32_t addr, cpsr;
> + uint32_t addr;
> target_siginfo_t info;
>
> /* Based on the 32 bit code in do_kernel_trap */
> @@ -478,7 +478,6 @@ static void arm_kernel_cmpxchg64_helper(CPUARMState *env)
> operations. However things like ldrex/strex are much harder so
> there's not much point trying. */
> start_exclusive();
> - cpsr = cpsr_read(env);
> addr = env->regs[2];
>
> if (get_user_u64(oldval, env->regs[0])) {
> @@ -505,12 +504,11 @@ static void arm_kernel_cmpxchg64_helper(CPUARMState
> *env)
> };
>
> env->regs[0] = 0;
> - cpsr |= CPSR_C;
> + env->CF = 1;
> } else {
> env->regs[0] = -1;
> - cpsr &= ~CPSR_C;
> + env->CF = 0;
> }
> - cpsr_write(env, cpsr, CPSR_C);
> end_exclusive();
> return;
>
> @@ -533,7 +531,6 @@ static int
> do_kernel_trap(CPUARMState *env)
> {
> uint32_t addr;
> - uint32_t cpsr;
> uint32_t val;
>
> switch (env->regs[15]) {
> @@ -546,7 +543,6 @@ do_kernel_trap(CPUARMState *env)
> operations. However things like ldrex/strex are much harder so
> there's not much point trying. */
> start_exclusive();
> - cpsr = save_state_to_spsr(env);
> addr = env->regs[2];
> /* FIXME: This should SEGV if the access fails. */
> if (get_user_u32(val, addr))
> @@ -556,12 +552,11 @@ do_kernel_trap(CPUARMState *env)
> /* FIXME: Check for segfaults. */
> put_user_u32(val, addr);
> env->regs[0] = 0;
> - cpsr |= CPSR_C;
> + env->CF = 1;
> } else {
> env->regs[0] = -1;
> - cpsr &= ~CPSR_C;
> + env->CF = 0;
> }
> - cpsr_write(env, cpsr, CPSR_C);
> end_exclusive();
> break;
> case 0xffff0fe0: /* __kernel_get_tls */
> --
> 2.0.1
>
- [Qemu-devel] [PATCH v2 00/10] aarch64 migration for TCG and KVM, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 01/10] target-arm/cpu.h: document various program state functions, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 02/10] target-arm/cpu.h: common pstate save/restore, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 03/10] target-arm: Support save/load for 64 bit CPUs, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 04/10] target-arm: replace cpsr/xpsr/pstate_read calls, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 05/10] arm/nwfps: replace cpsr_write with set_condition_codes, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 06/10] linux-user/main.c: __kernel_cmpxchg set env->CF directly, Alex Bennée, 2014/07/10
- Re: [Qemu-devel] [PATCH v2 06/10] linux-user/main.c: __kernel_cmpxchg set env->CF directly,
Riku Voipio <=
- [Qemu-devel] [PATCH v2 07/10] target-arm: remove last users of cpsr_write, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 08/10] target-arm: remove final users of pstate_write, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 09/10] target-arm/kvm.c: better error reporting, Alex Bennée, 2014/07/10
- [Qemu-devel] [PATCH v2 10/10] target-arm/kvm: make reg sync code common between kvm32/64, Alex Bennée, 2014/07/10