64 Bit support for hppa gdbstub

[PATCH 0/3] 64 Bit support for hppa gdbstub

Posted by Sven Schnelle 9 months ago

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(-)

-- 
2.43.2

Re: [PATCH 0/3] 64 Bit support for hppa gdbstub

Posted by Sven Schnelle 8 months, 1 week ago

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.

Re: [PATCH 0/3] 64 Bit support for hppa gdbstub

Posted by Richard Henderson 8 months, 1 week ago

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~

Re: [PATCH 0/3] 64 Bit support for hppa gdbstub

Posted by Sven Schnelle 8 months, 1 week ago

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!