On Mon, May 24 2021, "Cho, Yu-Chen" <a...@suse.com> wrote: Please add a line as to *why* you are splitting this off; that is not clear if you look at this change individually.
> Signed-off-by: Claudio Fontana <cfont...@suse.de> > Signed-off-by: Cho, Yu-Chen <a...@suse.com> > --- > target/s390x/cpu-dump.c | 131 +++++++++++++++++++++++++++++++++++++++ > target/s390x/helper.c | 107 -------------------------------- > target/s390x/meson.build | 1 + > 3 files changed, 132 insertions(+), 107 deletions(-) > create mode 100644 target/s390x/cpu-dump.c > > diff --git a/target/s390x/cpu-dump.c b/target/s390x/cpu-dump.c > new file mode 100644 Not sure whether this is worth an entire new file; but it does not hurt, either. Generally, looks sane. Acked-by: Cornelia Huck <coh...@redhat.com>