In ldlm_resource_add_lock(), call to ldlm_resource_dump()

starve other threads from the resource lock for a long time in
case of long waiting queue, so change the debug level from
D_OTHER to the less frequently used D_INFO.

b=15953
i=adilger
i=nathan.rutman

diff --git a/lustre/ChangeLog b/lustre/ChangeLog
index 27df13c..be61455 100644 (file)
--- a/lustre/ChangeLog
+++ b/lustre/ChangeLog
@@ -1112,6 +1112,14 @@ Details    : Directly associate cached pages to lock that protect those pages,
              this allows us to quickly find what pages to write and remove
             once lock callback is received.
 
+Severity   : normal
+Bugzilla   : 15953
+Description: more ldlm soft lockups
+Details    : In ldlm_resource_add_lock(), call to ldlm_resource_dump()
+             starve other threads from the resource lock for a long time in
+             case of long waiting queue, so change the debug level from
+             D_OTHER to the less frequently used D_INFO.
+
 --------------------------------------------------------------------------------
 
 2007-08-10         Cluster File Systems, Inc. <info@clusterfs.com>

diff --git a/lustre/ldlm/ldlm_resource.c b/lustre/ldlm/ldlm_resource.c
index 87c53cb..84b72fd 100644 (file)
--- a/lustre/ldlm/ldlm_resource.c
+++ b/lustre/ldlm/ldlm_resource.c
@@ -1008,9 +1008,9 @@ void ldlm_resource_add_lock(struct ldlm_resource *res, struct list_head *head,
 {
         check_res_locked(res);
 
-        ldlm_resource_dump(D_OTHER, res);
-        CDEBUG(D_OTHER, "About to add this lock:\n");
-        ldlm_lock_dump(D_OTHER, lock, 0);
+        ldlm_resource_dump(D_INFO, res);
+        CDEBUG(D_INFO, "About to add this lock:\n");
+        ldlm_lock_dump(D_INFO, lock, 0);
 
         if (lock->l_destroyed) {
                 CDEBUG(D_OTHER, "Lock destroyed, not adding to resource\n");