Messages in this thread | | | Date | Thu, 28 Jan 2016 11:34:40 +0000 | From | Matt Fleming <> | Subject | Re: [PATCH] sched: Make schedstats a runtime tunable that is disabled by default v2 |
| |
On Thu, 28 Jan, at 10:59:25AM, Mel Gorman wrote: > > It wasn't really. The patch increased the function size by enough that > I uninlined it and let the compiler make the decision. In this case, > it should automatically inline but I can leave the inline in.
Letting the compiler make the decision seems fine to me, but I'll let other people chime in with their 'static inline' opinions.
My concern was that you didn't intend to delete the keyword, and it was done by accident. But since that's not the case, it's no big deal.
| |