Commit be45e54a authored by Tomasz Zawadzki's avatar Tomasz Zawadzki Committed by Jim Harris
Browse files

lib/blob: simplify return path in serializing extent_rle 



This patch simplifies return path when returning from
serialization of extent_rle.

Both paths will share more code in upcoming patch.

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


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 3e372f35

lib/blob/blobstore.c

+4 −3
Original line number Diff line number Diff line
@@ -713,9 +713,8 @@ _spdk_blob_serialize_extent_rle(const struct spdk_blob *blob, 


		if (buf_sz < cur_sz) {

			/* If we ran out of buffer space, return */

			desc_extent_rle->length = sizeof(desc_extent_rle->extents[0]) * extent_idx;

			*next_cluster = i;

			return;

			goto finish;

		}



		lba = blob->active.clusters[i];
@@ -726,9 +725,11 @@ _spdk_blob_serialize_extent_rle(const struct spdk_blob *blob, 
	desc_extent_rle->extents[extent_idx].length = lba_count / lba_per_cluster;

	extent_idx++;



	desc_extent_rle->length = sizeof(desc_extent_rle->extents[0]) * extent_idx;

	*next_cluster = blob->active.num_clusters;



finish:

	desc_extent_rle->length = sizeof(desc_extent_rle->extents[0]) * extent_idx;



	return;

}