Whamcloud - gitweb
LU-13814 osc: Remove usage of cdp_cl_pages 89/52189/20
authorPatrick Farrell <pfarrell@whamcloud.com>
Mon, 4 Sep 2023 23:52:03 +0000 (19:52 -0400)
committerPatrick Farrell <pfarrell@whamcloud.com>
Tue, 26 Sep 2023 18:58:30 +0000 (14:58 -0400)
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 <pfarrell@whamcloud.com>
Change-Id: I9dc1053c542ce7a903a93f7b9a1fb0bfc6ac1641

lustre/osc/osc_cache.c

index 3b47f70..aa061a5 100644 (file)
@@ -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);