Commit 5dc88c6c authored Jan 22, 2020 by Tomasz Zawadzki

lib/blob: _spdk_bs_load_replay_md_parse_page() now takes only load ctx



_spdk_bs_load_replay_md_parse_page() is only used in
replay path during blobstore load.

Next patch will expand the load ctx with array of
extent pages to be read. It is filled out when reading
in-chain metadata of extent table descriptors.

Passing the load ctx here will make it simpler to
fill out the array when processing extent table.

Signed-off-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Change-Id: If96e6670560c8c4a3610f33ece14c354d7d5da39
Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/482412


Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Reviewed-by: Paul Luse <paul.e.luse@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>

parent b5e99348

lib/blob/blobstore.c

+4 −2

Original line number	Diff line number	Diff line
		@@ -3568,8 +3568,10 @@ _spdk_bs_load_read_used_pages(struct spdk_bs_load_ctx *ctx)
		}

		static int
		_spdk_bs_load_replay_md_parse_page(const struct spdk_blob_md_page page, struct spdk_blob_store bs)
		_spdk_bs_load_replay_md_parse_page(struct spdk_bs_load_ctx *ctx)
		{
		struct spdk_blob_store *bs = ctx->bs;
		struct spdk_blob_md_page *page = ctx->page;
		struct spdk_blob_md_descriptor *desc;
		size_t cur_desc = 0;

		@@ -3786,7 +3788,7 @@ _spdk_bs_load_replay_md_cpl(spdk_bs_sequence_t seq, void cb_arg, int bserrno)
		if (ctx->page->sequence_num == 0) {
		spdk_bit_array_set(ctx->bs->used_blobids, page_num);
		}
		if (_spdk_bs_load_replay_md_parse_page(ctx->page, ctx->bs)) {
		if (_spdk_bs_load_replay_md_parse_page(ctx)) {
		_spdk_bs_load_ctx_fail(ctx, -EILSEQ);
		return;
		}