Commit 427304da authored Oct 31, 2024 by Yankun Li Committed by Konrad Sztyber Nov 04, 2024

lib/reduce: Reset req->reduce_errno



Req is used repeatedly, if reduce_errno is not reset,
the req->reduce_errno checked in function _read_read_done
_write_read_done and _write_write_done may be the error
code returned from the last request.

Change-Id: Ie2a28a13ab6006ada87577979a63308505768135
Signed-off-by: Yankun Li <yankun@staff.sina.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/25399


Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Community CI Samsung <spdk.community.ci.samsung@gmail.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Jim Harris <jim.harris@samsung.com>

parent 60826719

lib/reduce/reduce.c

+3 −0

Original line number	Diff line number	Diff line
		@@ -1897,6 +1897,7 @@ spdk_reduce_vol_readv(struct spdk_reduce_vol *vol,
		req->copy_after_decompress = false;
		req->cb_fn = cb_fn;
		req->cb_arg = cb_arg;
		req->reduce_errno = 0;

		if (!overlapped) {
		_start_readv_request(req);
		@@ -1972,6 +1973,7 @@ spdk_reduce_vol_writev(struct spdk_reduce_vol *vol,
		req->copy_after_decompress = false;
		req->cb_fn = cb_fn;
		req->cb_arg = cb_arg;
		req->reduce_errno = 0;

		if (!overlapped) {
		_start_writev_request(req);
		@@ -2035,6 +2037,7 @@ _reduce_vol_unmap_full_chunk(struct spdk_reduce_vol *vol,
		req->copy_after_decompress = false;
		req->cb_fn = cb_fn;
		req->cb_arg = cb_arg;
		req->reduce_errno = 0;

		if (!overlapped) {
		_start_unmap_request_full_chunk(req);