On Mon, Jan 19, 2015 at 06:21:31PM +0400, Dmitry Monakhov wrote: > > + if (flags & I_DIRTY_INODE) > > + flags &= ~I_DIRTY_TIME; > > + dirtytime = flags & I_DIRTY_TIME; > TYPO? 'dirtytime' is always false because you have already cleared that bit. > Probably you want to do that: > dirtytime = flags & I_DIRTY_TIME; > if (flags & I_DIRTY_INODE) > flags &= ~I_DIRTY_TIME; Nice catch; I screwed that up in the -v7 version of the patch. I've fixed it in the -v8 version that I just sent out. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html