[v2] make system memory API available for common code

[PATCH v2 08/16] exec/memory-internal: remove dependency on cpu.h

Posted by Pierrick Bouvier 3 weeks, 2 days ago

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
---
 include/exec/memory-internal.h | 2 --
 1 file changed, 2 deletions(-)

diff --git a/include/exec/memory-internal.h b/include/exec/memory-internal.h
index 100c1237ac2..b729f3b25ad 100644
--- a/include/exec/memory-internal.h
+++ b/include/exec/memory-internal.h
@@ -20,8 +20,6 @@
 #ifndef MEMORY_INTERNAL_H
 #define MEMORY_INTERNAL_H
 
-#include "cpu.h"
-
 #ifndef CONFIG_USER_ONLY
 static inline AddressSpaceDispatch *flatview_to_dispatch(FlatView *fv)
 {
-- 
2.39.5

Re: [PATCH v2 08/16] exec/memory-internal: remove dependency on cpu.h

Posted by Philippe Mathieu-Daudé 3 weeks, 2 days ago

On 11/3/25 05:08, Pierrick Bouvier wrote:
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Missing the "why" justification we couldn't do that before.

> ---
>   include/exec/memory-internal.h | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/include/exec/memory-internal.h b/include/exec/memory-internal.h
> index 100c1237ac2..b729f3b25ad 100644
> --- a/include/exec/memory-internal.h
> +++ b/include/exec/memory-internal.h
> @@ -20,8 +20,6 @@
>   #ifndef MEMORY_INTERNAL_H
>   #define MEMORY_INTERNAL_H
>   
> -#include "cpu.h"
> -
>   #ifndef CONFIG_USER_ONLY
>   static inline AddressSpaceDispatch *flatview_to_dispatch(FlatView *fv)
>   {

Re: [PATCH v2 08/16] exec/memory-internal: remove dependency on cpu.h

Posted by Pierrick Bouvier 3 weeks, 1 day ago

On 3/11/25 00:26, Philippe Mathieu-Daudé wrote:
> On 11/3/25 05:08, Pierrick Bouvier wrote:
>> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
>> Signed-off-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>
> 
> Missing the "why" justification we couldn't do that before.
> 
>> ---
>>    include/exec/memory-internal.h | 2 --
>>    1 file changed, 2 deletions(-)
>>
>> diff --git a/include/exec/memory-internal.h b/include/exec/memory-internal.h
>> index 100c1237ac2..b729f3b25ad 100644
>> --- a/include/exec/memory-internal.h
>> +++ b/include/exec/memory-internal.h
>> @@ -20,8 +20,6 @@
>>    #ifndef MEMORY_INTERNAL_H
>>    #define MEMORY_INTERNAL_H
>>    
>> -#include "cpu.h"
>> -
>>    #ifndef CONFIG_USER_ONLY
>>    static inline AddressSpaceDispatch *flatview_to_dispatch(FlatView *fv)
>>    {
> 

No direct dependency, but when a common code will include that (like 
system/memory.c), we can't have a dependency on cpu.h anymore.
I can reorder or squash commits if you prefer.