On Wednesday 02 March 2016 23:50:01 Deepa Dinamani wrote: > + /* > + * Assign a default empty range [MAX_VFS_TIME, MIN_VFS_TIME]. > + * This will help VFS detect filesystems that do not populate > + * these fields in the superblock. > + */ > + s->s_time_min = MAX_VFS_TIME; > + s->s_time_max = MIN_VFS_TIME; > s->cleancache_poolid = CLEANCACHE_NO_POOL; I think this is still less clear than it should be, even with the comment explaining it. As I just replied to Dave, how about initializing both to -1 instead? Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html