Commit 68d8ff47 authored by Wojciech Panfil's avatar Wojciech Panfil Committed by Tomasz Zawadzki
Browse files

examples/perf: Adjust possibly misleading printf 



Without comma, this printf might have suggested that the dislayed value
is delta between g_queue_depth*entries and opts.io_queue_size.
Now, it's clear that it's the controller IO queue size that is printed
out.

Change-Id: I1492f183878ae2bce9a246f0ee41eb4d516e77b7
Signed-off-by: default avatarWojciech Panfil <wojciech.panfil@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/19882


Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
parent 219fbc90

examples/nvme/perf/perf.c

+2 −2
Original line number Diff line number Diff line
@@ -1284,9 +1284,9 @@ register_ns(struct spdk_nvme_ctrlr *ctrlr, struct spdk_nvme_ns *ns) 
	 */

	entries = (g_io_size_bytes - 1) / max_xfer_size + 2;

	if ((g_queue_depth * entries) > opts.io_queue_size) {

		printf("controller IO queue size %u less than required\n",

		printf("Controller IO queue size %u, less than required.\n",

		       opts.io_queue_size);

		printf("Consider using lower queue depth or small IO size because "

		printf("Consider using lower queue depth or smaller IO size, because "

		       "IO requests may be queued at the NVMe driver.\n");

	}

	/* For requests which have children requests, parent request itself