diff mbox series

arm: gdb-stub is broken by FEAT_HAFDBS

Message ID 20221124033802.meuiphlcskwu3aty@M910t (mailing list archive)
State New, archived
Headers show
Series arm: gdb-stub is broken by FEAT_HAFDBS | expand

Commit Message

Changbin Du Nov. 24, 2022, 3:38 a.m. UTC
Hello, Richard,
We just noticed the gdb-stub is broken and probably caused by commit 4a3585568
("target/arm: Plumb debug into S1Translate").

(gdb) target remote :1234
Remote debugging using :1234
0x000000000e1716d0 in ?? ()
=> 0x000000000e1716d0:  Cannot access memory at address 0xe1716d0

This issue can be workaround by below change.

Comments

Denis V. Lunev" via Nov. 24, 2022, 6:35 a.m. UTC | #1
On 2022/11/23 19:59, Peter Maydell wrote:
> On Wed, 23 Nov 2022 at 11:38, Changbin Du <changbin.du@huawei.com> wrote:
>>
>> Hello, Richard,
>> We just noticed the gdb-stub is broken and probably caused by commit 4a3585568
>> ("target/arm: Plumb debug into S1Translate").
>>
>> (gdb) target remote :1234
>> Remote debugging using :1234
>> 0x000000000e1716d0 in ?? ()
>> => 0x000000000e1716d0:  Cannot access memory at address 0xe1716d0
> 
> Hi -- is this fixed by commit 26ba00cf58e9f21b08f (just landed
> upstream last night) ?
> 
I pulled the mainline  which contains your commit 26ba00cf58e9f21b08f.
The problem is fixed.

Thank you very much.
> thanks
> -- PMM
diff mbox series

Patch

--- a/target/arm/ptw.c
+++ b/target/arm/ptw.c
@@ -2879,7 +2879,7 @@  hwaddr arm_cpu_get_phys_page_attrs_debug(CPUState *cs, vaddr addr,
     S1Translate ptw = {
         .in_mmu_idx = arm_mmu_idx(env),
         .in_secure = arm_is_secure(env),
-        .in_debug = true,
+        .in_debug = false,
     };

Could you take a look at this? Thank you!