Re: [PATCH 0/2] libblkid: don't use O_NONBLOCK for floppy

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On 14/12/2021 11:25, Jiri Kosina wrote:
On Thu, 9 Dec 2021, Vladimir Sementsov-Ogievskiy wrote:

Good day everyone!

The commit "floppy: reintroduce O_NDELAY fix" was removed from kernel,
so we faced the bug described and discussed here:
https://bugzilla.suse.com/show_bug.cgi?id=1181018

Discussion in kernel list on reverting the commit:
https://www.spinics.net/lists/stable/msg493061.html

In short, I can quote Jiri Kosina's comment:

    opening floppy device node with O_NONBLOCK is asking for all kinds
    of trouble

So opening floppy with O_NONBLOCK in blkid leads to failure of blkid,
probable failure of mount and unpleasant error messages in dmesg (see
also patch 02 for details).

Vladimir Sementsov-Ogievskiy (2):
   libblkid: introduce blkid_safe_open
   libblkid: reopen floppy without O_NONBLOCK
Ah, thanks, finally someone found the motivation to look into what it'd
take to fix this properly in libblkid. FWIW,

	Acked-by: Jiri Kosina <jkosina@xxxxxxx>

for the changes.

As the one who was affected by the issue (having some old scripts that
use floppy images to inject metadata as data source for cloud-init
instead of CD-Rom images), I definitely appreciate this being taken
care of. Thank you!

Changes look good to me, but I guess best would be to test things.
Let me know if you need me to do this ...

Best,

--
Kurt Garloff <kurt@xxxxxxxxxx>
Cologne, Germany




[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux