Re: [Qemu-devel] [PATCH v2] Replace bdrv_* to bdrv_aio_* functions in pio mode in fdc.c.

[Stefan Hajnoczi]

On Tue, Apr 3, 2012 at 4:33 PM, Stefan Hajnoczi <address@hidden> wrote:
> On Mon, Apr 2, 2012 at 2:08 PM, Paolo Bonzini <address@hidden> wrote:
>> Il 02/04/2012 14:07, Stefan Hajnoczi ha scritto:
>>> On Sat, Mar 31, 2012 at 09:15:10PM +0800, Li Zhi Hui wrote:
>>>> Replace bdrv_* to bdrv_aio_* functions in pio mode in fdc.c.
>>>>
>>>> Signed-off-by: Li Zhi Hui <address@hidden>
>>>> ---
>>>>  hw/fdc.c |  123 
>>>> +++++++++++++++++++++++++++++++++++++++++++++-----------------
>>>>  1 files changed, 89 insertions(+), 34 deletions(-)
>>>
>>> How can I test this patch?
>>
>> Looks like one of floppy=nodma or floppy.floppy=nodma should do it.
>
> I'm getting the following output:
> $ cat /dev/fd0 >/dev/null
> floppy0: long rw: 24 instead of 12
> rs=1 s=1
> rh=0 h=0
> rt=1 t=0
> heads=2 eoc=0
> spt=18 st=0 ss=1
> in_sector_offset=0
>
> It looks like pio mode doesn't even work with the Linux floppy driver today!

I just want to clarify that I tested unmodified qemu.git/master.  This
means hw/fdc.c doesn't even work with the Linux floppy driver today.

Stefan