On Tue, Oct 27, 2009 at 01:48:49PM +0100, Jan Kara wrote: > + /* > + * Transactions that contain inode's metadata needed to complete > + * fsync and fdatasync, respectively. > + */ > + atomic_t i_sync_tid; > + atomic_t i_datasync_tid; Why do we need an atomic_t here at all? It's not clear it's necessary. What specific race are you worried about? - 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