kernel: core_hook: tweak and rename path_umount handler

rename function to ksu_path_umount to clearly differntiate it from
sys_umount. Pass mnt string directly to avoid redundant resolution
for debug logging. Small touches, to make it neater.

Signed-off-by: backslashxx <[email protected]>

Author: backslashxx

kernel/core_hook.c

@@ -564,14 +564,10 @@ static bool should_umount(struct path *path)
 }
 
 #ifdef KSU_HAS_PATH_UMOUNT
-static int ksu_umount_mnt(struct path *path, int flags)
+static void ksu_path_umount(const char *mnt, struct path *path, int flags)
 {
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(5, 9, 0) || defined(KSU_UMOUNT)
-	return path_umount(path, flags);
-#else
-	// TODO: umount for non GKI kernel
-	return -ENOSYS;
-#endif
+	int err = path_umount(path, flags);
+	pr_info("%s: path: %s code: %d\n", __func__, mnt, err);
 }
 #else
 static void ksu_sys_umount(const char *mnt, int flags)
@@ -609,13 +605,10 @@ static void try_umount(const char *mnt, bool check_mnt, int flags)
 	}
 
 #ifdef KSU_HAS_PATH_UMOUNT
-	err = ksu_umount_mnt(&path, flags);
+	ksu_path_umount(mnt, &path, flags);
 #else
-	err = ksu_sys_umount(mnt, flags);
+	ksu_sys_umount(mnt, flags);
 #endif
-	if (err) {
-		pr_warn("umount %s failed: %d\n", mnt, err);
-	}
 }
 
 int ksu_handle_setuid(struct cred *new, const struct cred *old)