Page MenuHomeFreeBSD

Silently ignore FS_IOC_FIEMAP
ClosedPublic

Authored by trasz on Jun 18 2019, 9:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 2:04 PM
Unknown Object (File)
Dec 20 2023, 8:31 AM
Unknown Object (File)
Nov 22 2023, 6:05 PM
Unknown Object (File)
Nov 6 2023, 7:13 PM
Unknown Object (File)
Oct 24 2023, 9:32 AM
Unknown Object (File)
Jul 10 2023, 11:37 AM
Unknown Object (File)
Dec 26 2022, 10:39 PM
Subscribers

Details

Summary

Silently ignore FS_IOC_FIEMAP - man(1) calls it for no good reason.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emaste added a subscriber: emaste.

Minor point, I wouldn't call this "silently ignored" - the console message will be gone but we still return error, except a different error.

IMO I would make the commit message headline something like Return ENOTSUP for Linux FS_IOC_FIEMAP ioctl and in the body mention the man(1) issue and the side effect of avoiding the console spam.

sys/compat/linux/linux_ioctl.c
3584 ↗(On Diff #58777)

aside, this function could use some comments e.g. when this case is expected to be used vs. the handler lists above

This revision is now accepted and ready to land.Jun 19 2019, 2:21 PM
This revision was automatically updated to reflect the committed changes.