On Wed, Nov 07, 2018 at 03:10:54PM -0500, Josef Bacik wrote: > In order to make error injection triggering easier make > xfs_buf_ioapply_map STATIC so it's noinline and can be kprobe'ed. So per the whole STATIC discussion how about throwing in an explicit noinline instead? Then again if you really are about error injection we should probably have a real tag here. Either explicit error injection like we have in many places in XFS, or a trace point which should give a really nice hook, too.