Whamcloud - gitweb
LU-13814 lov: remove non-dio handling in lov_dio_submit 07/52107/15
authorPatrick Farrell <pfarrell@whamcloud.com>
Sun, 24 Sep 2023 20:10:55 +0000 (16:10 -0400)
committerPatrick Farrell <pfarrell@whamcloud.com>
Tue, 26 Sep 2023 18:58:04 +0000 (14:58 -0400)
lov_dio_submit is now specialized to dio.  Cleanup
accordingly.

Test-Parameters: fortestonly
Test-Parameters: forjanitoronly
Signed-off-by: Patrick Farrell <pfarrell@whamcloud.com>
Change-Id: I4e293011c77eb1b7b8f2c0ec78565458a6fe047c

lustre/lov/lov_io.c

index f9c3419..61642e1 100644 (file)
@@ -1295,8 +1295,6 @@ static int lov_dio_submit(const struct lu_env *env,
        struct cl_page          *page;
        struct cl_page_list     *qin;
        struct lov_io_sub       *sub;
-       struct cl_page          *tmp;
-       bool dio = false;
        int index;
        int rc = 0;
        ENTRY;
@@ -1307,9 +1305,6 @@ static int lov_dio_submit(const struct lu_env *env,
        qin = &queue->c2_qin;
        page = cl_page_list_first(qin);
 
-       if (page->cp_type == CPT_TRANSIENT)
-               dio = true;
-
        cl_page_list_init(plist);
        while (qin->pl_nr > 0) {
                struct cl_2queue  *cl2q = &lov_env_info(env)->lti_cl2q;
@@ -1333,17 +1328,7 @@ static int lov_dio_submit(const struct lu_env *env,
 
                index = page->cp_lov_index;
                /* DIO is already split by stripe */
-               if (!dio) {
-                       cl_page_list_for_each_safe(page, tmp, qin) {
-                               /* this page is not on this stripe */
-                               if (index != page->cp_lov_index)
-                                       continue;
-
-                               cl_page_list_move(&cl2q->c2_qin, qin, page);
-                       }
-               } else {
-                       cl_page_list_splice(qin, &cl2q->c2_qin);
-               }
+               cl_page_list_splice(qin, &cl2q->c2_qin);
 
                sub = lov_sub_get(env, lio, index);
                if (!IS_ERR(sub)) {