mbox series

[0/3] 64 Bit support for hppa gdbstub

Message ID 20240228201434.1515893-1-svens@stackframe.org (mailing list archive)
Headers show
Series 64 Bit support for hppa gdbstub | expand

Message

Sven Schnelle Feb. 28, 2024, 8:14 p.m. UTC
Hi List,

this patchset allows to debug the hppa target when running in wide (64 bit)
mode. gdb needs a small patch to switch to 64 bit mode. I pushed the
patch to https://github.com/bminor/binutils-gdb/commit/fd8662ec282d688d1f8100b4365823e57516857b
With this patch gdb will switch to the appropriate mode when connecting
to qemu/gdbstub.

Sven Schnelle (3):
  Revert "target/hppa: Drop attempted gdbstub support for hppa64"
  target/hppa: add 64 bit support to gdbstub
  target/hppa: mask CR_SAR register writes to 5/6 bit in gdbstub

 target/hppa/gdbstub.c | 66 +++++++++++++++++++++++++++++--------------
 1 file changed, 45 insertions(+), 21 deletions(-)

Comments

Sven Schnelle March 18, 2024, 6:32 a.m. UTC | #1
Hi Richard,

Sven Schnelle <svens@stackframe.org> writes:

> Hi List,
>
> this patchset allows to debug the hppa target when running in wide (64 bit)
> mode. gdb needs a small patch to switch to 64 bit mode. I pushed the
> patch to https://github.com/bminor/binutils-gdb/commit/fd8662ec282d688d1f8100b4365823e57516857b
> With this patch gdb will switch to the appropriate mode when connecting
> to qemu/gdbstub.
>
> Sven Schnelle (3):
>   Revert "target/hppa: Drop attempted gdbstub support for hppa64"
>   target/hppa: add 64 bit support to gdbstub
>   target/hppa: mask CR_SAR register writes to 5/6 bit in gdbstub
>
>  target/hppa/gdbstub.c | 66 +++++++++++++++++++++++++++++--------------
>  1 file changed, 45 insertions(+), 21 deletions(-)

gentle ping - if i followed correctly only one patch was reviewed so far.
Richard Henderson March 18, 2024, 5:09 p.m. UTC | #2
On 3/17/24 20:32, Sven Schnelle wrote:
> Hi Richard,
> 
> Sven Schnelle <svens@stackframe.org> writes:
> 
>> Hi List,
>>
>> this patchset allows to debug the hppa target when running in wide (64 bit)
>> mode. gdb needs a small patch to switch to 64 bit mode. I pushed the
>> patch to https://github.com/bminor/binutils-gdb/commit/fd8662ec282d688d1f8100b4365823e57516857b
>> With this patch gdb will switch to the appropriate mode when connecting
>> to qemu/gdbstub.
>>
>> Sven Schnelle (3):
>>    Revert "target/hppa: Drop attempted gdbstub support for hppa64"
>>    target/hppa: add 64 bit support to gdbstub
>>    target/hppa: mask CR_SAR register writes to 5/6 bit in gdbstub
>>
>>   target/hppa/gdbstub.c | 66 +++++++++++++++++++++++++++++--------------
>>   1 file changed, 45 insertions(+), 21 deletions(-)
> 
> gentle ping - if i followed correctly only one patch was reviewed so far.

We can't really proceed until you get your gdb patch reviewed upstream.
All that will happen in the meantime is that qemu make-check will fail.


r~
Sven Schnelle March 18, 2024, 5:24 p.m. UTC | #3
Richard Henderson <richard.henderson@linaro.org> writes:

> On 3/17/24 20:32, Sven Schnelle wrote:
>> Hi Richard,
>> Sven Schnelle <svens@stackframe.org> writes:
>> 
>>> Hi List,
>>>
>>> this patchset allows to debug the hppa target when running in wide (64 bit)
>>> mode. gdb needs a small patch to switch to 64 bit mode. I pushed the
>>> patch to https://github.com/bminor/binutils-gdb/commit/fd8662ec282d688d1f8100b4365823e57516857b
>>> With this patch gdb will switch to the appropriate mode when connecting
>>> to qemu/gdbstub.
>>>
>>> Sven Schnelle (3):
>>>    Revert "target/hppa: Drop attempted gdbstub support for hppa64"
>>>    target/hppa: add 64 bit support to gdbstub
>>>    target/hppa: mask CR_SAR register writes to 5/6 bit in gdbstub
>>>
>>>   target/hppa/gdbstub.c | 66 +++++++++++++++++++++++++++++--------------
>>>   1 file changed, 45 insertions(+), 21 deletions(-)
>> gentle ping - if i followed correctly only one patch was reviewed so
>> far.
>
> We can't really proceed until you get your gdb patch reviewed upstream.
> All that will happen in the meantime is that qemu make-check will fail.

I see. Thanks!