ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

nvme/perf: remove bad warning message

Browse Source 
When we specify a queue depth that may be higher than
the io_queue_size, we print a warning indicating this.
But it was also setting the g_warn flag which would
result in this message also getting printed:

WARNING: Some requested NVMe devices were skipped

But we don't actually skip any SSDs in this case.  So
remove setting of that flag in this case.

Signed-off-by: Jim Harris <james.r.harris@intel.com>
Change-Id: Ic63209f1a96990dac9664ce245c6406c37248494

Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/473064
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Alexey Marchuk <alexeymar@mellanox.com>
Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
This commit is contained in:
Jim Harris 2019-11-01 09:30:58 -07:00 committed by Tomasz Zawadzki
parent e0a0f90b0f
commit 766ef001f3
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
examples/nvme/perf/perf.c
View File

@ -695,7 +695,6 @@ register_ns(struct spdk_nvme_ctrlr *ctrlr, struct spdk_nvme_ns *ns)
opts.io_queue_size); opts.io_queue_size);
printf("Consider using lower queue depth or small IO size because " printf("Consider using lower queue depth or small IO size because "
"IO requests may be queued at the NVMe driver.\n"); "IO requests may be queued at the NVMe driver.\n");
g_warn = true;
} }
/* For requests which have children requests, parent request itself /* For requests which have children requests, parent request itself
* will also occupy 1 entry. * will also occupy 1 entry.