Eduardo Habkost <ehabk...@redhat.com> writes: > On Mon, Jun 24, 2019 at 01:22:07PM +0200, Markus Armbruster wrote: >> Daniel P. Berrangé <berra...@redhat.com> writes: >> >> > On Wed, Jun 19, 2019 at 10:10:40PM +0200, Markus Armbruster wrote: >> >> Suggested-by: Paolo Bonzini <pbonz...@redhat.com> >> >> Cc: Eduardo Habkost <ehabk...@redhat.com> >> >> Cc: Marcel Apfelbaum <marcel.apfelb...@gmail.com> >> >> Signed-off-by: Markus Armbruster <arm...@redhat.com> >> >> --- >> >> MAINTAINERS | 16 ++++------------ >> >> 1 file changed, 4 insertions(+), 12 deletions(-) >> > >> > Reviewed-by: Daniel P. Berrangé <berra...@redhat.com> >> > >> >> >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> >> index 1aef0afaf7..f18fddbbbb 100644 >> >> --- a/MAINTAINERS >> >> +++ b/MAINTAINERS >> >> @@ -1268,8 +1268,12 @@ S: Supported >> >> F: hw/core/machine.c >> >> F: hw/core/null-machine.c >> >> F: hw/cpu/cluster.c >> >> +F: numa.c >> >> +F: qom/cpu.c >> > >> > Not a blocker for this patch, but as a later patch, I'd encourage you to >> > move qom/cpu.c somewhere else. Pretty much anywhere else would be better >> > than under qom/ IMHO :-) >> >> Eduardo, any ideas? > > Sorry for the late reply. I suggest hw/core/cpu.c.
Can do. Also: include/qom/cpu.h -> include/hw/core/cpu.h. There's also hw/cpu/ and include/hw/cpu/. I presume they are less suited than hw/core and include/hw/core/.