[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/20] nvic: Add NS alias SCS region
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH 10/20] nvic: Add NS alias SCS region |
Date: |
Tue, 29 Aug 2017 09:00:12 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
On 08/22/2017 08:08 AM, Peter Maydell wrote:
> + regionlen = arm_feature(&s->cpu->env, ARM_FEATURE_V8) ? 0x21000 : 0x1000;
> + memory_region_init(&s->container, OBJECT(s), "nvic", regionlen);
> /* The system register region goes at the bottom of the priority
> * stack as it covers the whole page.
> */
> @@ -1185,6 +1242,13 @@ static void armv7m_nvic_realize(DeviceState *dev,
> Error **errp)
> sysbus_mmio_get_region(systick_sbd,
> 0),
> 1);
>
> + if (arm_feature(&s->cpu->env, ARM_FEATURE_V8)) {
> + memory_region_init_io(&s->sysreg_ns_mem, OBJECT(s),
> + &nvic_sysreg_ns_ops, s,
> + "nvic_sysregs_ns", 0x1000);
> + memory_region_add_subregion(&s->container, 0x20000,
> &s->sysreg_ns_mem);
There's a whole in between the two regions, which you are leaving mapped. Why
create a sub-region instead of two separate top-level regions for which you can
leave the whole unmapped?
r~
- [Qemu-devel] [PATCH 00/20] first steps towards v8M support, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 12/20] target/arm: Make MPU_MAIR0, MPU_MAIR1 registers banked for v8M, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 07/20] target/arm: Make PRIMASK register banked for v8M, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 10/20] nvic: Add NS alias SCS region, Peter Maydell, 2017/08/22
- Re: [Qemu-devel] [PATCH 10/20] nvic: Add NS alias SCS region,
Richard Henderson <=
- [Qemu-devel] [PATCH 06/20] target/arm: Make BASEPRI register banked for v8M, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 04/20] target/arm: Register second AddressSpace for secure v8M CPUs, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 11/20] target/arm: Make VTOR register banked for v8M, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 17/20] target/arm: Make MMFAR banked for v8M, Peter Maydell, 2017/08/22
- [Qemu-devel] [PATCH 08/20] target/arm: Make FAULTMASK register banked for v8M, Peter Maydell, 2017/08/22