[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 07/10] block: add a transfer rate for floppy
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 07/10] block: add a transfer rate for floppy types |
Date: |
Fri, 27 Jan 2012 09:33:39 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) |
Kevin Wolf <address@hidden> writes:
> Am 21.01.2012 20:02, schrieb Blue Swirl:
>> On Mon, Jan 16, 2012 at 10:18, Kevin Wolf <address@hidden> wrote:
>>> Am 15.01.2012 08:51, schrieb Hervé Poussineau:
>>>> Floppies must be read at a specific transfer rate, depending of its own
>>>> format.
>>>> Update floppy description table to include required transfer rate.
>>>>
>>>> Signed-off-by: Hervé Poussineau <address@hidden>
>>>> ---
>>>> block.c | 74
>>>> ++++++++++++++++++++++++++++++++-----------------------------
>>>> block.h | 10 +++++++-
>>>> hw/fdc.c | 3 +-
>>>> hw/pc.c | 3 +-
>>>
>>> Meh. Having any floppy-specific logic in the block layer is wrong. We
>>> need to finally get this moved into fdc.c.
>>
>> Well, actually this code was moved recently from fdc.c to block.c
>> (5bbdbb4676d17e782ae83055bac58e0751b25e4b). The other geometry
>> guessing functions (ATA CHS) are also there. If we supported native
>> floppy (or ATA) pass trough, the geometry would have to be read from
>> the host device, so I think it's logical to keep that in block level
>> instead of all devices. Maybe we could also split block.c into
>> block-fdc.c, block-ata.c etc.
>
> The geometry is guest state, so it shouldn't be in the block layer,
> which deals with host state.
Fully agree.
I tried to tackle geometry in my quest to untangle block device host and
guest state, but didn't get far.
> Maybe we could need some hw/block.c that
> deals with guest state concepts that are shared between multiple device.
> Images or passthrough backends could provide defaults.
>
> I'll admit that not having an obvious place for media (we only have it
> for images and guest devices) doesn't make the design decisions easier.
Yup.
- [Qemu-devel] [PATCH v2 05/10] fdc: handle read-only floppies (abort early on write commands), (continued)
[Qemu-devel] [PATCH v2 10/10] fdc: DIR (Digital Input Register) should return status of current drive..., Hervé Poussineau, 2012/01/15
[Qemu-devel] [PATCH v2 08/10] fdc: check if media rate is correct before doing any transfer, Hervé Poussineau, 2012/01/15
[Qemu-devel] [PATCH v2 09/10] fdc: fix seek command, which shouldn't check tracks, Hervé Poussineau, 2012/01/15