On Thu 18-08-16 07:32:39, Al Viro wrote: > On Thu, Aug 18, 2016 at 02:09:31AM -0400, Janani Ravichandran wrote: > > > static LIST_HEAD(super_blocks); > > @@ -64,6 +65,7 @@ static unsigned long super_cache_scan(struct shrinker > > *shrink, > > long inodes; > > > > sb = container_of(shrink, struct super_block, s_shrink); > > + trace_mm_shrinker_callback(shrink, sb->s_type->name); > > IOW, we are (should that patch be accepted) obliged to keep the function in > question and the guts of struct shrinker indefinitely.
I am not aware that trace points are considered a stable ABI. Is that documented anywhere? We have changed/removed some of them in the past. If there is a debugging tool parsing them the tool itself is responsible to keep track of any changes. -- Michal Hocko SUSE Labs

