exec: Restrict memory-internal.h to system/

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

Philippe Mathieu-Daudé posted 2 patches 2 weeks, 1 day ago

Download series mbox

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%)

Expand all Fold all

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

Posted by Philippe Mathieu-Daudé 2 weeks, 1 day ago

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

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

Posted by Richard Henderson 2 weeks, 1 day ago

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~

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

Posted by David Hildenbrand 2 weeks, 1 day ago

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