Re: [PATCH 13/13] system: Rename softmmu/ directory as system/

[BALATON Zoltan]

On Wed, 4 Oct 2023, Philippe Mathieu-Daudé wrote:
> On 4/10/23 11:06, Philippe Mathieu-Daudé wrote:
> > The softmmu/ directory contains files specific to system
> > emulation. Rename it as system/. Update meson rules, the
> > MAINTAINERS file and all the documentation and comments.
> >
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > ---
>
> >   include/sysemu/runstate-action.h        |  2 +-
>
> >   rename {softmmu => system}/timers-state.h (100%)
> >   create mode 100644 system/trace.h
> >   rename {softmmu => system}/arch_init.c (100%)
> >   rename {softmmu => system}/async-teardown.c (100%)
> >   rename {softmmu => system}/balloon.c (100%)
> >   rename {softmmu => system}/bootdevice.c (100%)
> >   rename {softmmu => system}/cpu-throttle.c (100%)
> >   rename {softmmu => system}/cpu-timers.c (100%)
> >   rename {softmmu => system}/cpus.c (100%)
> >   rename {softmmu => system}/datadir.c (100%)
> >   rename {softmmu => system}/device_tree.c (100%)
> >   rename {softmmu => system}/dirtylimit.c (100%)
> >   rename {softmmu => system}/dma-helpers.c (100%)
> >   rename {softmmu => system}/globals.c (100%)
> >   rename {softmmu => system}/icount.c (100%)
> >   rename {softmmu => system}/ioport.c (100%)
> >   rename {softmmu => system}/main.c (100%)
> >   rename {softmmu => system}/memory.c (100%)
> >   rename {softmmu => system}/memory_mapping.c (100%)
> >   rename {softmmu => system}/physmem.c (99%)
> >   rename {softmmu => system}/qdev-monitor.c (100%)
> >   rename {softmmu => system}/qemu-seccomp.c (100%)
> >   rename {softmmu => system}/qtest.c (100%)
> >   rename {softmmu => system}/rtc.c (100%)
> >   rename {softmmu => system}/runstate-action.c (100%)
> >   rename {softmmu => system}/runstate-hmp-cmds.c (100%)
> >   rename {softmmu => system}/runstate.c (100%)
> >   rename {softmmu => system}/tpm-hmp-cmds.c (100%)
> >   rename {softmmu => system}/tpm.c (100%)
> >   rename {softmmu => system}/vl.c (100%)
> >   rename {softmmu => system}/watchpoint.c (100%)
> >   rename {softmmu => system}/meson.build (100%)
> >   rename {softmmu => system}/trace-events (100%)
>
> Maybe I should also move include/sysemu/ -> include/system/
> to keep it coherent?

I think I've proposed some years ago to rename softmmu to sysemu because 
besides actual softmmu code it conrains a lot of system emulation. Calling 
it system is also similar, maybe less specific than sysemu but basically 
the same.

Regards,
BALATON Zoltan