Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-10-19 Thread Peter Maydell
On 19 October 2013 00:38, Roy Franz roy.fr...@linaro.org wrote: Glad to see this go in. Is your target-arm.next branch available in a public repo? No, not generally. -- PMM

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-10-18 Thread Peter Maydell
On 17 October 2013 11:38, Peter Maydell peter.mayd...@linaro.org wrote: On 10 July 2013 05:23, peter.crosthwa...@xilinx.com wrote: From: Nathan Rossi nathan.ro...@xilinx.com Added Vector Base Address remapping on ARM v7. Apologies for this dropping off my radar for so long. I've had a bit

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-10-18 Thread Peter Crosthwaite
Hi, -Original Message- From: Peter Maydell [mailto:peter.mayd...@linaro.org] Sent: Saturday, October 19, 2013 12:06 AM To: Peter Crosthwaite Cc: QEMU Developers Subject: Re: [PATCH v1 4/4] target-arm: Add CP15 VBAR support On 17 October 2013 11:38, Peter Maydell

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-10-18 Thread Roy Franz
On Fri, Oct 18, 2013 at 7:05 AM, Peter Maydell peter.mayd...@linaro.org wrote: On 17 October 2013 11:38, Peter Maydell peter.mayd...@linaro.org wrote: On 10 July 2013 05:23, peter.crosthwa...@xilinx.com wrote: From: Nathan Rossi nathan.ro...@xilinx.com Added Vector Base Address remapping on

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-10-17 Thread Peter Maydell
On 10 July 2013 05:23, peter.crosthwa...@xilinx.com wrote: From: Nathan Rossi nathan.ro...@xilinx.com Added Vector Base Address remapping on ARM v7. Apologies for this dropping off my radar for so long. I've had a bit of a think and I think that you're right that we can put in this register

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-09-17 Thread Sebastian Huber
On 2013-07-10 06:23, peter.crosthwa...@xilinx.com wrote: +static int vbar_write(CPUARMState *env, const ARMCPRegInfo *ri, + uint64_t value) +{ +value = (1 31); Is the above correct? +env-cp15.c12_vbar = value ~0x1Ful; +return 0; +} Shouldn't this be {

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-09-17 Thread Peter Maydell
On 16 September 2013 16:34, Sebastian Huber sebastian.hu...@embedded-brains.de wrote: it would be really nice to get the CP15 VBAR support integrated. This allows unit test suites of single address space real-time systems to catch NULL pointer read/write access for example. I agree it would

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-09-17 Thread Sebastian Huber
On 2013-09-17 10:37, Peter Maydell wrote: In any case a new arm_features bit is hardly a big deal: the actual question is whether it makes sense to provide this register that shouldn't really exist for the cpu configurations we're modelling. I use Qemu to run unit tests for a real-time

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-09-16 Thread Sebastian Huber
Hello, it would be really nice to get the CP15 VBAR support integrated. This allows unit test suites of single address space real-time systems to catch NULL pointer read/write access for example. The ARM documentation says that this is a banked register that is only present in an

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-07-14 Thread Peter Crosthwaite
Ping! On Wed, Jul 10, 2013 at 5:38 PM, Peter Maydell peter.mayd...@linaro.org wrote: On 10 July 2013 05:23, peter.crosthwa...@xilinx.com wrote: From: Nathan Rossi nathan.ro...@xilinx.com Added Vector Base Address remapping on ARM v7. This is only present on CPUs with TrustZone. On the

Re: [Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-07-10 Thread Peter Maydell
On 10 July 2013 05:23, peter.crosthwa...@xilinx.com wrote: From: Nathan Rossi nathan.ro...@xilinx.com Added Vector Base Address remapping on ARM v7. This is only present on CPUs with TrustZone. On the other hand we already implement one or two TZ-only registers for pragmatic get-code-working

[Qemu-devel] [PATCH v1 4/4] target-arm: Add CP15 VBAR support

2013-07-09 Thread peter . crosthwaite
From: Nathan Rossi nathan.ro...@xilinx.com Added Vector Base Address remapping on ARM v7. Signed-off-by: Nathan Rossi nathan.ro...@xilinx.com Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com --- changed since v1: Removed obsolete VMSD logic (rebase) Forced lower 5 bits of VBAR ro 0