Skip to content

Memory Region Naming - take 2

Peter Crosthwaite qemu-devel 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v2 Latest3 patches 7 repliesIn review

Changes in v2 from the cover letter

Constify xen physmap name string (Stefan W review). Change memory_region_name to caching based approach.

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3xen-hvm: Constify stringxen-hvm.c+2 -21 reply
2/3xen: hvm: Abstract away memory region name refxen-hvm.c+6 -30 replies
3/3memory: Lazy init name from QOM name as neededmemory.c+4 -15 replies

Reviews trailers per patch

2 of 3 patches have a Reviewed-by or Acked-by.

PatchStefan Weil
1/3R-b
2/3R-b
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [Qemu-devel] [PATCH memory v2 0/3] Memory Region Naming - take 2

Hi,

Here is an attempt at a proper fix to the memory region naming bugs as reported by Peter. The Xen compile bug and the memory leak.

Regards,
Peter

Changed since v1:
Constify xen physmap name string (Stefan W review). Change memory_region_name to caching based approach.

Peter Crosthwaite (3):
xen-hvm: Constify string
xen: hvm: Abstract away memory region name ref
memory: Lazy init name from QOM name as needed

 memory.c  |  5 ++++-
 xen-hvm.c | 11 +++++++----
 2 files changed, 11 insertions(+), 5 deletions(-)
Signature · 1 line
2.1.0.1.g27b9230