From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49463) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aDuyA-0002pm-VP for qemu-devel@nongnu.org; Tue, 29 Dec 2015 09:09:47 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aDuy9-0002cQ-SB for qemu-devel@nongnu.org; Tue, 29 Dec 2015 09:09:46 -0500 Date: Tue, 29 Dec 2015 15:09:36 +0100 From: Igor Mammedov Message-ID: <20151229150936.53a6e4bf@nial.brq.redhat.com> In-Reply-To: <1451055852-30910-1-git-send-email-rkagan@virtuozzo.com> References: <1451055852-30910-1-git-send-email-rkagan@virtuozzo.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 0/4] i386: expose floppy-related objects in SSDT List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Roman Kagan Cc: Kevin Wolf , Eduardo Habkost , qemu-block@nongnu.org, "Michael S. Tsirkin" , qemu-stable@nongnu.org, qemu-devel@nongnu.org, Paolo Bonzini , John Snow , Richard Henderson On Fri, 25 Dec 2015 18:04:08 +0300 Roman Kagan wrote: > Windows on UEFI systems is only capable of detecting the presence and > the type of floppy drives via corresponding ACPI objects. > > Those objects are added in the last patch of the series; the three > preceding ones pave the way to it, by making the necessary data > public and by moving the whole floppy drive controller description into > runtime-generated SSDT. > > Note that the series conflicts with Igor's patchset for dynamic DSDT, in > particular, with "[PATCH 50/74] pc: acpi: move FDC0 device from DSDT > to SSDT"; I haven't managed to avoid that while trying to meet > maintainer's comments. To remove conflicts and to make it more suitable for stable, I'd drop "2/4 i386/acpi: make floppy controller object dynamic" and split test blob out of "4/4 i386: populate floppy drive information in SSDT" into a separate patch so one can see effects of applying 4/4 and then update blobs if resulting ASL diff is as expected. > > Roman Kagan (4): > i386/pc: expose identifying the floppy controller > i386/acpi: make floppy controller object dynamic > expose floppy drive geometry and CMOS type > i386: populate floppy drive information in SSDT > > Signed-off-by: Roman Kagan > Cc: "Michael S. Tsirkin" > Cc: Eduardo Habkost > Cc: Igor Mammedov > Cc: John Snow > Cc: Kevin Wolf > Cc: Paolo Bonzini > Cc: Richard Henderson > Cc: qemu-block@nongnu.org > Cc: qemu-stable@nongnu.org > --- > changes since v3: > - make FDC object fully dynamic in a separate patch > - split out support patches > - include test data updates with the respective patches to maintain > bisectability > > changes since v2: > - explicit endianness for buffer data > - reorder code to reduce conflicts with dynamic DSDT patchset > - update test data > > > hw/block/fdc.c | 11 +++++ > hw/i386/acpi-build.c | 92 ++++++++++++++++++++++++++++++++++++ > hw/i386/acpi-dsdt-isa.dsl | 18 ------- > hw/i386/acpi-dsdt.dsl | 1 - > hw/i386/pc.c | 46 ++++++++++-------- > hw/i386/q35-acpi-dsdt.dsl | 7 +-- > include/hw/block/fdc.h | 2 + > include/hw/i386/pc.h | 3 ++ > tests/acpi-test-data/pc/DSDT | Bin 3028 -> 2946 bytes > tests/acpi-test-data/pc/SSDT | Bin 2486 -> 2635 bytes > tests/acpi-test-data/pc/SSDT.bridge | Bin 4345 -> 4494 bytes > tests/acpi-test-data/q35/DSDT | Bin 7666 -> 7578 bytes > 12 files changed, 137 insertions(+), 43 deletions(-) >