diff mbox series

MAINTAINERS: Fully cover target/i386/*

Message ID 20231030085336.2681386-1-armbru@redhat.com
State New
Headers show
Series MAINTAINERS: Fully cover target/i386/* | expand

Commit Message

Markus Armbruster Oct. 30, 2023, 8:53 a.m. UTC 
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(+)

Comments

Philippe Mathieu-Daudé Oct. 30, 2023, 10:21 a.m. UTC | #1 
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>
Markus Armbruster Nov. 15, 2023, 6:35 a.m. UTC | #2 
Thomas addressed this differently in commit d229996b402 (MAINTAINERS:
Add a general architecture section for x86).  Ignore my patch.
diff mbox series

Patch

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/