Commit 9b143bca authored by Pierre Lestringant's avatar Pierre Lestringant Committed by Konrad Sztyber
Browse files

nvmf/rpc: Remove newline at the end of error messages 



Change-Id: If34e705f4d284f9264dbe5896cf5a793129b576e
Signed-off-by: default avatarPierre Lestringant <plestringant@kalray.eu>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/24241


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Reviewed-by: default avatarTomasz Zawadzki <tomasz@tzawadzki.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 916ec3db

lib/nvmf/nvmf_rpc.c

+7 −7
Original line number Diff line number Diff line
@@ -448,7 +448,7 @@ rpc_nvmf_create_subsystem(struct spdk_jsonrpc_request *request, 
		SPDK_ERRLOG("Subsystem %s: invalid max_write_zeroes_size_kib %"PRIu64"\n", req->nqn,

			    req->max_write_zeroes_size_kib);

		spdk_jsonrpc_send_error_response_fmt(request, SPDK_JSONRPC_ERROR_INVALID_PARAMS,

						     "Invalid max_write_zeroes_size_kib %"PRIu64"\n", req->max_write_zeroes_size_kib);

						     "Invalid max_write_zeroes_size_kib %"PRIu64, req->max_write_zeroes_size_kib);

		goto cleanup;

	}
@@ -966,7 +966,7 @@ rpc_nvmf_subsystem_add_listener(struct spdk_jsonrpc_request *request, 
	if (rc != 0) {

		if (rc == -EBUSY) {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

							 "subsystem busy, retry later.\n");

							 "subsystem busy, retry later.");

		} else {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR, "Internal error");

		}
@@ -1045,7 +1045,7 @@ rpc_nvmf_subsystem_remove_listener(struct spdk_jsonrpc_request *request, 
	if (rc != 0) {

		if (rc == -EBUSY) {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

							 "subsystem busy, retry later.\n");

							 "subsystem busy, retry later.");

		} else {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR, "Internal error");

		}
@@ -1360,7 +1360,7 @@ rpc_nvmf_subsystem_listener_set_ana_state(struct spdk_jsonrpc_request *request, 
	if (!tgt) {

		SPDK_ERRLOG("Unable to find a target object.\n");

		spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

						 "Unable to find a target.\n");

						 "Unable to find a target.");

		nvmf_rpc_listener_ctx_free(ctx);

		return;

	}
@@ -1612,7 +1612,7 @@ rpc_nvmf_subsystem_add_ns(struct spdk_jsonrpc_request *request, 
	if (rc != 0) {

		if (rc == -EBUSY) {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

							 "subsystem busy, retry later.\n");

							 "subsystem busy, retry later.");

		} else {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR, "Internal error");

		}
@@ -1732,7 +1732,7 @@ rpc_nvmf_subsystem_remove_ns(struct spdk_jsonrpc_request *request, 
	if (rc != 0) {

		if (rc == -EBUSY) {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

							 "subsystem busy, retry later.\n");

							 "subsystem busy, retry later.");

		} else {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR, "Internal error");

		}
@@ -1857,7 +1857,7 @@ nvmf_rpc_ns_visible(struct spdk_jsonrpc_request *request, 
	if (rc != 0) {

		if (rc == -EBUSY) {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR,

							 "subsystem busy, retry later.\n");

							 "subsystem busy, retry later.");

		} else {

			spdk_jsonrpc_send_error_response(request, SPDK_JSONRPC_ERROR_INTERNAL_ERROR, "Internal error");

		}