Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756614AbcDLQte (ORCPT ); Tue, 12 Apr 2016 12:49:34 -0400 Received: from mx6-phx2.redhat.com ([209.132.183.39]:44077 "EHLO mx6-phx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751856AbcDLQtc (ORCPT ); Tue, 12 Apr 2016 12:49:32 -0400 Date: Tue, 12 Apr 2016 12:49:31 -0400 (EDT) From: Bob Peterson To: Denys Vlasenko Cc: Steven Whitehouse , cluster-devel@redhat.com, linux-kernel@vger.kernel.org Message-ID: <897828634.50168035.1460479771845.JavaMail.zimbra@redhat.com> In-Reply-To: <1460141926-13069-4-git-send-email-dvlasenk@redhat.com> References: <1460141926-13069-1-git-send-email-dvlasenk@redhat.com> <1460141926-13069-4-git-send-email-dvlasenk@redhat.com> Subject: Re: [PATCH] fs/gfs2/glock.c: Deinline do_error, save 1856 bytes MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Originating-IP: [10.3.113.25] X-Mailer: Zimbra 8.0.6_GA_5922 (ZimbraWebClient - FF45 (Linux)/8.0.6_GA_5922) Thread-Topic: fs/gfs2/glock.c: Deinline do_error, save 1856 bytes Thread-Index: AUeheMc9PaQuCNS9EibdVVE9ZYCgLg== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1098 Lines: 41 ----- Original Message ----- > This function compiles to 522 bytes of machine code. > > Error paths are not very time critical. > > Signed-off-by: Denys Vlasenko > CC: Steven Whitehouse > CC: Bob Peterson > CC: cluster-devel@redhat.com > CC: linux-kernel@vger.kernel.org > --- > fs/gfs2/glock.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/fs/gfs2/glock.c b/fs/gfs2/glock.c > index 6539131..c3d5172 100644 > --- a/fs/gfs2/glock.c > +++ b/fs/gfs2/glock.c > @@ -218,7 +218,7 @@ static void gfs2_holder_wake(struct gfs2_holder *gh) > * > */ > > -static inline void do_error(struct gfs2_glock *gl, const int ret) > +static void do_error(struct gfs2_glock *gl, const int ret) > { > struct gfs2_holder *gh, *tmp; > > -- > 2.1.0 > > Hi, Thanks. This is now applied to the for-next branch of the linux-gfs2 tree: https://git.kernel.org/cgit/linux/kernel/git/gfs2/linux-gfs2.git/commit/fs/gfs2?h=for-next&id=a527b38e1475211b67eb59b3fadb40689f035529 Regards, Bob Peterson Red Hat File Systems