Commit dc53a9de authored by Seth Howell's avatar Seth Howell Committed by Changpeng Liu
Browse files

lib/ftl: remove local phase variable 



Since the local is only used in the SPDK_DEBUGLOG call, it was causing
the build to fail when the configure options --disable-debug and
--enable-werror were supplied together. This can be seen in the most
recent nightly builds.

Change-Id: I32112cf832a705292783da4e841badaeed17dbb6
Signed-off-by: default avatarSeth Howell <seth.howell@intel.com>
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/461746


Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarChangpeng Liu <changpeng.liu@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 69642141

lib/ftl/ftl_restore.c

+1 −2
Original line number Diff line number Diff line
@@ -457,7 +457,6 @@ static void 
ftl_nv_cache_restore_done(struct ftl_nv_cache_restore *restore, uint64_t current_addr)

{

	struct ftl_nv_cache *nv_cache = restore->nv_cache;

	unsigned int phase = nv_cache->phase;



	pthread_spin_lock(&nv_cache->lock);

	nv_cache->current_addr = current_addr;
@@ -465,7 +464,7 @@ ftl_nv_cache_restore_done(struct ftl_nv_cache_restore *restore, uint64_t current 
	pthread_spin_unlock(&nv_cache->lock);



	SPDK_DEBUGLOG(SPDK_LOG_FTL_INIT, "Enabling non-volatile cache (phase: %u, addr: %"

		      PRIu64")\n", phase, current_addr);

		      PRIu64")\n", nv_cache->phase, current_addr);



	ftl_nv_cache_restore_complete(restore, 0);

}