On 05/04/2018 04:40 PM, Tetsuo Handa wrote:
> The loop module ignores sysfs_create_group() failure and pretends that
> LOOP_SET_FD request succeeded. I guess that the author of commit
> ee86273062cbb310 ("loop: add some basic read-only sysfs attributes")
> assumed that it is not a fatal error enough to abort LOOP_SET_FD request.IIRC we added sysfs attributes to easily access loop info for a regular user in lsblk command (and perhaps even in udev rules). The ioctl interface was still controlling the loop device, so the failure here was not meant to be fatal. TBH I think was not a great idea. > Do we want to abort LOOP_SET_FD request if sysfs_create_group() failed? I would prefer failure - there are several utilities that expects attributes in sysfs to be valid (for example I print info from here in cryptsetup status if the backing image is an image), so ignoring failure put the system in inconsistent state. Thanks for fixing this, Milan
