Whamcloud - gitweb
LU-13814 lov: remove non-dio handling in lov_dio_submit 07/52107/26
authorPatrick Farrell <pfarrell@whamcloud.com>
Fri, 23 Feb 2024 16:28:11 +0000 (11:28 -0500)
committerOleg Drokin <green@whamcloud.com>
Thu, 10 Apr 2025 06:49:48 +0000 (06:49 +0000)
lov_dio_submit is now specialized to dio.  Cleanup
accordingly.

Signed-off-by: Patrick Farrell <patrick.farrell@oracle.com>
Change-Id: I4e293011c77eb1b7b8f2c0ec78565458a6fe047c
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/52107
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Marc Vef <mvef@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Qian Yingjin <qian@ddn.com>
lustre/lov/lov_io.c

index 79a994d..dcf41b5 100644 (file)
@@ -1327,8 +1327,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;
@@ -1339,9 +1337,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;
@@ -1365,17 +1360,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)) {