On Wed, Jul 06, 2011 at 11:12:30AM -0700, Eric Gouriou wrote: > > In all the trace points that capture the logical block/physical > block/length of an extent, it would be nicer to pass the extent itself > as a parameter and perform the extraction in the TP_fast_assign() > block. This requires adding an #include <ext4_extents.h> above the > include of <trace/trace_events/ext4.h> in fs/ext4/super.c in order to > have ext4_ext_pblock() and ext4_ext_get_actual_len() available. > > By itself this should not be ground to respin the whole patch series, > a follow-on cleanup patch would do (I volunteer to create one if > desired). Fortunately this wouldn't cause the trace record format to > change, so it wouldn't cause any backward/forward compatibility issue. Good suggestion, I've made this change. - 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
