Commit 3031b0f5 authored by Yankun Li's avatar Yankun Li Committed by Jim Harris
Browse files

lib/reduce: Delete logic of persisting old chunk map 



We don't need to persist the clearing of the old chunk map here.
The old chunk map becomes invalid after we update the logical map,
since the old chunk map will no longer have a reference to it
in the logical map.

Change-Id: Ie2ad3814e7b2ee0fa256d44ea3445c0bc2f9b9b8
Signed-off-by: default avatarYankun Li <yankun@staff.sina.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25474


Community-CI: Mellanox Build Bot
Reviewed-by: default avatarJim Harris <jim.harris@nvidia.com>
Reviewed-by: default avatarGangCao <gang.cao@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.com>
Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>
Reviewed-by: default avatarChangpeng Liu <changpeliu@tencent.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent 9366d22d

lib/reduce/reduce.c

+0 −3
Original line number Diff line number Diff line
@@ -1993,9 +1993,6 @@ _start_unmap_request_full_chunk(void *ctx) 
	chunk_map_index = vol->pm_logical_map[req->logical_map_index];

	if (chunk_map_index != REDUCE_EMPTY_MAP_ENTRY) {

		_reduce_vol_reset_chunk(vol, chunk_map_index);

		req->chunk = _reduce_vol_get_chunk_map(vol, chunk_map_index);

		_reduce_persist(vol, req->chunk,

				_reduce_vol_get_chunk_struct_size(vol->backing_io_units_per_chunk));

		vol->pm_logical_map[req->logical_map_index] = REDUCE_EMPTY_MAP_ENTRY;

		_reduce_persist(vol, &vol->pm_logical_map[req->logical_map_index], sizeof(uint64_t));

	}