diff options
author | Nikunj A Dadhania <nikunj@linux.vnet.ibm.com> | 2017-04-27 10:48:24 +0530 |
---|---|---|
committer | David Gibson <david@gibson.dropbear.id.au> | 2017-05-11 09:45:15 +1000 |
commit | 139d9023f1410591726bbe35908dcfd9d127aad2 (patch) | |
tree | e85a37ba6a8b8b71704cc592ec2b63ae0e6ba68a /memory_mapping.c | |
parent | f0b0685d6694a28c66018f438e822596243b1250 (diff) | |
download | qemu-139d9023f1410591726bbe35908dcfd9d127aad2.zip qemu-139d9023f1410591726bbe35908dcfd9d127aad2.tar.gz qemu-139d9023f1410591726bbe35908dcfd9d127aad2.tar.bz2 |
target/ppc: do not reset reserve_addr in exec_enter
In case when atomic operation is not supported, exit_atomic is called
and we stop the world and execute the atomic operation. This results
in a following call chain:
tcg_gen_atomic_cmpxchg_tl()
-> gen_helper_exit_atomic()
-> HELPER(exit_atomic)
-> cpu_loop_exit_atomic() -> EXCP_ATOMIC
-> qemu_tcg_cpu_thread_fn() => case EXCP_ATOMIC
-> cpu_exec_step_atomic()
-> cpu_step_atomic()
-> cc->cpu_exec_enter() = ppc_cpu_exec_enter()
Sets env->reserve_addr = -1;
But by the time it return back, the reservation is erased and the code
fails, this continues forever and the lock is never taken.
Instead set this in powerpc_excp()
Now that ppc_cpu_exec_enter() doesn't have anything meaningful to do,
let us get rid of the function.
Signed-off-by: Nikunj A Dadhania <nikunj@linux.vnet.ibm.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Diffstat (limited to 'memory_mapping.c')
0 files changed, 0 insertions, 0 deletions