[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-arm] [PATCH 2/6] armv7m: Replace armv7m.hack with
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [Qemu-arm] [PATCH 2/6] armv7m: Replace armv7m.hack with unassigned_access handler |
Date: |
Tue, 24 Jan 2017 16:53:23 +0000 |
On 24 January 2017 at 16:31, Alex Bennée <address@hidden> wrote:
>
> Peter Maydell <address@hidden> writes:
>> - /* Hack to map an additional page of ram at the top of the address
>> - space. This stops qemu complaining about executing code outside RAM
>> - when returning from an exception. */
>> - memory_region_init_ram(hack, NULL, "armv7m.hack", 0x1000, &error_fatal);
>> - vmstate_register_ram_global(hack);
>> - memory_region_add_subregion(system_memory, 0xfffff000, hack);
>> -
>
> What stops a model inadvertently registering a block of memory on-top?
> Should we check the memory region really is unassigned?
It would just be a board code bug, and a pretty implausible one
I think (at least no less plausible than any other typoing of
a device or memory address, 99% of which are uncheckable).
thanks
-- PMM
- [Qemu-devel] [PATCH 0/6] ARMv7M: some simple bugfixes and cleanups, Peter Maydell, 2017/01/20
- [Qemu-devel] [PATCH 6/6] armv7m: Clear FAULTMASK on return from non-NMI exceptions, Peter Maydell, 2017/01/20
- [Qemu-devel] [PATCH 1/6] armv7m: MRS/MSR: handle unprivileged access, Peter Maydell, 2017/01/20
- [Qemu-devel] [PATCH 3/6] armv7m: Explicit error for bad vector table, Peter Maydell, 2017/01/20
- [Qemu-devel] [PATCH 2/6] armv7m: Replace armv7m.hack with unassigned_access handler, Peter Maydell, 2017/01/20
- [Qemu-devel] [PATCH 5/6] armv7m: Fix reads of CONTROL register bit 1, Peter Maydell, 2017/01/20
- [Qemu-devel] [PATCH 4/6] hw/registerfields.h: Pull FIELD etc macros out of hw/register.h, Peter Maydell, 2017/01/20
- Re: [Qemu-devel] [PATCH 0/6] ARMv7M: some simple bugfixes and cleanups, no-reply, 2017/01/20
- Re: [Qemu-devel] [Qemu-arm] [PATCH 0/6] ARMv7M: some simple bugfixes and cleanups, Alex Bennée, 2017/01/24