[PATCH 0/2] exec: Restrict memory-internal.h to system/

[PATCH 0/2] exec: Restrict memory-internal.h to system/

[Philippe Mathieu-Daudé]

Only file units within the system/ directory need access to
"memory-internal.h". Move it to system/ to restrict its scope.

Based-on: <20250314173139.2122904-1-pierrick.bouvier@linaro.org>

Philippe Mathieu-Daudé (2):
  accel/tcg: Remove unnecesary inclusion of memory-internal.h in
    cputlb.c
  exec: Restrict memory-internal.h to system/

 MAINTAINERS                                | 2 +-
 {include/exec => system}/memory-internal.h | 6 ------
 accel/tcg/cputlb.c                         | 1 -
 system/memory.c                            | 3 ++-
 system/physmem.c                           | 3 ++-
 5 files changed, 5 insertions(+), 10 deletions(-)
 rename {include/exec => system}/memory-internal.h (88%)

-- 
2.47.1

[PATCH 1/2] accel/tcg: Remove unnecesary inclusion of memory-internal.h in cputlb.c

[Philippe Mathieu-Daudé]

At some point cputlb.c stopped depending on the
"exec/memory-internal.h" header. Clean that now.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 accel/tcg/cputlb.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c
index fb22048876e..5007bdbcd75 100644
--- a/accel/tcg/cputlb.c
+++ b/accel/tcg/cputlb.c
@@ -26,7 +26,6 @@
 #include "exec/cpu_ldst.h"
 #include "exec/cputlb.h"
 #include "exec/tb-flush.h"
-#include "exec/memory-internal.h"
 #include "exec/ram_addr.h"
 #include "exec/mmu-access-type.h"
 #include "exec/tlb-common.h"
-- 
2.47.1

[PATCH 2/2] exec: Restrict memory-internal.h to system/

[Philippe Mathieu-Daudé]

Only file units within the system/ directory need access to
"memory-internal.h". Restrict its scope by moving it there.

The comment from commit 9d70618c684 ("memory-internal.h:
Remove obsolete claim that header is obsolete") is now obsolete,
remove it.

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 MAINTAINERS                                | 2 +-
 {include/exec => system}/memory-internal.h | 6 ------
 system/memory.c                            | 3 ++-
 system/physmem.c                           | 3 ++-
 4 files changed, 5 insertions(+), 9 deletions(-)
 rename {include/exec => system}/memory-internal.h (88%)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8f470a1c9b7..b9f6c3cf15c 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3115,7 +3115,7 @@ F: system/ioport.c
 F: system/memory.c
 F: system/memory_mapping.c
 F: system/physmem.c
-F: include/exec/memory-internal.h
+F: system/memory-internal.h
 F: scripts/coccinelle/memory-region-housekeeping.cocci
 
 Memory devices
diff --git a/include/exec/memory-internal.h b/system/memory-internal.h
similarity index 88%
rename from include/exec/memory-internal.h
rename to system/memory-internal.h
index c75178a3d6b..085e81a9fe4 100644
--- a/include/exec/memory-internal.h
+++ b/system/memory-internal.h
@@ -11,12 +11,6 @@
  *
  */
 
-/*
- * This header is for use by exec.c, memory.c and accel/tcg/cputlb.c ONLY,
- * for declarations which are shared between the memory subsystem's
- * internals and the TCG TLB code. Do not include it from elsewhere.
- */
-
 #ifndef MEMORY_INTERNAL_H
 #define MEMORY_INTERNAL_H
 
diff --git a/system/memory.c b/system/memory.c
index eddd21a6cdb..f9b34cf4254 100644
--- a/system/memory.c
+++ b/system/memory.c
@@ -25,7 +25,6 @@
 #include "qom/object.h"
 #include "trace.h"
 
-#include "exec/memory-internal.h"
 #include "exec/ram_addr.h"
 #include "system/kvm.h"
 #include "system/runstate.h"
@@ -35,6 +34,8 @@
 #include "migration/vmstate.h"
 #include "exec/address-spaces.h"
 
+#include "memory-internal.h"
+
 //#define DEBUG_UNASSIGNED
 
 static unsigned memory_region_transaction_depth;
diff --git a/system/physmem.c b/system/physmem.c
index e97de3ef65c..d4fbda9310d 100644
--- a/system/physmem.c
+++ b/system/physmem.c
@@ -66,7 +66,6 @@
 #include "qemu/main-loop.h"
 #include "system/replay.h"
 
-#include "exec/memory-internal.h"
 #include "exec/ram_addr.h"
 
 #include "qemu/pmem.h"
@@ -88,6 +87,8 @@
 #include <daxctl/libdaxctl.h>
 #endif
 
+#include "memory-internal.h"
+
 //#define DEBUG_SUBPAGE
 
 /* ram_list is read under rcu_read_lock()/rcu_read_unlock().  Writes
-- 
2.47.1

Re: [PATCH 0/2] exec: Restrict memory-internal.h to system/

[David Hildenbrand]

On 17.03.25 17:13, Philippe Mathieu-Daudé wrote:
> Only file units within the system/ directory need access to
> "memory-internal.h". Move it to system/ to restrict its scope.
> 
> Based-on: <20250314173139.2122904-1-pierrick.bouvier@linaro.org>

Reviewed-by: David Hildenbrand <david@redhat.com>

-- 
Cheers,

David / dhildenb

Re: [PATCH 1/2] accel/tcg: Remove unnecesary inclusion of memory-internal.h in cputlb.c

[Pierrick Bouvier]

On 3/17/25 09:13, Philippe Mathieu-Daudé wrote:
> At some point cputlb.c stopped depending on the
> "exec/memory-internal.h" header. Clean that now.
> 
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   accel/tcg/cputlb.c | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/accel/tcg/cputlb.c b/accel/tcg/cputlb.c
> index fb22048876e..5007bdbcd75 100644
> --- a/accel/tcg/cputlb.c
> +++ b/accel/tcg/cputlb.c
> @@ -26,7 +26,6 @@
>   #include "exec/cpu_ldst.h"
>   #include "exec/cputlb.h"
>   #include "exec/tb-flush.h"
> -#include "exec/memory-internal.h"
>   #include "exec/ram_addr.h"
>   #include "exec/mmu-access-type.h"
>   #include "exec/tlb-common.h"

Reviewed-by: Pierrick Bouvier <pierrick.bouvier@linaro.org>

Re: [PATCH 0/2] exec: Restrict memory-internal.h to system/

[Richard Henderson]

On 3/17/25 09:13, Philippe Mathieu-Daudé wrote:
> Only file units within the system/ directory need access to
> "memory-internal.h". Move it to system/ to restrict its scope.
> 
> Based-on: <20250314173139.2122904-1-pierrick.bouvier@linaro.org>
> 
> Philippe Mathieu-Daudé (2):
>    accel/tcg: Remove unnecesary inclusion of memory-internal.h in
>      cputlb.c
>    exec: Restrict memory-internal.h to system/
> 
>   MAINTAINERS                                | 2 +-
>   {include/exec => system}/memory-internal.h | 6 ------
>   accel/tcg/cputlb.c                         | 1 -
>   system/memory.c                            | 3 ++-
>   system/physmem.c                           | 3 ++-
>   5 files changed, 5 insertions(+), 10 deletions(-)
>   rename {include/exec => system}/memory-internal.h (88%)
> 

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


r~