From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45393) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDgsY-000811-5y for qemu-devel@nongnu.org; Tue, 20 Jan 2015 17:02:34 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YDgsT-0005Rw-SN for qemu-devel@nongnu.org; Tue, 20 Jan 2015 17:02:30 -0500 Received: from mx1.redhat.com ([209.132.183.28]:45688) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDgsT-0005Rf-LY for qemu-devel@nongnu.org; Tue, 20 Jan 2015 17:02:25 -0500 Date: Wed, 21 Jan 2015 00:02:18 +0200 From: "Michael S. Tsirkin" Message-ID: <20150120220218.GB31412@redhat.com> References: <1421769152-23285-1-git-send-email-imammedo@redhat.com> <20150120162907.GA1740@redhat.com> <20150120213818.5c5b3074@igors-macbook-pro.local> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150120213818.5c5b3074@igors-macbook-pro.local> Subject: Re: [Qemu-devel] [PATCH v4 0/7] pc: acpi: various fixes and cleanups List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: claudio.fontana@huawei.com, qemu-devel@nongnu.org, marcel.a@redhat.com On Tue, Jan 20, 2015 at 09:38:18PM +0100, Igor Mammedov wrote: > On Tue, 20 Jan 2015 18:29:07 +0200 > "Michael S. Tsirkin" wrote: > > > On Tue, Jan 20, 2015 at 03:52:25PM +0000, Igor Mammedov wrote: > > > NOTE to maintainer: please update test data (ACPI blobs) in test > > > cases > > > > > > 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 > > > > > > previous revision: > > > [PATCH v3 0/8] pc: acpi: various fixes and cleanups > > > http://lists.gnu.org/archive/html/qemu-devel/2014-12/msg03057.html > > > > > > Igor Mammedov (7): > > > pc: acpi: fix WindowsXP BSOD when memory hotplug is enabled > > > > At least this one is already in pci tree isn't it? > > Could you pls rebase? > > Or if there's a replacement patch, pls send fixup or > > mention old one needs to be retracted in commit log. > I've rebased it on PCI tree, there was a trivial conflict in 4/7 > you can pull result from: > https://github.com/imammedo/qemu/commits/acpi_pci_gen_simplification_v4_PCI That's not a git url. Please just repost, it's easier. > > > > > > > > > pc: acpi-build: cleanup AcpiPmInfo initialization > > > acpi: build_append_nameseg(): add padding if necessary > > > 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 | 464 > > > +++++++++---------------------------- > > > hw/i386/acpi-dsdt-cpu-hotplug.dsl | 1 + > > > include/hw/acpi/acpi-build-utils.h | 23 ++ 5 files changed, 398 > > > insertions(+), 360 deletions(-) create mode 100644 > > > hw/acpi/acpi-build-utils.c create mode 100644 > > > include/hw/acpi/acpi-build-utils.h > > > > > > -- > > > 1.8.3.1