Whamcloud - gitweb
LU-13805 clio: Trivial DIO cleanups 87/49987/22
authorPatrick Farrell <pfarrell@whamcloud.com>
Sat, 11 Feb 2023 18:12:14 +0000 (13:12 -0500)
committerOleg Drokin <green@whamcloud.com>
Wed, 28 Jun 2023 21:46:09 +0000 (21:46 +0000)
This is some minor DIO refactoring and an additional debug
message discovered while working on this.  Extremely minor.

test-parameters: trivial

Signed-off-by: Patrick Farrell <pfarrell@whamcloud.com>
Change-Id: Ica2a7340ac02d3ba0d8f65662ff4b39026078e81
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/49987
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
lustre/obdclass/cl_io.c
lustre/osc/osc_cache.c

index e7a6fbd..88a578f 100644 (file)
@@ -1386,14 +1386,11 @@ void cl_sync_io_note(const struct lu_env *env, struct cl_sync_io *anchor,
 
                spin_unlock(&anchor->csi_waitq.lock);
 
-               if (csi_dio_aio) {
-                       if (end_io == cl_dio_aio_end) {
-                               if (!creator_free)
-                                       cl_dio_aio_free(env, dio_aio);
-                       } else if (end_io == cl_sub_dio_end) {
-                               if (!creator_free)
-                                       cl_sub_dio_free(sub_dio_aio);
-                       }
+               if (csi_dio_aio && !creator_free) {
+                       if (end_io == cl_dio_aio_end)
+                               cl_dio_aio_free(env, dio_aio);
+                       else if (end_io == cl_sub_dio_end)
+                               cl_sub_dio_free(sub_dio_aio);
                }
        }
        EXIT;
index 9710fdc..65ad501 100644 (file)
@@ -2206,6 +2206,7 @@ __must_hold(&cli->cl_loi_list_lock)
 
                spin_lock(&cli->cl_loi_list_lock);
        }
+       EXIT;
 }
 
 int osc_io_unplug0(const struct lu_env *env, struct client_obd *cli,