[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH v5 0/6] i386: expose floppy-related
From: |
Laszlo Ersek |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH v5 0/6] i386: expose floppy-related objects in SSDT |
Date: |
Wed, 13 Jan 2016 15:36:18 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 |
Hi Roman,
On 12/30/15 21:11, Roman Kagan wrote:
> Windows on UEFI systems is only capable of detecting the presence and
> the type of floppy drives via corresponding ACPI objects.
I'm late to the party, but please allow me a question:
how did you figure out that UEFI Windows requires this?
In general, what the ACPI specification says is at best a "guideline"
for Windows. So how did you prove this was a requirement for Windows?
Thanks
Laszlo
>
> 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.
>
> 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 <address@hidden>
> Cc: "Michael S. Tsirkin" <address@hidden>
> Cc: Eduardo Habkost <address@hidden>
> Cc: Igor Mammedov <address@hidden>
> Cc: John Snow <address@hidden>
> Cc: Kevin Wolf <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: Richard Henderson <address@hidden>
> Cc: address@hidden
> Cc: address@hidden
> ---
> 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(-)
>
- Re: [Qemu-block] [Qemu-devel] [PATCH v5 0/6] i386: expose floppy-related objects in SSDT,
Laszlo Ersek <=