From: Yi Zhang <yizhan@redhat.com>
Date: Tue, 18 Apr 2017 09:20:24 +0000 (+0800)
Subject: nvme-cli: return 0 if disconnect operation with cfg.nqn successfully
X-Git-Tag: v1.3~13
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=f4806f68a54f664b533d3a269d19b400e136d04d;p=users%2Fsagi%2Fnvme-cli.git

nvme-cli: return 0 if disconnect operation with cfg.nqn successfully

disconnect_by_nqn function will return the number of controllers
successfully disconnected, so change ret = 0 if disconnect operation
with nqn successfully.

$ ./nvme disconnect -n testnqn
NQN:testnqn disconnected 1 controller(s)
$ echo $?
1

Signed-off-by: Yi Zhang <yizhan@redhat.com>
Signed-off-by: Keith Busch <keith.busch@intel.com>
---

diff --git a/fabrics.c b/fabrics.c
index cd671c7a..a2e8f602 100644
--- a/fabrics.c
+++ b/fabrics.c
@@ -934,8 +934,10 @@ int disconnect(const char *desc, int argc, char **argv)
 		if (ret < 0)
 			fprintf(stderr, "Failed to disconnect by NQN: %s\n",
 				cfg.nqn);
-		else
+		else {
 			printf("NQN:%s disconnected %d controller(s)\n", cfg.nqn, ret);
+			ret = 0;
+		}
 	}
 
 	if (cfg.device) {