[PATCH] MAINTAINERS: Fully cover target/i386/*

Markus Armbruster posted 1 patch 1 year ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231030085336.2681386-1-armbru@redhat.com
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
[PATCH] MAINTAINERS: Fully cover target/i386/*
Posted by Markus Armbruster 1 year ago
Many files in target/i386/ are not covered:

    target/i386/arch_dump.c
    target/i386/arch_memory_mapping.c
    target/i386/cpu-dump.c
    target/i386/cpu-internal.h
    target/i386/cpu-param.h
    target/i386/cpu-qom.h
    target/i386/cpu-sysemu.c
    target/i386/cpu.c
    target/i386/cpu.h
    target/i386/cpuid_register
    target/i386/gdbstub.c
    target/i386/helper.c
    target/i386/helper.h
    target/i386/host-cpu.c
    target/i386/host-cpu.h
    target/i386/machine.c
    target/i386/meson.build
    target/i386/monitor.c
    target/i386/ops_sse.h
    target/i386/svm.h
    target/i386/trace-events
    target/i386/trace.h
    target/i386/xsave_helper.c

Similar files in other target/$t/ are covered by "$t TCG CPUs".  Do
the same here, taking care not to hijack files that are already
covered by other sections.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index cd8d6b140f..60020ac734 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -396,6 +396,8 @@ M: Paolo Bonzini <pbonzini@redhat.com>
 M: Richard Henderson <richard.henderson@linaro.org>
 M: Eduardo Habkost <eduardo@habkost.net>
 S: Maintained
+F: target/i386/*
+X: target/i386/sev*
 F: target/i386/tcg/
 F: tests/tcg/i386/
 F: tests/tcg/x86_64/
-- 
2.41.0
Re: [PATCH] MAINTAINERS: Fully cover target/i386/*
Posted by Markus Armbruster 1 year ago
Thomas addressed this differently in commit d229996b402 (MAINTAINERS:
Add a general architecture section for x86).  Ignore my patch.
Re: [PATCH] MAINTAINERS: Fully cover target/i386/*
Posted by Philippe Mathieu-Daudé 1 year ago
On 30/10/23 09:53, Markus Armbruster wrote:
> Many files in target/i386/ are not covered:
> 
>      target/i386/arch_dump.c
>      target/i386/arch_memory_mapping.c
>      target/i386/cpu-dump.c
>      target/i386/cpu-internal.h
>      target/i386/cpu-param.h
>      target/i386/cpu-qom.h
>      target/i386/cpu-sysemu.c
>      target/i386/cpu.c
>      target/i386/cpu.h
>      target/i386/cpuid_register
>      target/i386/gdbstub.c
>      target/i386/helper.c
>      target/i386/helper.h
>      target/i386/host-cpu.c
>      target/i386/host-cpu.h
>      target/i386/machine.c
>      target/i386/meson.build
>      target/i386/monitor.c
>      target/i386/ops_sse.h
>      target/i386/svm.h
>      target/i386/trace-events
>      target/i386/trace.h
>      target/i386/xsave_helper.c
> 
> Similar files in other target/$t/ are covered by "$t TCG CPUs".  Do
> the same here, taking care not to hijack files that are already
> covered by other sections.
> 
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>   MAINTAINERS | 2 ++
>   1 file changed, 2 insertions(+)

Tested-by: Philippe Mathieu-Daudé <philmd@linaro.org>