From patchwork Fri Jul 12 16:32:02 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chuck Lever X-Patchwork-Id: 2827016 Return-Path: X-Original-To: patchwork-linux-nfs@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork1.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork1.web.kernel.org (Postfix) with ESMTP id B85FF9F756 for ; Fri, 12 Jul 2013 16:39:05 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id AB6292013A for ; Fri, 12 Jul 2013 16:39:04 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id B47A22012D for ; Fri, 12 Jul 2013 16:39:03 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964956Ab3GLQjB (ORCPT ); Fri, 12 Jul 2013 12:39:01 -0400 Received: from mail-ye0-f171.google.com ([209.85.213.171]:55799 "EHLO mail-ye0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964936Ab3GLQjB (ORCPT ); Fri, 12 Jul 2013 12:39:01 -0400 X-Greylist: delayed 415 seconds by postgrey-1.27 at vger.kernel.org; Fri, 12 Jul 2013 12:39:01 EDT Received: by mail-ye0-f171.google.com with SMTP id q14so3253727yen.30 for ; Fri, 12 Jul 2013 09:39:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:subject:to:from:date:message-id:in-reply-to:references :user-agent:mime-version:content-type:content-transfer-encoding; bh=r/6ITNau7pdC0ZNNumgPa+9NBYwkcsMJmCjOsDhm6wE=; b=Kg+drUpJCQ0tnilKgOEbpzVmVPwBRG8e/Jp3qVZrFYHIDDM0RyOR55a0keNmzeIu68 O6apaXHJSMPqZ0BXmiDjuM5VJf5Nxh2WzbewcmYDvXhDATig0Ecky6o93pt4tLRgqUFA PttB/9Ltp7/mRYM6ueOGyw2KRkYLwnPxu/KgqM3K9rc5X2ZqVR8Jc3uiVSApRfhYdkyV BhxIStg6vxVsWIxnSQ6jFtPgcZtEUeLOBJx/wAZ5r9FDydOCpkidVkt72Kzpkg0NIU2d 4CYr5uNjKQ4Sf+bj+KPmX1fniNyChpTlEkWX2xqab2ksNWomJ6+56xrLg4dHRbQHFwkN jnCQ== X-Received: by 10.236.159.196 with SMTP id s44mr24371048yhk.105.1373646723669; Fri, 12 Jul 2013 09:32:03 -0700 (PDT) Received: from seurat.1015granger.net ([2604:8800:100:81fc:20c:29ff:fe93:815b]) by mx.google.com with ESMTPSA id y70sm68995072yhe.15.2013.07.12.09.32.02 for (version=TLSv1 cipher=RC4-SHA bits=128/128); Fri, 12 Jul 2013 09:32:03 -0700 (PDT) Subject: [PATCH v1 04/19] NFS: Refactor nfs4_call_data_sequence() To: linux-nfs@vger.kernel.org From: Chuck Lever Date: Fri, 12 Jul 2013 12:32:02 -0400 Message-ID: <20130712163202.1444.45080.stgit@seurat.1015granger.net> In-Reply-To: <20130712155303.1444.62697.stgit@seurat.1015granger.net> References: <20130712155303.1444.62697.stgit@seurat.1015granger.net> User-Agent: StGit/0.16 MIME-Version: 1.0 Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_HI,RP_MATCHES_RCVD,T_DKIM_INVALID,UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP Pull main logic out of nfs4_call_data_sequence() so it can eventually be shared with the NFSv4.0 call_sync path. The new helper function is left behind the NFSv4.1 CONFIG switch for now to keep compiler warning noise at a minimum. Signed-off-by: Chuck Lever --- fs/nfs/nfs4proc.c | 28 ++++++++++++++++++---------- 1 file changed, 18 insertions(+), 10 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/fs/nfs/nfs4proc.c b/fs/nfs/nfs4proc.c index 2f6f3b9..e52d2b3 100644 --- a/fs/nfs/nfs4proc.c +++ b/fs/nfs/nfs4proc.c @@ -393,6 +393,23 @@ static void renew_lease(const struct nfs_server *server, unsigned long timestamp #if defined(CONFIG_NFS_V4_1) +static int nfs4_run_rpc_task(struct rpc_clnt *clnt, + struct rpc_message *msg, + struct rpc_task_setup *task_setup) +{ + struct rpc_task *task; + int ret; + + task = rpc_run_task(task_setup); + if (IS_ERR(task)) + ret = PTR_ERR(task); + else { + ret = task->tk_status; + rpc_put_task(task); + } + return ret; +} + static void nfs41_sequence_free_slot(struct nfs4_sequence_res *res) { struct nfs4_session *session; @@ -674,8 +691,6 @@ static int nfs4_call_sync_sequence(struct rpc_clnt *clnt, struct nfs4_sequence_args *args, struct nfs4_sequence_res *res) { - int ret; - struct rpc_task *task; struct nfs41_call_sync_data data = { .seq_server = server, .seq_args = args, @@ -688,14 +703,7 @@ static int nfs4_call_sync_sequence(struct rpc_clnt *clnt, .callback_data = &data }; - task = rpc_run_task(&task_setup); - if (IS_ERR(task)) - ret = PTR_ERR(task); - else { - ret = task->tk_status; - rpc_put_task(task); - } - return ret; + return nfs4_run_rpc_task(clnt, msg, &task_setup); } #else