Commit 3d12e6d4 authored Aug 08, 2019 by Changpeng Liu

bdev/nvme: deprecate ResetControllerOnTimeout configuration option



'ResetControllerOnTimeout' was renamed to 'ActionOnTimeout' for a
long time, so we deprecate the support for this old configuration
option.

Change-Id: Ia44035a8749ca046c1df16f82601389b5bafeb76
Signed-off-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/464491


Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Broadcom SPDK FC-NVMe CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>

parent 7cbe1ccd

lib/bdev/nvme/bdev_nvme.c

+0 −11

Original line number	Diff line number	Diff line
		@@ -1404,17 +1404,6 @@ bdev_nvme_library_init(void)
		} else if (!strcasecmp(val, "Abort")) {
		g_opts.action_on_timeout = SPDK_BDEV_NVME_TIMEOUT_ACTION_ABORT;
		}
		} else {
		/* Handle old name for backward compatibility */
		val = spdk_conf_section_get_val(sp, "ResetControllerOnTimeout");
		if (val) {
		SPDK_WARNLOG("ResetControllerOnTimeout was renamed to ActionOnTimeout\n");
		SPDK_WARNLOG("Please update your configuration file\n");

		if (spdk_conf_section_get_boolval(sp, "ResetControllerOnTimeout", false)) {
		g_opts.action_on_timeout = SPDK_BDEV_NVME_TIMEOUT_ACTION_RESET;
		}
		}
		}
		}