Whamcloud - gitweb
LU-13814 osc: do not call osc_lru_use for transient 87/52087/22
authorPatrick Farrell <pfarrell@whamcloud.com>
Fri, 23 Feb 2024 16:22:20 +0000 (11:22 -0500)
committerOleg Drokin <green@whamcloud.com>
Thu, 8 Aug 2024 00:03:44 +0000 (00:03 +0000)
Transient pages are never added to the LRU, because they
can't be cached.  osc_lru_use already skips them because
they don't have the flag set, but make it explicit that
this is not called for transient pages.

Test-Parameters: trivial
Signed-off-by: Patrick Farrell <patrick.farrell@oracle.com>
Change-Id: I2c92ccb52380faefbcba3bfa35508dac2b601bd4
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/52087
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Qian Yingjin <qian@ddn.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
lustre/osc/osc_page.c

index d706cad..bb8b350 100644 (file)
@@ -300,9 +300,10 @@ void osc_page_submit(const struct lu_env *env, struct osc_page *opg,
        if (oio->oi_cap_sys_resource)
                oap->oap_brw_flags |= OBD_BRW_SYS_RESOURCE;
 
-       if (page->cp_type != CPT_TRANSIENT)
+       if (page->cp_type != CPT_TRANSIENT) {
                osc_page_transfer_get(opg, "transfer\0imm");
-       osc_lru_use(osc_cli(obj), opg);
+               osc_lru_use(osc_cli(obj), opg);
+       }
 }
 
 /* --------------- LRU page management ------------------ */