Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-15 Thread Ulrich Weigand
Alexander Graf wrote on 14.01.2014 23:40:20: > On 14.01.2014, at 23:06, Thomas Falcon wrote: > > > This patch allows registers to be properly read from and written to > > when using the gdbstub to debug a ppc guest running in little > > endian mode. It accomplishes this goal by byte swapping the

Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-15 Thread Alexander Graf
On 15.01.2014, at 18:29, Ulrich Weigand wrote: > Alexander Graf wrote on 14.01.2014 23:40:20: >> On 14.01.2014, at 23:06, Thomas Falcon > wrote: >> >>> This patch allows registers to be properly read from and written to >>> when using the gdbstub to debug a ppc guest running in little >>> end

Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Peter Maydell
On 14 January 2014 23:01, Alexander Graf wrote: >> Is the underlying issue here that we might have a CPU which is >> in littleendian mode but in a QEMU executable compiled with >> TARGET_WORDS_BIGENDIAN ? (If so I can't help feeling that >> the gdb stub is only the tip of the iceberg for things t

Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Alexander Graf
On 14.01.2014, at 23:55, Peter Maydell wrote: > On 14 January 2014 22:40, Alexander Graf wrote: >> Uli, I thought ppc64le gdb wasn't finalized yet? What does the gdbstub >> layout look like? Are all fields the same as ppc64(be) but simply byte >> swapped - including FPR ones? > >> This is quit

Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Peter Maydell
On 14 January 2014 22:40, Alexander Graf wrote: > Uli, I thought ppc64le gdb wasn't finalized yet? What does the gdbstub > layout look like? Are all fields the same as ppc64(be) but simply byte > swapped - including FPR ones? > This is quite invasive (and prone to get wrong). If we really just ha

Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Alexander Graf
On 14.01.2014, at 23:06, Thomas Falcon wrote: > This patch allows registers to be properly read from and written to > when using the gdbstub to debug a ppc guest running in little > endian mode. It accomplishes this goal by byte swapping the values of > any registers only if the MSR:LE value is

[Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Thomas Falcon
This patch allows registers to be properly read from and written to when using the gdbstub to debug a ppc guest running in little endian mode. It accomplishes this goal by byte swapping the values of any registers only if the MSR:LE value is set and if the host machine is big endian. Signed-off-b

Re: [Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Peter Maydell
On 14 January 2014 22:06, Thomas Falcon wrote: > This patch allows registers to be properly read from and written to > when using the gdbstub to debug a ppc guest running in little > endian mode. It accomplishes this goal by byte swapping the values of > any registers only if the MSR:LE value is

[Qemu-devel] [PATCH] gdbstub: allow byte swapping for reading/writing registers

2014-01-14 Thread Thomas Falcon
This patch allows registers to be properly read from and written to when using the gdbstub to debug a ppc guest running in little endian mode. It accomplishes this goal by byte swapping the values of any registers only if the MSR:LE value is set and if the host machine is big endian. Signed-off-