From a1ff8ae0666ffcbe78ae7e28812dd30db6bb7131 Mon Sep 17 00:00:00 2001 From: Marcel Apfelbaum Date: Mon, 16 Sep 2013 11:21:14 +0300 Subject: memory: Change MemoryRegion priorities from unsigned to signed When memory regions overlap, priority can be used to specify which of them takes priority. By making the priority values signed rather than unsigned, we make it more convenient to implement a situation where one "background" region should appear only where no other region exists: rather than having to explicitly specify a high priority for all the other regions, we can let them take the default (zero) priority and specify a negative priority for the background region. Reviewed-by: Peter Maydell Acked-by: Michael S. Tsirkin Signed-off-by: Marcel Apfelbaum Signed-off-by: Michael S. Tsirkin --- memory.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'memory.c') diff --git a/memory.c b/memory.c index 5a10fd0bde..f49d31ae72 100644 --- a/memory.c +++ b/memory.c @@ -1473,7 +1473,7 @@ void memory_region_add_subregion(MemoryRegion *mr, void memory_region_add_subregion_overlap(MemoryRegion *mr, hwaddr offset, MemoryRegion *subregion, - unsigned priority) + int priority) { subregion->may_overlap = true; subregion->priority = priority; @@ -1506,7 +1506,7 @@ void memory_region_set_enabled(MemoryRegion *mr, bool enabled) void memory_region_set_address(MemoryRegion *mr, hwaddr addr) { MemoryRegion *parent = mr->parent; - unsigned priority = mr->priority; + int priority = mr->priority; bool may_overlap = mr->may_overlap; if (addr == mr->addr || !parent) { -- cgit v1.2.1