Messages in this thread | | | Date | Thu, 15 Nov 2012 18:29:45 +0900 | From | Kamezawa Hiroyuki <> | Subject | Re: [PATCH 4/7] memcg: replace __always_inline with plain inline |
| |
(2012/11/15 11:54), Glauber Costa wrote: > Following the pattern found in the allocators, where we do our best to > the fast paths function-call free, all the externally visible functions > for kmemcg were marked __always_inline. > > It is fair to say, however, that this should be up to the compiler. We > will still keep as much of the flag testing as we can in memcontrol.h to > give the compiler the option to inline it, but won't force it. > > I tested this with 4.7.2, it will inline all three functions anyway when > compiling with -O2, and will refrain from it when compiling with -Os. > This seems like a good behavior. > > Signed-off-by: Glauber Costa <glommer@parallels.com> > CC: Michal Hocko <mhocko@suse.cz> > CC: Kamezawa Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com> > CC: Johannes Weiner <hannes@cmpxchg.org> > CC: Andrew Morton <akpm@linux-foundation.org>
I'm O.K. with this. Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
| |