From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:43330) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1fZvHa-0005Nw-4E for qemu-devel@nongnu.org; Mon, 02 Jul 2018 05:38:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1fZvHV-0006iv-7G for qemu-devel@nongnu.org; Mon, 02 Jul 2018 05:38:06 -0400 From: David Hildenbrand Date: Mon, 2 Jul 2018 11:37:51 +0200 Message-Id: <20180702093755.7384-1-david@redhat.com> Subject: [Qemu-devel] [PATCH v3 0/4] pc-dimm: pre_plug "slot" and "addr" assignment List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: qemu-ppc@nongnu.org, Eduardo Habkost , Igor Mammedov , "Michael S . Tsirkin" , Marcel Apfelbaum , Paolo Bonzini , Richard Henderson , Xiao Guangrong , David Gibson , Alexander Graf , Stefan Weil , David Hildenbrand As requested by Igor, assign and verify "slot" and "addr" in the pre_plug handler. Factor out the compatibility handling/configuration for detecting the alignment to be used when searching for an address in guest physical memory for a memory device. This is another part of the original series [PATCH v4 00/14] MemoryDevice: use multi stage hotplug handlers This refactoring is the last step before factoring out pre_plug, plug and unplug logic of memory devices completely into memory-device.c v2 -> v3: - "util/oslib-win32: indicate alignment for qemu_anon_ram_alloc()" -- take allocation granularity into account -- drop comment - "pc-dimm: assign and verify the "addr" property during pre_plug" -- compat_align -> legacy_align -- shortened align detection as proposed by Igor -- trace address asignment in pre_plug v1 -> v2: - dropped "machine: factor out enforce_aligned_dimm into memory_device_align" - dropped "pc-dimm/memory-device: detect alignment internally" - added "util/oslib-win32: indicate alignment for qemu_anon_ram_alloc()" -- indicate the alignment and therefor simplify compat handling for Windows as we can ... - added "pc: drop memory region alignment check for 0" -- ... drop a check for memory_region_get_alignment(mr), allowing us to factor out compat handling for pc DIMM/NVDIMM alignment in a clean way. - "pc-dimm: assign and verify the "addr" property during pre_plug" -- Default to "memory_region_get_alignment(mr)" if no compat alignment has been specified. David Hildenbrand (4): pc-dimm: assign and verify the "slot" property during pre_plug util/oslib-win32: indicate alignment for qemu_anon_ram_alloc() pc: drop memory region alignment check for 0 pc-dimm: assign and verify the "addr" property during pre_plug hw/i386/pc.c | 16 ++++------- hw/mem/pc-dimm.c | 61 +++++++++++++++++++++------------------- hw/ppc/spapr.c | 14 ++++++--- include/hw/mem/pc-dimm.h | 5 ++-- util/oslib-win32.c | 15 ++++++++-- 5 files changed, 63 insertions(+), 48 deletions(-) -- 2.17.1