Commit 414500c9 authored by Simon A. F. Lund's avatar Simon A. F. Lund Committed by Tomasz Zawadzki
Browse files

examples/nvme_fio_plugin: fix log-msgs. in report_zones() and reset_wp() 



In spdk_fio_report_zones(), log_err did not prefix messages with
"spdk/nvme", making it hard to determine who dumped the error-message.
In spdk_fio_reset_wp() log_err described the wrong function.

This change fixes the above.

Signed-off-by: default avatarSimon A. F. Lund <simon.lund@samsung.com>
Change-Id: I41df6d451e88942806c8b5a3cf9a0902d98cb186
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/4916


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarNiklas Cassel <niklas.cassel@wdc.com>
parent 12a44d47

examples/nvme/fio_plugin/fio_plugin.c

+3 −3
Original line number Diff line number Diff line
@@ -1188,7 +1188,7 @@ spdk_fio_report_zones(struct thread_data *td, struct fio_file *f, uint64_t offse 
			break;



		default:

			log_err("%s: invalid zone-type: 0x%x\n", f->file_name, zdesc->zt);

			log_err("spdk/nvme: %s: inv. zone-type: 0x%x\n", f->file_name, zdesc->zt);

			err = -EIO;

			goto exit;

		}
@@ -1217,7 +1217,7 @@ spdk_fio_report_zones(struct thread_data *td, struct fio_file *f, uint64_t offse 
			break;



		default:

			log_err("%s: invalid zone-state: 0x%x\n", f->file_name, zdesc->zs);

			log_err("spdk/nvme: %s: inv. zone-state: 0x%x\n", f->file_name, zdesc->zs);

			err = -EIO;

			goto exit;

		}
@@ -1263,7 +1263,7 @@ spdk_fio_reset_wp(struct thread_data *td, struct fio_file *f, uint64_t offset, u 
		err = spdk_nvme_zns_reset_zone(fio_qpair->ns, fio_qpair->qpair, cur / lba_nbytes,

					       false, pcu_cb, &completed);

		if (err || pcu(fio_qpair->qpair, &completed) || completed < 0) {

			log_err("spdk/nvme: report_zones(): err: %d, cpl: %d\n", err, completed);

			log_err("spdk/nvme: zns_reset_zone(): err: %d, cpl: %d\n", err, completed);

			err = err ? err : -EIO;

			break;

		}