From c85bf0511d8023edfa9a8c2a6f4d1091be73565a Mon Sep 17 00:00:00 2001 From: Patrick Farrell Date: Mon, 4 Sep 2023 19:52:03 -0400 Subject: [PATCH] LU-13814 osc: Remove usage of cdp_cl_pages As part of eliminating cl_page in the DIO path, we need to clear out all uses of it. This is one more minor one - there's no need to clear this list before returning from this function, because if it fails we give up entirely. Test-Parameters: trivial Test-Parameters: testlist=sanity-sec env=ONLY=52,59a,59b Signed-off-by: Patrick Farrell Change-Id: I9dc1053c542ce7a903a93f7b9a1fb0bfc6ac1641 --- lustre/osc/osc_cache.c | 7 ------- 1 file changed, 7 deletions(-) diff --git a/lustre/osc/osc_cache.c b/lustre/osc/osc_cache.c index 3b47f70..aa061a5 100644 --- a/lustre/osc/osc_cache.c +++ b/lustre/osc/osc_cache.c @@ -2591,13 +2591,6 @@ int osc_queue_dio_pages(const struct lu_env *env, struct cl_io *io, ext = osc_extent_alloc(obj); if (ext == NULL) { - for (i = from_page; i <= to_page; i++) { - page = cdp->cdp_cl_pages[i]; - opg = osc_cl_page_osc(page, obj); - oap = &opg->ops_oap; - - list_del_init(&oap->oap_pending_item); - } osc_dio_completion(env, obj, cdp, crt, to_page - from_page + 1, brw_flags, -ENOMEM); -- 1.8.3.1