On Thu, Jul 11, 2013 at 10:31:22AM -0400, Chris Mason wrote: > Do you have benchmark numbers for how much these help? I hesitate to > bring in the likely/unlikely unless we can see it on the benchmarks.
Seconded, I doubt that this particular function gets any measurable speed improvement with the prediction hints. They're eg suitable for branches that almost never happen like error conditions or shortcuts to the exit. There's a config option CONFIG_PROFILE_ANNOTATED_BRANCHES to profile all likely/unlikely, so if you want to see the hint effects yourself feel free to experiment with that. Theres' 'perf branch' patchset, able to gather information from the Branch Trace Store (BTS), http://lwn.net/Articles/444885/ . There probably are functions in btrfs code where even a small improvement could bring some performance, first guess is btrfs_search_slot + the callees, but I've never seen it up in the perf top profile. david -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html