From: J. Bruce Fields <bfields@redhat.com>
Date: Wed, 21 Jul 2010 22:31:42 +0000 (-0400)
Subject: nfsd: minor nfsd_svc() cleanup
X-Git-Tag: upstream/snapshot3+hdmi~13082^2~19
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=af4718f3f996925f770e540004ec9224243d1682;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

nfsd: minor nfsd_svc() cleanup

More idiomatic to put the error case in the if clause.

Signed-off-by: J. Bruce Fields <bfields@redhat.com>
---

diff --git a/fs/nfsd/nfssvc.c b/fs/nfsd/nfssvc.c
index 62a6c44..92173bd 100644
--- a/fs/nfsd/nfssvc.c
+++ b/fs/nfsd/nfssvc.c
@@ -443,12 +443,13 @@ nfsd_svc(unsigned short port, int nrservs)
 	if (error)
 		goto out_shutdown;
 	error = svc_set_num_threads(nfsd_serv, NULL, nrservs);
-	if (error == 0)
-		/* We are holding a reference to nfsd_serv which
-		 * we don't want to count in the return value,
-		 * so subtract 1
-		 */
-		error = nfsd_serv->sv_nrthreads - 1;
+	if (error)
+		goto out_destroy;
+	/* We are holding a reference to nfsd_serv which
+	 * we don't want to count in the return value,
+	 * so subtract 1
+	 */
+	error = nfsd_serv->sv_nrthreads - 1;
 out_destroy:
 	svc_destroy(nfsd_serv);		/* Release server */
 out_shutdown: