target/misc: Protect few system binaries for target_ulong use

[PATCH 4/9] target/openrisc: Remove unused 'gdbstub/helpers.h' header in helper.c

Posted by Philippe Mathieu-Daudé 1 month ago

"gdbstub/helpers.h" uses target-specific symbols, but we don't
need it, so remove it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 target/openrisc/interrupt.c | 1 -
 target/openrisc/mmu.c       | 1 -
 2 files changed, 2 deletions(-)

diff --git a/target/openrisc/interrupt.c b/target/openrisc/interrupt.c
index 5528b80e7b8..6e89157639a 100644
--- a/target/openrisc/interrupt.c
+++ b/target/openrisc/interrupt.c
@@ -20,7 +20,6 @@
 #include "qemu/osdep.h"
 #include "qemu/log.h"
 #include "cpu.h"
-#include "gdbstub/helpers.h"
 #include "qemu/host-utils.h"
 #ifndef CONFIG_USER_ONLY
 #include "hw/core/loader.h"
diff --git a/target/openrisc/mmu.c b/target/openrisc/mmu.c
index b2b2b3c4a98..315debaf3e5 100644
--- a/target/openrisc/mmu.c
+++ b/target/openrisc/mmu.c
@@ -24,7 +24,6 @@
 #include "exec/cputlb.h"
 #include "exec/page-protection.h"
 #include "exec/target_page.h"
-#include "gdbstub/helpers.h"
 #include "qemu/host-utils.h"
 #include "hw/core/loader.h"
 
-- 
2.52.0

Re: [PATCH 4/9] target/openrisc: Remove unused 'gdbstub/helpers.h' header in helper.c

Posted by Richard Henderson 4 weeks, 1 day ago

On 1/8/26 07:06, Philippe Mathieu-Daudé wrote:
> "gdbstub/helpers.h" uses target-specific symbols, but we don't
> need it, so remove it.
> 
> Signed-off-by: Philippe Mathieu-Daudé<philmd@linaro.org>
> ---
>   target/openrisc/interrupt.c | 1 -
>   target/openrisc/mmu.c       | 1 -
>   2 files changed, 2 deletions(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~