From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44472) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDrI0-0000zP-Lc for qemu-devel@nongnu.org; Wed, 21 Jan 2015 04:09:32 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YDrHv-0000p9-RK for qemu-devel@nongnu.org; Wed, 21 Jan 2015 04:09:28 -0500 Received: from mx1.redhat.com ([209.132.183.28]:41509) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDrHv-0000oh-IG for qemu-devel@nongnu.org; Wed, 21 Jan 2015 04:09:23 -0500 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 (8.14.4/8.14.4) with ESMTP id t0L99L2J008567 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Wed, 21 Jan 2015 04:09:22 -0500 From: Igor Mammedov Date: Wed, 21 Jan 2015 09:09:08 +0000 Message-Id: <1421831353-5507-1-git-send-email-imammedo@redhat.com> Subject: [Qemu-devel] [PATCH v5 0/5] pc: acpi: various fixes and cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: marcel.a@redhat.com, mst@redhat.com NOTE to maintainer: please update test data (ACPI blobs) in test cases changes from v4: * rebased on top of PCI tree, dropping 2 patches that are already there changes from v3: * rename acpi_gen_utils.[ch] to acpi-build-utils.[ch] * copy GLP license block from acpi-build.c * assert on wrong Segcount earlier and extend condition to seg_count > 0 && seg_count <= 255 * drop "pc: acpi: decribe bridge device as not hotpluggable" * keep original logic of creating bridge devices as it was done in 133a2da48 "pc: acpi: generate AML only for PCI0 ..." * if bus is non hotpluggable, add child slots to bus as non hotpluggable as it was done in original code. changes from v2: * codding style fixups * check for SegCount earlier * use hotpluggable device object instead of not hotpluggable for non present devices, and add it only when bus itself is hotpluggable changes from v1: * drop: [PATCH 7/9] acpi: replace opencoded notify codes with named values * use Michael's suggestion to improve build_append_nameseg() * drop long scope names and go back to recursion, but still significantly simplify building of PCI tree this series is an attempt to shave off a bunch of not directly related patches from already big dynamic AML series (although it's dependency for it) Tested: on XPsp3 to WS2012R2 and REHL6/7 guests. Git tree for testing: https://github.com/imammedo/qemu/commits/acpi_pci_gen_simplification_v4 Igor Mammedov (5): pc: acpi-build: cleanup AcpiPmInfo initialization acpi: move generic aml building helpers into dedictated file acpi: add build_append_namestring() helper acpi: drop min-bytes in build_package() pc: acpi-build: simplify PCI bus tree generation hw/acpi/Makefile.objs | 1 + hw/acpi/acpi-build-utils.c | 269 +++++++++++++++++++++ hw/i386/acpi-build.c | 469 ++++++++----------------------------- include/hw/acpi/acpi-build-utils.h | 23 ++ 4 files changed, 397 insertions(+), 365 deletions(-) create mode 100644 hw/acpi/acpi-build-utils.c create mode 100644 include/hw/acpi/acpi-build-utils.h -- 1.8.3.1