Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-04-05 Thread Philippe Mathieu-Daudé
Hi Richard, On 18/3/25 22:31, Richard Henderson wrote: The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY. We already have the plumbing common to user and system mode. Using MMU_USER_IDX requires the cpu.h for a specific target, and

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-04-05 Thread Richard Henderson
On 3/19/25 10:16, Pierrick Bouvier wrote: If I understand correctly, this is simply an (arbitrary) choice related to each target architecture implemented in QEMU, and it does not match any property of a *real* cpu. Is that correct? Correct. In other words, it could have been implemented in a

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-04-02 Thread Philippe Mathieu-Daudé
On 2/4/25 20:33, Richard Henderson wrote: On 4/2/25 04:26, Philippe Mathieu-Daudé wrote: Hi Richard, On 18/3/25 22:31, Richard Henderson wrote: The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY.  We already have the plumbing common

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-04-02 Thread Richard Henderson
On 4/2/25 04:26, Philippe Mathieu-Daudé wrote: Hi Richard, On 18/3/25 22:31, Richard Henderson wrote: The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY.  We already have the plumbing common to user and system mode.  Using MMU_USER_ID

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-03-20 Thread Pierrick Bouvier
On 3/20/25 07:58, Richard Henderson wrote: On 3/19/25 10:16, Pierrick Bouvier wrote: If I understand correctly, this is simply an (arbitrary) choice related to each target architecture implemented in QEMU, and it does not match any property of a *real* cpu. Is that correct? Correct. In oth

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-03-19 Thread Pierrick Bouvier
On 3/18/25 18:16, Richard Henderson wrote: On 3/18/25 17:02, Pierrick Bouvier wrote: On 3/18/25 14:31, Richard Henderson wrote: The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY.  We already have the plumbing common to user and syste

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-03-18 Thread Richard Henderson
On 3/18/25 17:02, Pierrick Bouvier wrote: On 3/18/25 14:31, Richard Henderson wrote: The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY.  We already have the plumbing common to user and system mode.  Using MMU_USER_IDX requires the cpu

Re: [PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-03-18 Thread Pierrick Bouvier
On 3/18/25 14:31, Richard Henderson wrote: The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY. We already have the plumbing common to user and system mode. Using MMU_USER_IDX requires the cpu.h for a specific target, and so is restric

[PATCH v2 06/42] include/exec: Split out cpu-mmu-index.h

2025-03-18 Thread Richard Henderson
The implementation of cpu_mmu_index was split between cpu-common.h and cpu-all.h, depending on CONFIG_USER_ONLY. We already have the plumbing common to user and system mode. Using MMU_USER_IDX requires the cpu.h for a specific target, and so is restricted to when we're compiling per-target. Incl