[v4] cmpxchg and lock cmpxchg should not touch accumulator

[PATCH v4 0/1] cmpxchg and lock cmpxchg should not touch accumulator

Wei Li posted 1 patch 2 years, 1 month ago

Diff against v1 v2 v3
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220323013417.409858-1-lw945lw945@yahoo.com

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>

target/i386/tcg/translate.c | 41 +++++++++++++++++--------------------
1 file changed, 19 insertions(+), 22 deletions(-)

Expand all Fold all

[PATCH v4 0/1] cmpxchg and lock cmpxchg should not touch accumulator

Posted by Wei Li 2 years, 1 month ago

This series fix a bug reported on issues 508.
The problem is cmpxchg and lock cmpxchg would touch accumulator when
the accumulator is equal to the TEMP.

Changes from v3
* Give a consolidated description of the problem fixed.

v3 link:
https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg05584.html

Changes from v2
* Give a better code struture to reduce more code duplication.

v2 link:
https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg05410.html

Changes from v1
* cmpxchg uses the lock cmpxchg path whenever mod != 3 to reduce code
  duplication.
* lock cmpxchg uses movcond to replace branch.
* Combine the two patches into one patch because cmpxchg uses the lock
  cmpxchg path.

v1 link:
https://lists.gnu.org/archive/html/qemu-devel/2022-03/msg05023.html

Wei Li (1):
  fix cmpxchg and lock cmpxchg instruction

 target/i386/tcg/translate.c | 41 +++++++++++++++++--------------------
 1 file changed, 19 insertions(+), 22 deletions(-)

-- 
2.30.2