Commit 437dce1a authored by Jim Harris's avatar Jim Harris Committed by Tomasz Zawadzki
Browse files

nvme/perf: remove unnecessary logging 



There's little use to the "Starting thread on core XX"
messages - they may have served a purpose at some point,
but the other messages showing where each controller
is associated is sufficient.

Also remove the "Attaching to..." message - the
"Attached to..." message is sufficient here too.

Signed-off-by: default avatarJim Harris <james.r.harris@intel.com>
Change-Id: I9543e8f3f6d7b6e28238cba27fc1e4194908f3b7

Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/473337


Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarAlexey Marchuk <alexeymar@mellanox.com>
parent 262ecf0e

examples/nvme/perf/perf.c

+1 −10
Original line number Diff line number Diff line
@@ -1017,8 +1017,6 @@ work_fn(void *arg) 
	struct ns_worker_ctx *ns_ctx = NULL;

	uint32_t unfinished_ns_ctx;



	printf("Starting thread on core %u\n", worker->lcore);



	/* Allocate queue pairs for each namespace. */

	ns_ctx = worker->ns_ctx;

	while (ns_ctx != NULL) {
@@ -1812,20 +1810,13 @@ static bool 
probe_cb(void *cb_ctx, const struct spdk_nvme_transport_id *trid,

	 struct spdk_nvme_ctrlr_opts *opts)

{

	if (trid->trtype != SPDK_NVME_TRANSPORT_PCIE) {

		printf("Attaching to NVMe over Fabrics controller at %s:%s: %s\n",

		       trid->traddr, trid->trsvcid,

		       trid->subnqn);

	} else {

	if (trid->trtype == SPDK_NVME_TRANSPORT_PCIE) {

		if (g_disable_sq_cmb) {

			opts->use_cmb_sqs = false;

		}

		if (g_no_shn_notification) {

			opts->no_shn_notification = true;

		}



		printf("Attaching to NVMe Controller at %s\n",

		       trid->traddr);

	}



	/* Set io_queue_size to UINT16_MAX, NVMe driver