« Previous | Next » 

Revision 1f69c2b0

ID1f69c2b022710222ff0379678e49f8bfb6c91233

Added by Markus Armbruster almost 12 years ago

fdc: Drop broken code for user-defined floppy geometry

bdrv_get_floppy_geometry_hint() fails to store through its parameter
drive when bs has a geometry hint. Makes fd_revalidate() assign
random crap to drv->drive.

Has been broken that way for ages. Harmless, because:

  • The only way to set a geometry hint is -drive if=none,cyls=...
    Since commit c219331e, probably unintentional.
  • The only use of drv->drive is as argument to another
    bdrv_get_floppy_geometry_hint(). Which doesn't use it, since the
    geometry hint is still there.

Drop the broken code, ignore -drive parameter cyls, heads and secs for
floppies even with if=none, just like before commit c219331e. Matches
-help, which explains cyls, heads, secs as "hard disk physical
geometry".

Signed-off-by: Markus Armbruster <>
Signed-off-by: Kevin Wolf <>

Files

  • added
  • modified
  • copied
  • renamed
  • deleted

View differences