Re: [PATCH 1/3] kref: Inline all functions

From: Greg KH
Date: Tue Dec 13 2011 - 12:33:25 EST


On Tue, Dec 13, 2011 at 10:36:20AM +0100, Peter Zijlstra wrote:
> On Mon, 2011-12-12 at 14:11 -0800, Greg KH wrote:
> > Nice, but this breaks the build:
>
> Sorry about that, this one includes the missing include and just booted
> on my test-box.

Thanks, that worked.

>
> ---
> Subject: kref: Inline all functions
> From: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
> Date: Sat Dec 10 11:29:57 CET 2011
>
> These are tiny functions, there's no point in having them out-of-line.
>
> Cc: adobriyan@xxxxxxxxx
> Cc: eric.dumazet@xxxxxxxxx
> Cc: mingo@xxxxxxx
> Signed-off-by: Peter Zijlstra <a.p.zijlstra@xxxxxxxxx>
> Link: http://lkml.kernel.org/n/tip-8eccvi2ur2fzgi00xdjlbf5z@xxxxxxxxxxxxxx
> ---
> include/linux/kref.h | 80 +++++++++++++++++++++++++++++++++++++++---
> lib/Makefile | 2 -
> lib/kref.c | 97 ---------------------------------------------------
> 3 files changed, 76 insertions(+), 103 deletions(-)
> Index: linux-2.6/include/linux/kref.h
> ===================================================================

What's this stuff for, we aren't using cvs anymore :)

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/