Whamcloud - gitweb
LU-17572 lov: remove noise from lov_init_sub() 25/54125/3
authorAlex Zhuravlev <bzzz@whamcloud.com>
Wed, 21 Feb 2024 11:47:32 +0000 (14:47 +0300)
committerOleg Drokin <green@whamcloud.com>
Sat, 23 Mar 2024 06:12:37 +0000 (06:12 +0000)
lov_init_sub() generates too many messages in applications like racer.
let's make it a bit less noisy.

Test-Parameters: trivial
Signed-off-by: Alex Zhuravlev <bzzz@whamcloud.com>
Change-Id: I00ae75597b550c29122d8fb9d34d4e0d24c38dd5
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/54125
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Arshad Hussain <arshad.hussain@aeoncomputing.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
lustre/lov/lov_object.c

index dabc8e9..865635e 100644 (file)
@@ -164,7 +164,6 @@ static int lov_init_sub(const struct lu_env *env, struct lov_object *lov,
        } else {
                struct lu_object  *old_obj;
                struct lov_object *old_lov;
-               unsigned int mask = D_INODE;
 
                spin_unlock(&subhdr->coh_attr_guard);
                old_obj = lu_object_locate(&parent->coh_lu, &lov_device_type);
@@ -176,14 +175,13 @@ static int lov_init_sub(const struct lu_env *env, struct lov_object *lov,
                        lu_object_unhash(env, &subobj->co_lu);
                        result = -EAGAIN;
                } else {
-                       mask = D_ERROR;
                        result = -EIO;
                }
 
-               LU_OBJECT_DEBUG(mask, env, &subobj->co_lu,
+               LU_OBJECT_DEBUG(D_INODE, env, &subobj->co_lu,
                                "stripe %d is already owned.", idx);
-               LU_OBJECT_DEBUG(mask, env, old_obj, "owned.");
-               LU_OBJECT_HEADER(mask, env, lov2lu(lov), "try to own.\n");
+               LU_OBJECT_DEBUG(D_INODE, env, old_obj, "owned.");
+               LU_OBJECT_HEADER(D_INODE, env, lov2lu(lov), "try to own.\n");
                cl_object_put(env, subobj);
        }
        return result;