machine: Use host_memory_backend_is_mapped() in machine_consume_memdev()
commiteef3a7abff8a5eab840868fffd6195d8a2a555d0
authorDavid Hildenbrand <david@redhat.com>
Tue, 2 Nov 2021 16:43:15 +0000 (2 17:43 +0100)
committerPhilippe Mathieu-Daudé <f4bug@amsat.org>
Tue, 18 Jan 2022 09:45:35 +0000 (18 10:45 +0100)
tree3a0c992aa08c0863fb05019583330ad24a7e39d9
parent7bdbf99aa2aac3bf483db6a08d1cea5bf1053c74
machine: Use host_memory_backend_is_mapped() in machine_consume_memdev()

memory_region_is_mapped() is the wrong check, we actually want to check
whether the backend is already marked mapped.

For example, memory regions mapped via an alias, such as NVDIMMs,
currently don't make memory_region_is_mapped() return "true". As the
machine is initialized before any memory devices (and thereby before
NVDIMMs are initialized), this isn't a fix but merely a cleanup.

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Peter Xu <peterx@redhat.com>
Signed-off-by: David Hildenbrand <david@redhat.com>
Message-Id: <20211102164317.45658-2-david@redhat.com>
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
hw/core/machine.c