Commit 7ed0ec68 authored by Tomasz Zawadzki's avatar Tomasz Zawadzki
Browse files

lib/blob: removed unused idx variable from persist ctx 



This variable went unused, since logic in
_spdk_blob_persist_write_page_chain() already dealt with
writing metadata from last to first page.

Signed-off-by: default avatarTomasz Zawadzki <tomasz.zawadzki@intel.com>
Change-Id: Ic70c47df1ea3bb01c8031244339c42e9936f28b0
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/467248


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

lib/blob/blobstore.c

+0 −4
Original line number Diff line number Diff line
@@ -1084,8 +1084,6 @@ struct spdk_blob_persist_ctx { 


	struct spdk_blob_md_page	*pages;



	uint64_t			idx;



	spdk_bs_sequence_t		*seq;

	spdk_bs_sequence_cpl		cb_fn;

	void				*cb_arg;
@@ -1427,7 +1425,6 @@ _spdk_blob_persist_start(struct spdk_blob_persist_ctx *ctx) 
		/* This is the signal that the blob should be deleted.

		 * Immediately jump to the clean up routine. */

		assert(blob->clean.num_pages > 0);

		ctx->idx = blob->clean.num_pages - 1;

		blob->state = SPDK_BLOB_STATE_CLEAN;

		_spdk_blob_persist_zero_pages(seq, ctx, 0);

		return;
@@ -1479,7 +1476,6 @@ _spdk_blob_persist_start(struct spdk_blob_persist_ctx *ctx) 
	}

	ctx->pages[i - 1].crc = _spdk_blob_md_page_calc_crc(&ctx->pages[i - 1]);

	/* Start writing the metadata from last page to first */

	ctx->idx = blob->active.num_pages - 1;

	blob->state = SPDK_BLOB_STATE_CLEAN;

	_spdk_blob_persist_write_page_chain(seq, ctx, 0);

}