[v6] target/mips: Use probe_access_full() in Atomic Load/Store helpers

[PATCH v6 0/4] target/mips: Use probe_access_full() in Atomic Load/Store helpers

Philippe Mathieu-Daudé posted 4 patches 1 month, 1 week ago

Diff against v4 v5
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260417042620.35329-1-philmd@linaro.org

Maintainers: "Philippe Mathieu-Daudé" <philmd@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Aleksandar Rikalo <arikalo@gmail.com>

target/mips/helper.h                      |  4 ++--
target/mips/tcg/ldst_helper.c             | 27 ++++++++++++++---------
target/mips/tcg/translate.c               | 19 ++++++++++------
target/mips/tcg/micromips_translate.c.inc |  2 +-
target/mips/tcg/mips16e_translate.c.inc   |  8 ++++---
target/mips/tcg/nanomips_translate.c.inc  |  5 +++--
6 files changed, 40 insertions(+), 25 deletions(-)

Expand all Fold all

[PATCH v6 0/4] target/mips: Use probe_access_full() in Atomic Load/Store helpers

Posted by Philippe Mathieu-Daudé 1 month, 1 week ago

Since v5:
- Remove dead code (rth)
- Reword commit desc (rth)
- Inline translator_ld*

Since v4:
- Moved MO_ALIGN change

Following Richard suggestion in [*], use probe_access_full()
and replace legacy cpu_ld*_mmuidx_ra() by cpu_ld*_mmu() in Atomic
Load/Store helpers.

[*] https://lore.kernel.org/qemu-devel/ee4ebd4e-b475-4d76-8fd2-8b04c30b183c@linaro.org/

Philippe Mathieu-Daudé (4):
  target/mips: Pass MemOpIdx argument to Linked Load/Store helpers
  target/mips: Inline cpu_ld/st_mmuidx_ra() calls in Atomic LD/ST
    helpers
  target/mips: Inline translator_ld[uw,l,q]() calls
  target/mips: Remove last MO_TE use

 target/mips/helper.h                      |  4 ++--
 target/mips/tcg/ldst_helper.c             | 27 ++++++++++++++---------
 target/mips/tcg/translate.c               | 19 ++++++++++------
 target/mips/tcg/micromips_translate.c.inc |  2 +-
 target/mips/tcg/mips16e_translate.c.inc   |  8 ++++---
 target/mips/tcg/nanomips_translate.c.inc  |  5 +++--
 6 files changed, 40 insertions(+), 25 deletions(-)

-- 
2.53.0

Re: [PATCH v6 0/4] target/mips: Use probe_access_full() in Atomic Load/Store helpers

Posted by Philippe Mathieu-Daudé 1 month, 1 week ago

On 17/4/26 06:26, Philippe Mathieu-Daudé wrote:

> Philippe Mathieu-Daudé (4):
>    target/mips: Pass MemOpIdx argument to Linked Load/Store helpers
>    target/mips: Inline cpu_ld/st_mmuidx_ra() calls in Atomic LD/ST
>      helpers
>    target/mips: Inline translator_ld[uw,l,q]() calls
>    target/mips: Remove last MO_TE use

Series queued, thanks.