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

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

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