hw/misc/a9scu: Verify CPU count is valid and simplify a bit

[PATCH 0/4] hw/misc/a9scu: Verify CPU count is valid and simplify a bit

Philippe Mathieu-Daudé posted 4 patches 5 years, 2 months ago

Download series mbox

Test docker-mingw@fedora failed

Test docker-quick@centos7 failed

Test checkpatch passed

Test FreeBSD failed

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

hw/misc/a9scu.c | 59 +++++++++++++++++++++++--------------------------
1 file changed, 28 insertions(+), 31 deletions(-)

Expand all Fold all

[PATCH 0/4] hw/misc/a9scu: Verify CPU count is valid and simplify a bit

Posted by Philippe Mathieu-Daudé 5 years, 2 months ago

Trivial patches:
- verify the A9 CPU count is in range,
- simplify using MemoryRegionOps valid/impl,
- log unimplemented registers.

Philippe Mathieu-Daudé (4):
  hw/misc/a9scu: Do not allow invalid CPU count
  hw/misc/a9scu: Simplify setting MemoryRegionOps::valid fields
  hw/misc/a9scu: Simplify setting MemoryRegionOps::impl fields
  hw/misc/a9scu: Report unimplemented accesses with qemu_log_mask(UNIMP)

 hw/misc/a9scu.c | 59 +++++++++++++++++++++++--------------------------
 1 file changed, 28 insertions(+), 31 deletions(-)

-- 
2.26.2

Re: [PATCH 0/4] hw/misc/a9scu: Verify CPU count is valid and simplify a bit

Posted by Richard Henderson 5 years, 2 months ago

On 9/1/20 7:40 AM, Philippe Mathieu-Daudé wrote:
> Trivial patches:
> - verify the A9 CPU count is in range,
> - simplify using MemoryRegionOps valid/impl,
> - log unimplemented registers.
> 
> Philippe Mathieu-Daudé (4):
>   hw/misc/a9scu: Do not allow invalid CPU count
>   hw/misc/a9scu: Simplify setting MemoryRegionOps::valid fields
>   hw/misc/a9scu: Simplify setting MemoryRegionOps::impl fields
>   hw/misc/a9scu: Report unimplemented accesses with qemu_log_mask(UNIMP)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~

Re: [PATCH 0/4] hw/misc/a9scu: Verify CPU count is valid and simplify a bit

Posted by Peter Maydell 5 years, 2 months ago

On Tue, 1 Sep 2020 at 15:41, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Trivial patches:
> - verify the A9 CPU count is in range,
> - simplify using MemoryRegionOps valid/impl,
> - log unimplemented registers.
>
> Philippe Mathieu-Daudé (4):
>   hw/misc/a9scu: Do not allow invalid CPU count
>   hw/misc/a9scu: Simplify setting MemoryRegionOps::valid fields
>   hw/misc/a9scu: Simplify setting MemoryRegionOps::impl fields
>   hw/misc/a9scu: Report unimplemented accesses with qemu_log_mask(UNIMP)
>
>  hw/misc/a9scu.c | 59 +++++++++++++++++++++++--------------------------
>  1 file changed, 28 insertions(+), 31 deletions(-)



Applied to target-arm.next, thanks.

-- PMM