From 14e98f0bf65b1586611c4dd55b8370cf5ea222aa Mon Sep 17 00:00:00 2001 From: Patrick Farrell Date: Sat, 28 Oct 2023 16:30:56 -0400 Subject: [PATCH] EX-7601 ofd: do not overwrite rc in unmerge_chunk unmerge_chunk should not be responsible for setting the lnb rc, because this overwrites the result of any previous activity on the lnb. Plus, unmerge_chunk can't fail. Signed-off-by: Patrick Farrell Change-Id: Id1ce590c7f1da3ab7faddbd685d264a33c08d639 Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/52876 Tested-by: jenkins Tested-by: Andreas Dilger Reviewed-by: Andreas Dilger Reviewed-by: Artem Blagodarenko --- lustre/obdclass/lustre_compr.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/lustre/obdclass/lustre_compr.c b/lustre/obdclass/lustre_compr.c index cf601b4..33cb305 100644 --- a/lustre/obdclass/lustre_compr.c +++ b/lustre/obdclass/lustre_compr.c @@ -313,8 +313,6 @@ void unmerge_chunk(struct brw_page **pga, struct niobuf_local *lnb, int first, *pg_len = PAGE_SIZE; left -= PAGE_SIZE; } - if (lnb) - lnb[first + i].lnb_rc = 0; CDEBUG(D_SEC, "pg_len: %u, left %u\n", *pg_len, left); } -- 1.8.3.1