On Tue, May 10, 2011 at 06:48:19PM +0800, Tao Ma wrote: > From: Tao Ma <boyu.mt@xxxxxxxxxx> > > t_max_wait is added in commit 8e85fb3f to indicate how long we > were waiting for new transaction to start. In commit 6d0bf005, > it is moved to another function named update_t_max_wait to > avoid a build warning. But the wrong thing is that the original > 'ts' is initialized in the start of function start_this_handle > and we can calculate t_max_wait in the right way. while with > this change, ts is initialized within the function and t_max_wait > can never be calculated right. > > This patch moves the initialization of ts to the original beginning > of start_this_handle and pass it to function update_t_max_wait so > that it can be calculated right and the build warning is avoided also. > > Cc: Jan Kara <jack@xxxxxxx> > Cc: "Theodore Ts'o" <tytso@xxxxxxx> > Signed-off-by: Tao Ma <boyu.mt@xxxxxxxxxx> Thanks, added to the ext4 tree. - 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
