Add symbols for memory listener priority

[PATCH 0/3] Add symbols for memory listener priority

Isaku Yamahata posted 3 patches 11 months ago

Download series mbox

Failed in applying to current master (apply log)

Maintainers: Cameron Esfahani <dirty@apple.com>, Roman Bolshakov <r.bolshakov@yadro.com>, Paolo Bonzini <pbonzini@redhat.com>, Peter Maydell <peter.maydell@linaro.org>, Stefano Stabellini <sstabellini@kernel.org>, Anthony Perard <anthony.perard@citrix.com>, Paul Durrant <paul@xen.org>, "Michael S. Tsirkin" <mst@redhat.com>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, Elena Ufimtseva <elena.ufimtseva@oracle.com>, Jagannathan Raman <jag.raman@oracle.com>, Peter Xu <peterx@redhat.com>, David Hildenbrand <david@redhat.com>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Reinoud Zandijk <reinoud@netbsd.org>, Sunil Muthuswamy <sunilmut@microsoft.com>

accel/hvf/hvf-accel-ops.c         | 2 +-
accel/kvm/kvm-all.c               | 5 +++--
hw/arm/xen_arm.c                  | 2 +-
hw/i386/xen/xen-hvm.c             | 2 +-
hw/remote/proxy-memory-listener.c | 2 +-
hw/virtio/vhost.c                 | 2 +-
hw/xen/xen-hvm-common.c           | 2 +-
hw/xen/xen_pt.c                   | 4 ++--
include/exec/memory.h             | 4 ++++
target/arm/kvm.c                  | 1 +
target/i386/hax/hax-mem.c         | 2 +-
target/i386/nvmm/nvmm-all.c       | 2 +-
target/i386/whpx/whpx-all.c       | 2 +-
13 files changed, 19 insertions(+), 13 deletions(-)

Expand all Fold all

[PATCH 0/3] Add symbols for memory listener priority

Posted by Isaku Yamahata 11 months ago

The hard-coded value, 10, is used for memory_listener_register().  Add symbolic
values for priority of struct MemoryLister.  Replace those hard-coded values
with symbols.

The background is KVM guest memory[1] or TDX support.  I'd like to add one more
memory listener whose priority is higher than the KVM memory listener.  And I
don't want to hard-code 10 + 1.

[1] KVM gmem patches
https://github.com/sean-jc/linux/tree/x86/kvm_gmem_solo

Isaku Yamahata (3):
  exec/memory: Add symbolic value for memory listener priority for accel
  exec/memory: Add symbol for memory listener priority for dev backend
  exec/memory: Add symbol for the min value of memory listener priority

 accel/hvf/hvf-accel-ops.c         | 2 +-
 accel/kvm/kvm-all.c               | 5 +++--
 hw/arm/xen_arm.c                  | 2 +-
 hw/i386/xen/xen-hvm.c             | 2 +-
 hw/remote/proxy-memory-listener.c | 2 +-
 hw/virtio/vhost.c                 | 2 +-
 hw/xen/xen-hvm-common.c           | 2 +-
 hw/xen/xen_pt.c                   | 4 ++--
 include/exec/memory.h             | 4 ++++
 target/arm/kvm.c                  | 1 +
 target/i386/hax/hax-mem.c         | 2 +-
 target/i386/nvmm/nvmm-all.c       | 2 +-
 target/i386/whpx/whpx-all.c       | 2 +-
 13 files changed, 19 insertions(+), 13 deletions(-)


base-commit: cab35c73be9d579db105ef73fa8a60728a890098
-- 
2.25.1

Re: [PATCH 0/3] Add symbols for memory listener priority

Posted by Philippe Mathieu-Daudé 10 months, 3 weeks ago

On 20/6/23 18:50, Isaku Yamahata wrote:
> The hard-coded value, 10, is used for memory_listener_register().  Add symbolic
> values for priority of struct MemoryLister.  Replace those hard-coded values
> with symbols.
> 
> The background is KVM guest memory[1] or TDX support.  I'd like to add one more
> memory listener whose priority is higher than the KVM memory listener.  And I
> don't want to hard-code 10 + 1.
> 
> [1] KVM gmem patches
> https://github.com/sean-jc/linux/tree/x86/kvm_gmem_solo
> 
> Isaku Yamahata (3):
>    exec/memory: Add symbolic value for memory listener priority for accel
>    exec/memory: Add symbol for memory listener priority for dev backend
>    exec/memory: Add symbol for the min value of memory listener priority

Series:
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

and queued fixing the typos, thanks.