From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55536) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aH8Fg-0004ef-Qv for qemu-devel@nongnu.org; Thu, 07 Jan 2016 05:57:09 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aH8Ff-0006O9-OJ for qemu-devel@nongnu.org; Thu, 07 Jan 2016 05:57:08 -0500 Date: Thu, 7 Jan 2016 12:56:58 +0200 From: "Michael S. Tsirkin" Message-ID: <20160107125636-mutt-send-email-mst@redhat.com> References: <1451506316-31975-1-git-send-email-rkagan@virtuozzo.com> <20160106150440.107c5690@nial.brq.redhat.com> <20160107125533-mutt-send-email-mst@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160107125533-mutt-send-email-mst@redhat.com> Subject: Re: [Qemu-devel] [PATCH v5 0/6] i386: expose floppy-related objects in SSDT List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: Kevin Wolf , Eduardo Habkost , qemu-block@nongnu.org, qemu-devel@nongnu.org, qemu-stable@nongnu.org, Roman Kagan , Paolo Bonzini , John Snow , Richard Henderson On Thu, Jan 07, 2016 at 12:56:09PM +0200, Michael S. Tsirkin wrote: > On Wed, Jan 06, 2016 at 03:04:40PM +0100, Igor Mammedov wrote: > > On Wed, 30 Dec 2015 23:11:50 +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 patch 5; the 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 v2 27/51] pc: acpi: move FDC0 device from DSDT > > > to SSDT"; I haven't managed to avoid that while trying to meet > > > maintainer's comments. > > > > Tested with XPsp3 WS2008R2 WS2012R2, no regressions so far it boots fine and can read floppy. > > > > So for whole series: > > Reviewed-by: Igor Mammedov > > > > Igor, could you pls rebase this on top of your patches? > I've merged them in my tree. Pls remember to keep the author information intact though. > > > Roman Kagan (6): > > > i386/pc: expose identifying the floppy controller > > > i386/acpi: make floppy controller object dynamic > > > tests/acpi: update test data > > > expose floppy drive geometry and CMOS type > > > i386: populate floppy drive information in SSDT > > > tests/acpi: update test data > > > > > > 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 v4: > > > - re-split out code changes from test data updates > > > > > > 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(-) > > >