On Fri, May 18, 2018 at 01:17:16PM +0800, Qu Wenruo wrote: > > > On 2018年05月18日 11:00, Liu Bo wrote: > > If parent_transid "0" is passed to btrfs_buffer_uptodate(), > > btrfs_buffer_uptodate() is equivalent to extent_buffer_uptodate(), but > > extent_buffer_uptodate() is preferred since we don't have to look into > > verify_parent_transid(). > > > > Signed-off-by: Liu Bo <bo.liu@xxxxxxxxxxxxxxxxx> > > Reviewed-by: Qu Wenruo <wqu@xxxxxxxx> > > And considering how little extra work we do in btrfs_buffer_uptodate(), > what about make it an inline function? The actual picture as the compiler sees the function can be quite different, eg if there are static functions called from that one, they could get inlined and suddenly btrfs_buffer_uptodate becomes btrfs_buffer_uptodate + verify_parent_transid. IOW, leave the inlining on the compiler and use 'static inline' only for really lightweight functions. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
