On Mon, Jan 22, 2018 at 12:25 PM, Linus Torvalds email@example.com wrote:
On Sun, Jan 21, 2018 at 6:04 PM, Deepa Dinamani firstname.lastname@example.org wrote:
t.tv_nsec -= t.tv_nsec % gran;
This doesn't actuall ywork if tv_nsec is negative.
Which may not be an issue in most cases, but did somebody check utimensat() or whatever?
I checked POSIX again. There is no mention of tv_nsec being positive always for utimes. And, the long term plan is to replace all the callers of timespec_trunc() to use this new api instead for filesystems. So this will need to be fixed. I will fix this and post an update.
WARN(1, "illegal file time granularity: %u", gran);
.. small nit: we generally should use 'invalid' rather than 'illegal'.
Will update this as well.