Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-22 Thread Pierrick Bouvier
On 4/22/25 11:24, Markus Armbruster wrote: Effectively duplicates generated enum SysEmuTarget. Can you explain why that's useful? In terms of code, it doesn't change anything. we could reuse SysEmuTarget. I just think it's more clear to have the enum defined next to the structure using it, co

Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-22 Thread Markus Armbruster
Pierrick Bouvier writes: > On 4/21/25 23:04, Markus Armbruster wrote: >> Pierrick Bouvier writes: >> [...] >> >>> At this point, I would like to focus on having a first version of >>> TargetInfo API, and not reviewing any other changes, as things may be >>> modified, and they would need to be

Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-22 Thread Pierrick Bouvier
On 4/21/25 23:04, Markus Armbruster wrote: Pierrick Bouvier writes: [...] At this point, I would like to focus on having a first version of TargetInfo API, and not reviewing any other changes, as things may be modified, and they would need to be reviewed again. It's hard to follow the same

Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-21 Thread Markus Armbruster
Pierrick Bouvier writes: [...] > At this point, I would like to focus on having a first version of TargetInfo > API, and not reviewing any other changes, as things may be modified, and they > would need to be reviewed again. It's hard to follow the same abstraction > done multiple times in mu

Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-19 Thread Pierrick Bouvier
On 4/19/25 05:54, Philippe Mathieu-Daudé wrote: I don't think doing strcmp is a good move here, even temporarily. A short term solution is making target_info.c target specific, and use: return TARGET_AARCH64; IIUC as https://lore.kernel.org/qemu-devel/20231122183048.17150-3-phi...@linaro.org/?

Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-19 Thread Philippe Mathieu-Daudé
On 19/4/25 03:09, Pierrick Bouvier wrote: On 4/18/25 10:29, Philippe Mathieu-Daudé wrote: Add a helper to distinct the binary is targetting Aarch64 or not. Start with a dump strcmp() implementation, leaving room for future optimizations. Signed-off-by: Philippe Mathieu-Daudé ---   include/qem

Re: [RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-18 Thread Pierrick Bouvier
On 4/18/25 10:29, Philippe Mathieu-Daudé wrote: Add a helper to distinct the binary is targetting Aarch64 or not. Start with a dump strcmp() implementation, leaving room for future optimizations. Signed-off-by: Philippe Mathieu-Daudé --- include/qemu/target_info.h | 7 +++ target_info.c

[RFC PATCH v3 13/14] qemu/target_info: Add target_aarch64() helper

2025-04-18 Thread Philippe Mathieu-Daudé
Add a helper to distinct the binary is targetting Aarch64 or not. Start with a dump strcmp() implementation, leaving room for future optimizations. Signed-off-by: Philippe Mathieu-Daudé --- include/qemu/target_info.h | 7 +++ target_info.c | 5 + 2 files changed, 12 inserti