Re: [Qemu-devel] [Qemu-arm] [PATCH 2/8] cpu: Define new cpu_transaction_failed() hook

2017-08-05 Thread Peter Maydell
On 5 August 2017 at 02:12, Edgar E. Iglesias wrote: > BTW, a question. I don't know of any from memory but does any arch > have the ability to report the payload that failed for stores? > I guess it's easy enough to add that if needed though. I think maybe m68k bus fault stack frames have store p

Re: [Qemu-devel] [Qemu-arm] [PATCH 2/8] cpu: Define new cpu_transaction_failed() hook

2017-08-05 Thread Peter Maydell
On 5 August 2017 at 02:06, Edgar E. Iglesias wrote: > On Fri, Aug 04, 2017 at 06:20:43PM +0100, Peter Maydell wrote: >> @@ -85,8 +85,10 @@ struct TranslationBlock; >> * @has_work: Callback for checking if there is work to do. >> * @do_interrupt: Callback for interrupt handling. >> * @do_unas

Re: [Qemu-devel] [Qemu-arm] [PATCH 2/8] cpu: Define new cpu_transaction_failed() hook

2017-08-04 Thread Edgar E. Iglesias
On Fri, Aug 04, 2017 at 06:20:43PM +0100, Peter Maydell wrote: > Currently we have a rather half-baked setup for allowing CPUs to > generate exceptions on accesses to invalid memory: the CPU has a > cpu_unassigned_access() hook which the memory system calls in > unassigned_mem_write() and unassigne

Re: [Qemu-devel] [Qemu-arm] [PATCH 2/8] cpu: Define new cpu_transaction_failed() hook

2017-08-04 Thread Edgar E. Iglesias
On Fri, Aug 04, 2017 at 06:20:43PM +0100, Peter Maydell wrote: > Currently we have a rather half-baked setup for allowing CPUs to > generate exceptions on accesses to invalid memory: the CPU has a > cpu_unassigned_access() hook which the memory system calls in > unassigned_mem_write() and unassigne