Commit 9c7f460f authored by Changpeng Liu's avatar Changpeng Liu Committed by Tomasz Zawadzki
Browse files

vfio-user: optimize notice log to debug log 



Change-Id: I2e47b148209ce4c232dbdc5f20c90548be995e1a
Signed-off-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/7334


Community-CI: Broadcom CI
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent b163a887

lib/nvme/nvme_vfio_user.c

+1 −1
Original line number Diff line number Diff line
@@ -320,7 +320,7 @@ nvme_vfio_ctrlr_scan(struct spdk_nvme_probe_ctx *probe_ctx, 
		SPDK_ERRLOG("Error to access file %s\n", probe_ctx->trid.traddr);

		return ret;

	}

	SPDK_NOTICELOG("Scan controller : %s\n", probe_ctx->trid.traddr);

	SPDK_DEBUGLOG(nvme_vfio, "Scan controller : %s\n", probe_ctx->trid.traddr);



	return nvme_ctrlr_probe(&probe_ctx->trid, probe_ctx, NULL);

}

lib/nvmf/vfio_user.c

+2 −2
Original line number Diff line number Diff line
@@ -1565,7 +1565,7 @@ destroy_ctrlr(struct nvmf_vfio_user_ctrlr *ctrlr) 
{

	assert(ctrlr != NULL);



	SPDK_NOTICELOG("destroy %s\n", ctrlr_id(ctrlr));

	SPDK_DEBUGLOG(nvmf_vfio, "destroy %s\n", ctrlr_id(ctrlr));



	if (ctrlr->thread == spdk_get_thread()) {

		_destroy_ctrlr(ctrlr);
@@ -1696,7 +1696,7 @@ nvmf_vfio_user_listen(struct spdk_nvmf_transport *transport, 


	pthread_mutex_init(&endpoint->lock, NULL);

	TAILQ_INSERT_TAIL(&vu_transport->endpoints, endpoint, link);

	SPDK_NOTICELOG("%s: doorbells %p\n", uuid, endpoint->doorbells);

	SPDK_DEBUGLOG(nvmf_vfio, "%s: doorbells %p\n", uuid, endpoint->doorbells);



out:

	if (err != 0) {

lib/vfio_user/vfio_user.c

+2 −2
Original line number Diff line number Diff line
@@ -266,7 +266,7 @@ vfio_user_check_version(struct vfio_device *dev) 
		return ret;

	}



	SPDK_NOTICELOG("%s Negotiate version %u.%u\n", vfio_user_message_str[VFIO_USER_VERSION],

	SPDK_DEBUGLOG(vfio_user, "%s Negotiate version %u.%u\n", vfio_user_message_str[VFIO_USER_VERSION],

		      version->major, version->minor);



	return 0;

lib/vfio_user/vfio_user_pci.c

+6 −6
Original line number Diff line number Diff line
@@ -256,9 +256,9 @@ vfio_device_setup_sparse_mmaps(struct vfio_device *device, int index, 
					return -EIO;

				}

			} else {

				SPDK_NOTICELOG("No valid fd, skip mmap for bar %d region %u\n", index, i);

				SPDK_DEBUGLOG(vfio_pci, "No valid fd, skip mmap for bar %d region %u\n", index, i);

			}

			SPDK_NOTICELOG("Sparse region %u, Size 0x%llx, Offset 0x%llx, Map addr %p\n",

			SPDK_DEBUGLOG(vfio_pci, "Sparse region %u, Size 0x%llx, Offset 0x%llx, Map addr %p\n",

				      i, sparse->areas[i].size, sparse->areas[i].offset,

				      region->mmaps[j].mem);

			j++;
@@ -406,7 +406,7 @@ spdk_vfio_user_setup(const char *path) 
		goto cleanup;

	}



	SPDK_NOTICELOG("Device %s, Path %s Setup Successfully\n", device->name, device->path);

	SPDK_DEBUGLOG(vfio_pci, "Device %s, Path %s Setup Successfully\n", device->name, device->path);

	TAILQ_INSERT_TAIL(&g_vfio_devices, device, link);



	return device;
@@ -420,7 +420,7 @@ cleanup: 
void

spdk_vfio_user_release(struct vfio_device *dev)

{

	SPDK_NOTICELOG("Release file %s\n", dev->path);

	SPDK_DEBUGLOG(vfio_pci, "Release file %s\n", dev->path);



	vfio_device_unmap_bars(dev);

	if (dev->map) {