Re: [PATCH] memory: Do not print MR priority in flatview HMP output

[Philippe Mathieu-Daudé]

On 28/12/22 17:26, Mark Burton wrote:
> Is there any chance between 7.1 and 7.2 ‘something’ happened to make it 
> so that Qemu ‘cares more’ about e.g. when memory regions are added/removed?
>
> I seem to get an abort because a memory region has not been completely 
> setup in 7.2 (while it is being flattened actually) - In 7.1 that never 
> seemed to happen….?

I couldn't find any relevant changes in softmmu/. Maybe TCG related?

$ git diff --stat v7.1.0..v7.2.0 -- softmmu/
 softmmu/cpus.c         | 18 +++++++++++++-----
 softmmu/device_tree.c  | 58 
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 softmmu/dirtylimit.c   |  4 ++--
 softmmu/main.c         | 10 +++++-----
 softmmu/memory.c       | 72 
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 softmmu/physmem.c      | 36 ++++++++++++++++++------------------
 softmmu/qemu-seccomp.c | 13 +++++++++++++
 softmmu/qtest.c        | 12 +++++++++---
 softmmu/runstate.c     | 11 ++++++++---
 softmmu/vl.c           | 85 
+++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------
 10 files changed, 249 insertions(+), 70 deletions(-)

$ git diff --stat v7.1.0..v7.2.0 -- accel/tcg/
 accel/tcg/cpu-exec-common.c     |    2 +-
 accel/tcg/cpu-exec.c            |  229 +++++++++++++++------------
 accel/tcg/cputlb.c              |  350 
+++++++++++++++++++++---------------------
 accel/tcg/internal.h            |  101 ++++++++++++
 accel/tcg/meson.build           |    1 +
 accel/tcg/plugin-gen.c          |   22 +--
 accel/tcg/tb-hash.h             |    1 +
 accel/tcg/tb-jmp-cache.h        |   65 ++++++++
 accel/tcg/tb-maint.c            |  704 
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 accel/tcg/tcg-accel-ops-mttcg.c |    5 +-
 accel/tcg/tcg-accel-ops-rr.c    |    9 +-
 accel/tcg/tcg-accel-ops.c       |   98 ++++++++++++
 accel/tcg/tcg-all.c             |   17 +++
 accel/tcg/translate-all.c       | 1249 
+++++++++++++++++++---------------------------------------------------------------------------------------------------------------------------------
 accel/tcg/translator.c          |  138 ++++++++++++-----
 accel/tcg/user-exec.c           |   59 ++++++-
 16 files changed, 1628 insertions(+), 1422 deletions(-)

I'll have a look at

  $ git log -p v7.1.0..v7.2.0 accel/tcg/

> (Putting it all async work seems ‘excessive’ but maybe it’s needed now?)
>
>
> Cheers
> Mark