bugGNU GRUB - Bugs: bug #66854, search --no-floppy searches floppy

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #66854: search --no-floppy searches floppy

Submitter:  Andreas Klauer <aklauer>
Submitted:  Fri 28 Feb 2025 09:19:25 AM UTC
   
 
Category:  None Severity:  Major
Priority:  5 - Normal Item Group:  None
Status:  None Privacy:  Public
Assigned to:  None Originator Name: 
Open/Closed:  Open Release:  Git master
Release:  Reproducibility:  None
Planned Release:  None

Fri 28 Feb 2025 09:19:25 AM UTC, original submission:  

Transcript:

  grub> set debug=disk
  grub> search --no-floppy --file filename
  […]
  disk: Opening 'fd0'.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: fd0 read failed.
  disk: Closing 'fd0'.

commit 68ba54c229 seems to have changed behavior. When commands/search.c :: iterate_device() encounters fd device and NO_FLOPPY is set, it returns 0 (was 1 before).

However this is executed in context of kern/device.c :: iterate_disk which then proceeds to iterate_partition ... thus the device is opened and checked for partition tables, which adds noticable delay (in Qemu which emulates floppy by default for some reason).

So the goal of skipping the floppy device was not achieved.

Andreas Klauer <aklauer>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by aklauer (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    No changes have been made to this item

    Back to the top

    Powered by Savane 3.15-e6e5.
    Corresponding source code