From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36922) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqdM6-0006tm-6Z for qemu-devel@nongnu.org; Mon, 26 Oct 2015 04:42:15 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZqdM3-00038e-0S for qemu-devel@nongnu.org; Mon, 26 Oct 2015 04:42:14 -0400 Received: from mx1.redhat.com ([209.132.183.28]:34484) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZqdM2-00038P-RA for qemu-devel@nongnu.org; Mon, 26 Oct 2015 04:42:10 -0400 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (Postfix) with ESMTPS id DD157C0AEE27 for ; Mon, 26 Oct 2015 08:42:09 +0000 (UTC) From: Igor Mammedov Date: Mon, 26 Oct 2015 09:42:05 +0100 Message-Id: <1445848925-84796-1-git-send-email-imammedo@redhat.com> Subject: [Qemu-devel] [PATCH] pc: memhp: enforce minimal 128Mb alignment for pc-dimm List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: pbonzini@redhat.com, ehabkost@redhat.com, mst@redhat.com commit aa8580cd "pc: memhp: force gaps between DIMM's GPA" regressed memory hot-unplug for linux guests triggering following BUGON ===== kernel BUG at mm/memory_hotplug.c:703! ... [] acpi_memory_device_remove+0x79/0xa5 [] acpi_bus_trim+0x5a/0x8d [] acpi_device_hotplug+0x1b7/0x418 === BUG_ON(phys_start_pfn & ~PAGE_SECTION_MASK); === reson for it is that x86-64 linux guest supports memory hotplug in chunks of 128Mb and memory section also should be 128Mb aligned. However gaps forced between 128Mb DIMMs with backend's natural alignment of 2Mb make the 2nd and following DIMMs not being aligned on 128Mb boundary as it was originally. To fix regression enforce minimal 128Mb alignment like it was done for PPC. Signed-off-by: Igor Mammedov --- hw/i386/pc.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/hw/i386/pc.c b/hw/i386/pc.c index 3d958ba..cd68169 100644 --- a/hw/i386/pc.c +++ b/hw/i386/pc.c @@ -1610,6 +1610,8 @@ void ioapic_init_gsi(GSIState *gsi_state, const char *parent_name) } } +#define MIN_DIMM_ALIGNMENT (1ULL << 27) /* 128Mb */ + static void pc_dimm_plug(HotplugHandler *hotplug_dev, DeviceState *dev, Error **errp) { @@ -1624,6 +1626,9 @@ static void pc_dimm_plug(HotplugHandler *hotplug_dev, if (memory_region_get_alignment(mr) && pcms->enforce_aligned_dimm) { align = memory_region_get_alignment(mr); + if (pcmc->inter_dimm_gap && (align < MIN_DIMM_ALIGNMENT)) { + align = MIN_DIMM_ALIGNMENT; + } } if (!pcms->acpi_dev) { -- 1.8.3.1