[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/5] ARM BE8/BE32 semihosting and gdbstub suppor
Re: [Qemu-devel] [PATCH 1/5] ARM BE8/BE32 semihosting and gdbstub support.
Fri, 4 Nov 2016 12:01:37 +0100
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0
On 04/11/2016 11:25, Julian Brown wrote:
> > The change to arm_cpu_memory_rw_debug for BE32 is also interesting.
> > gdb documentation says
> > The stub need not use any particular size or alignment when
> > gathering data from memory for the response; even if ADDR is
> > word-aligned and LENGTH is a multiple of the word size, the stub
> > is free to use byte accesses, or not.
> > while your change means that gdb actually wants you to do byte
> > accesses.
> The splitting-into-bytes is just an implementation convenience -- the
> simplest way I could see of handling the low-order address bit reversal
> without breaking abstractions more or shuffling lots of code around.
> I'm not sure if GDB was actually requesting sub-word access sizes.
Right, the question is what GDB actually wants. I don't really
understand how BE32 is compatible with "the stub need not use any
particular size or alignment", unless GDB takes care of tweaking the
address on its end (and possibly taking care of always reading full
words when len>4?!?).
Also, it would not depend on SCTLR.B, but rather on the CFGEND property
and on CFGEND being tied to SCTLR.B.
[Qemu-devel] [PATCH 3/5] Fix arm_semi_flen_cb for BE32 system mode., Julian Brown, 2016/11/03
[Qemu-devel] [PATCH 4/5] ARM BE32 watchpoint fix., Julian Brown, 2016/11/03
[Qemu-devel] [PATCH 5/5] Fix typo in arm_cpu_do_interrupt_aarch32., Julian Brown, 2016/11/03
Re: [Qemu-devel] [PATCH 0/5] ARM BE8/BE32 big-endian system-mode fixes (semihosting, gdbstub), no-reply, 2016/11/03