From af10f514b5e8713f0821c34fadfe22c91d137720 Mon Sep 17 00:00:00 2001 From: Jens Axboe Date: Tue, 4 Apr 2023 08:51:31 -0600 Subject: [PATCH] engines/nvme: cache errno value It's pretty pointless to do a bunch of things in between getting a -1/errno value, and then expect errno to still be what you want. Signed-off-by: Jens Axboe --- engines/nvme.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/engines/nvme.c b/engines/nvme.c index ac908687..fd2161f3 100644 --- a/engines/nvme.c +++ b/engines/nvme.c @@ -112,9 +112,10 @@ int fio_nvme_get_info(struct fio_file *f, __u32 *nsid, __u32 *lba_sz, namespace_id = ioctl(fd, NVME_IOCTL_ID); if (namespace_id < 0) { + err = -errno; log_err("failed to fetch namespace-id"); close(fd); - return -errno; + return err; } /* @@ -414,6 +415,7 @@ int fio_nvme_iomgmt_ruhs(struct thread_data *td, struct fio_file *f, } else errno = 0; + ret = -errno; close(fd); - return -errno; + return ret; } -- 2.25.1