Re: [PATCH 34/37] include/system: Remove ifndef CONFIG_USER_ONLY

2025-03-13 Thread Pierrick Bouvier
On 3/12/25 20:45, Richard Henderson wrote: This is include/system, so CONFIG_USER_ONLY will never be true. Some build configurations have this symbol missing. https://github.com/pbo-linaro/qemu-ci/actions/runs/13826820446 Signed-off-by: Richard Henderson --- include/system/cpu-timers.h |

Re: [PATCH 34/37] include/system: Remove ifndef CONFIG_USER_ONLY

2025-03-13 Thread Philippe Mathieu-Daudé
On 13/3/25 04:45, Richard Henderson wrote: This is include/system, so CONFIG_USER_ONLY will never be true. Ideally yes. I started moving headers in include/system/ to clarify APIs, but that doesn't mean some of these APIs aren't being (incorrectly) used in user emulation. Signed-off-by: Richa

[PATCH 34/37] include/system: Remove ifndef CONFIG_USER_ONLY

2025-03-12 Thread Richard Henderson
This is include/system, so CONFIG_USER_ONLY will never be true. Signed-off-by: Richard Henderson --- include/system/cpu-timers.h | 2 +- include/system/qtest.h | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/include/system/cpu-timers.h b/include/system/cpu-timers.h ind