[PATCH 0/5] mips: Sanitize Multi-Threading ASE

Philippe Mathieu-Daudé posted 5 patches 4 years, 11 months ago
Test checkpatch passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201204222622.2743175-1-f4bug@amsat.org
Maintainers: Aleksandar Rikalo <aleksandar.rikalo@syrmia.com>, Jiaxun Yang <jiaxun.yang@flygoat.com>, "Philippe Mathieu-Daudé" <f4bug@amsat.org>, Aurelien Jarno <aurelien@aurel32.net>
target/mips/cpu.h                |  7 +++++++
hw/mips/cps.c                    |  3 +--
hw/mips/malta.c                  | 10 ++++++++--
target/mips/cp0_helper.c         |  2 +-
target/mips/cpu.c                |  2 +-
target/mips/helper.c             |  2 +-
target/mips/translate.c          |  4 ++--
target/mips/translate_init.c.inc |  6 +++++-
8 files changed, 26 insertions(+), 10 deletions(-)
[PATCH 0/5] mips: Sanitize Multi-Threading ASE
Posted by Philippe Mathieu-Daudé 4 years, 11 months ago
Reviewing the MIPS code, ASE after ASE.
Time for MT ASE.

- Introduce/use ase_mt_available() helper to check
  if MT ASE is present
- Avoid setting MT specific registers if MT ASE is absent

Philippe Mathieu-Daudé (5):
  target/mips: Remove mips_def_t unused argument from mvp_init()
  target/mips: Introduce ase_mt_available() helper
  target/mips: Do not initialize MT registers if MT ASE absent
  hw/mips/malta: Do not initialize MT registers if MT ASE absent
  hw/mips/malta: Rewrite CP0_MVPConf0 access using deposit()

 target/mips/cpu.h                |  7 +++++++
 hw/mips/cps.c                    |  3 +--
 hw/mips/malta.c                  | 10 ++++++++--
 target/mips/cp0_helper.c         |  2 +-
 target/mips/cpu.c                |  2 +-
 target/mips/helper.c             |  2 +-
 target/mips/translate.c          |  4 ++--
 target/mips/translate_init.c.inc |  6 +++++-
 8 files changed, 26 insertions(+), 10 deletions(-)

-- 
2.26.2

Re: [PATCH 0/5] mips: Sanitize Multi-Threading ASE
Posted by Philippe Mathieu-Daudé 4 years, 11 months ago
On 12/4/20 11:26 PM, Philippe Mathieu-Daudé wrote:
> Reviewing the MIPS code, ASE after ASE.
> Time for MT ASE.
> 
> - Introduce/use ase_mt_available() helper to check
>   if MT ASE is present
> - Avoid setting MT specific registers if MT ASE is absent
> 
> Philippe Mathieu-Daudé (5):
>   target/mips: Remove mips_def_t unused argument from mvp_init()
>   target/mips: Introduce ase_mt_available() helper
>   target/mips: Do not initialize MT registers if MT ASE absent
>   hw/mips/malta: Do not initialize MT registers if MT ASE absent
>   hw/mips/malta: Rewrite CP0_MVPConf0 access using deposit()
> 
>  target/mips/cpu.h                |  7 +++++++
>  hw/mips/cps.c                    |  3 +--
>  hw/mips/malta.c                  | 10 ++++++++--
>  target/mips/cp0_helper.c         |  2 +-
>  target/mips/cpu.c                |  2 +-
>  target/mips/helper.c             |  2 +-
>  target/mips/translate.c          |  4 ++--
>  target/mips/translate_init.c.inc |  6 +++++-
>  8 files changed, 26 insertions(+), 10 deletions(-)

Thanks, applied to mips-next.