fix '< 0' comparisons of unsigned expressions

Fix the error handling of nvme_get_nsid().

Signed-off-by: Klaus Jensen <k.jensen@samsung.com>

diff --git a/nvme.c b/nvme.c
index ac63f036a95a91fe066034c33a5ec21ceea54c87..2cc6f244219c5ef902b733ba7c85cffa267848f5 100644 (file)
--- a/nvme.c
+++ b/nvme.c
@@ -1030,13 +1030,9 @@ static int delete_ns(int argc, char **argv, struct command *cmd, struct plugin *
                goto ret;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id == 0) {
-                       err = -EINVAL;
-                       goto close_fd;
-               }
-               else if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -1493,9 +1489,9 @@ static int ns_descs(int argc, char **argv, struct command *cmd, struct plugin *p
                flags = BINARY;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -1575,15 +1571,9 @@ static int id_ns(int argc, char **argv, struct command *cmd, struct plugin *plug
                flags |= VERBOSE;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
-                       goto close_fd;
-               }
-               else if (!cfg.namespace_id) {
-                       fprintf(stderr,
-                               "Error: requesting namespace-id from non-block device\n");
-                       err = -ENOTBLK;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -2837,13 +2827,11 @@ static int format(int argc, char **argv, struct command *cmd, struct plugin *plu
                 * format of all namespaces.
                 */
                cfg.namespace_id = NVME_NSID_ALL;
-       } else {
-               if (!cfg.namespace_id) {
-                       cfg.namespace_id = nvme_get_nsid(fd);
-                       if (cfg.namespace_id < 0) {
-                               err = cfg.namespace_id;
-                               goto close_fd;
-                       }
+       } else if (!cfg.namespace_id) {
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
+                       goto close_fd;
                }
        }
 
@@ -3379,9 +3367,9 @@ static int write_uncor(int argc, char **argv, struct command *cmd, struct plugin
                goto ret;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -3471,9 +3459,9 @@ static int write_zeroes(int argc, char **argv, struct command *cmd, struct plugi
        if (cfg.deac)
                control |= NVME_RW_DEAC;
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -3564,9 +3552,9 @@ static int dsm(int argc, char **argv, struct command *cmd, struct plugin *plugin
        }
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -3694,9 +3682,9 @@ static int copy(int argc, char **argv, struct command *cmd, struct plugin *plugi
                goto close_fd;
        }
 
-       namespace_id = nvme_get_nsid(fd);
-       if (namespace_id == 0) {
-               err = -EINVAL;
+       err = namespace_id = nvme_get_nsid(fd);
+       if (err < 0) {
+               perror("get-namespace-id");
                goto close_fd;
        }
 
@@ -3753,9 +3741,9 @@ static int flush(int argc, char **argv, struct command *cmd, struct plugin *plug
                goto ret;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -3821,9 +3809,9 @@ static int resv_acquire(int argc, char **argv, struct command *cmd, struct plugi
                goto ret;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -3893,9 +3881,9 @@ static int resv_register(int argc, char **argv, struct command *cmd, struct plug
                goto ret;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -3973,9 +3961,9 @@ static int resv_release(int argc, char **argv, struct command *cmd, struct plugi
                goto ret;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -4051,9 +4039,9 @@ static int resv_report(int argc, char **argv, struct command *cmd, struct plugin
                flags = BINARY;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }
@@ -4433,9 +4421,9 @@ static int verify_cmd(int argc, char **argv, struct command *cmd, struct plugin
                control |= NVME_RW_FUA;
 
        if (!cfg.namespace_id) {
-               cfg.namespace_id = nvme_get_nsid(fd);
-               if (cfg.namespace_id < 0) {
-                       err = cfg.namespace_id;
+               err = cfg.namespace_id = nvme_get_nsid(fd);
+               if (err < 0) {
+                       perror("get-namespace-id");
                        goto close_fd;
                }
        }