[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 09/12] fdc: add physical disk sizes
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v4 09/12] fdc: add physical disk sizes |
Date: |
Wed, 20 Jan 2016 18:06:58 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 01/20/2016 05:48 PM, Eric Blake wrote:
> On 01/19/2016 11:51 PM, John Snow wrote:
>> 2.88MB capable drives can accept 1.44MB floppies,
>> for instance. To rework the pick_geometry function,
>> we need to know if our current drive can even accept
>> the type of disks we're considering.
>>
>> NB: This allows us to distinguish between all of the
>> "total sectors" collisions between 1.20MB and 1.44MB
>> diskette types, by using the physical drive size as a
>> differentiator.
>>
>> Signed-off-by: John Snow <address@hidden>
>> ---
>> hw/block/fdc.c | 40 ++++++++++++++++++++++++++++++++--------
>> 1 file changed, 32 insertions(+), 8 deletions(-)
>>
>
>> +/* In many cases, the total sector size of a format is enough to uniquely
>> + * identify it. However, there are some total sector collisions between
>> + * formats of different physical size, and these are noted below by
>> + * highlighting the total sector size for entries with collisions. */
>> static const FDFormat fd_formats[] = {
>> /* First entry is default format */
>> /* 1.44 MB 3"1/2 floppy disks */
>> - { FLOPPY_DRIVE_TYPE_144, 18, 80, 1, FDRIVE_RATE_500K, },
>> - { FLOPPY_DRIVE_TYPE_144, 20, 80, 1, FDRIVE_RATE_500K, },
>> + { FLOPPY_DRIVE_TYPE_144, 18, 80, 1, FDRIVE_RATE_500K, }, /* 3.5" 2880 */
>> + { FLOPPY_DRIVE_TYPE_144, 20, 80, 1, FDRIVE_RATE_500K, }, /* 3.5" 3200 */
>> { FLOPPY_DRIVE_TYPE_144, 21, 80, 1, FDRIVE_RATE_500K, },
>> { FLOPPY_DRIVE_TYPE_144, 21, 82, 1, FDRIVE_RATE_500K, },
>> { FLOPPY_DRIVE_TYPE_144, 21, 83, 1, FDRIVE_RATE_500K, },
>> @@ -85,7 +95,7 @@ static const FDFormat fd_formats[] = {
>> { FLOPPY_DRIVE_TYPE_288, 44, 80, 1, FDRIVE_RATE_1M, },
>> { FLOPPY_DRIVE_TYPE_288, 48, 80, 1, FDRIVE_RATE_1M, },
>> /* 720 kB 3"1/2 floppy disks */
>> - { FLOPPY_DRIVE_TYPE_144, 9, 80, 1, FDRIVE_RATE_250K, },
>> + { FLOPPY_DRIVE_TYPE_144, 9, 80, 1, FDRIVE_RATE_250K, }, /* 3.5" 1400 */
>
> Typo? I think you meant 1440.
>
Good catch.
>> +__attribute__((__unused__))
>> +static FDriveSize drive_size(FloppyDriveType drive)
>> +{
>> + switch (drive) {
>> + case FLOPPY_DRIVE_TYPE_120:
>> + return FDRIVE_SIZE_525;
>> + case FLOPPY_DRIVE_TYPE_144:
>> + case FLOPPY_DRIVE_TYPE_288:
>> + return FDRIVE_SIZE_350;
>> + default:
>> + return FDRIVE_SIZE_UNKNOWN;
>> + }
>> +}
>
> With that fixed,
> Reviewed-by: Eric Blake <address@hidden>
>
Thanks,
--js
- [Qemu-devel] [PATCH v4 12/12] fdc: change auto fallback drive for ISA FDC to 288, (continued)
- [Qemu-devel] [PATCH v4 12/12] fdc: change auto fallback drive for ISA FDC to 288, John Snow, 2016/01/20
- [Qemu-devel] [PATCH v4 01/12] fdc: move pick_geometry, John Snow, 2016/01/20
- [Qemu-devel] [PATCH v4 05/12] fdc: Throw an assertion on misconfigured fd_formats table, John Snow, 2016/01/20
- [Qemu-devel] [PATCH v4 08/12] fdc: add drive type option, John Snow, 2016/01/20
- [Qemu-devel] [PATCH v4 09/12] fdc: add physical disk sizes, John Snow, 2016/01/20
- Re: [Qemu-devel] [Qemu-block] [PATCH v4 00/12] fdc: fix 2.88mb floppy diskette support, Denis V. Lunev, 2016/01/20