From patchwork Sun Aug 20 18:39:44 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: SF Markus Elfring X-Patchwork-Id: 9911297 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 243CA603F3 for ; Sun, 20 Aug 2017 18:40:42 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 1A00628751 for ; Sun, 20 Aug 2017 18:40:42 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 072232875F; Sun, 20 Aug 2017 18:40:42 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A92EE28751 for ; Sun, 20 Aug 2017 18:40:41 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753422AbdHTSjt (ORCPT ); Sun, 20 Aug 2017 14:39:49 -0400 Received: from mout.web.de ([217.72.192.78]:56650 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753384AbdHTSjr (ORCPT ); Sun, 20 Aug 2017 14:39:47 -0400 Received: from [192.168.1.2] ([92.225.240.160]) by smtp.web.de (mrweb101 [213.165.67.124]) with ESMTPSA (Nemesis) id 0Lp6xk-1d6qSj2u3A-00es9p; Sun, 20 Aug 2017 20:39:45 +0200 Subject: [PATCH 2/3] ceph: Delete an unnecessary return statement in update_dentry_lease() From: SF Markus Elfring To: ceph-devel@vger.kernel.org, Ilya Dryomov , Sage Weil , "Yan, Zheng" Cc: LKML , kernel-janitors@vger.kernel.org References: Message-ID: <8a9a3d91-91b2-81ed-eafc-66fc637de539@users.sourceforge.net> Date: Sun, 20 Aug 2017 20:39:44 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: Content-Language: en-GB X-Provags-ID: V03:K0:A7gSmiqI+/3mI7G50tEMzv2lG2SZaXu9RlSknYKC8GM7/Ow6nmS iYw5v6JrkZrEBd171h0DKGd9X8lZTKsSFeRfU+o0GvOK+AayPQyXGkJGyXt1ahcP+m6Yqlm DzcHi1XXW7nh2r6PId5qHu6Toccx3LbeY0i5KZLYsSFsVlEq8vDpSC630oruNxypvb6yr6M cMqkmXxaTfZc8oioWQdLA== X-UI-Out-Filterresults: notjunk:1; V01:K0:ogzmVpuPT5U=:eQ2UURYhHVudSHlshlEXEl Z7h1VkEe6UWGiRlG3l+RhGW1+qFQXTC/+GtIwxkltIX3nxasYq7en2ySDLkmtD85xB/ZQT6/D YC9HfKIgmQrvC67eKi46bTpzIfnlLSmWV3TRr6jm9ROIostiGX3eh6mSw95mby3oIQz94tjcZ 7gvBli7IIR/sCEXrZoX93+PnwxEBKsZUfSKs3xWcRmjy+5rVy0XXz13vENu/KeUOmjR+K6JD4 hU8m2nGi0eXcDDRAS7Ull1TFts1wHRB13bimSkTlvAkbFsBut84LaOaB+fXgAYGgT8RMdyOk+ XeLSYaGAJ4LUuJOVUsIKr4jhRhLFA8Lm6IRklcNLDxGmgGr3rEQ9Man2y3+IBbnVPLNuMBobf 9G6QuBeZVTIcM46maAPLNsKJ07ItfBd+u962/b1wvsRCNpSw0J/SQVOYo/iaqDOCKCmXEBhum uuLEaR8duy1DTc09+XK/gKXQIKknYP3qpEMemaSm0sq7c1Ix1OINQx5TWtJU4libVWO5jrAv6 tXjEZoK8vYnM2kmnINg38seJufpCTtfBE3CFA3L5kJJ7CVlNy6jGXpRrtA8alVIrgu1xVCNYR ibFvataxS29PBMfgzM9eweVCJ7yfjd+K4bdomfHDVjRgSjNQBCFaVi3MtGBXOvj7udBmHjT2z 9Nvc14c7aPXuOi8R99/HbrrK2cHOZ7qVIoX0FXet84EmD/uzRQp3/yXQ0zMZMrCx3AUn6WzgQ 3dPtOJHncLXsqJcfJrXWH9zX9Dq4wzk17r1VC0u6kftlWrFygGEjhp3ZKldHgYoqT1n4pA0gF tsd6ZcZqr7ob8WAYkHUoaSft+dyWSpLJfosyQNQgshwdxO8vec= Sender: ceph-devel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: ceph-devel@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Markus Elfring Date: Sun, 20 Aug 2017 20:08:25 +0200 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function. Signed-off-by: Markus Elfring --- fs/ceph/inode.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/ceph/inode.c b/fs/ceph/inode.c index 010e7be768e6..8eae56b2d1d6 100644 --- a/fs/ceph/inode.c +++ b/fs/ceph/inode.c @@ -1067,7 +1067,6 @@ static void update_dentry_lease(struct dentry *dentry, spin_unlock(&dentry->d_lock); if (old_lease_session) ceph_put_mds_session(old_lease_session); - return; } /*