From f8412db9911dbbfe0e25d357400cb17b9a7402bd Mon Sep 17 00:00:00 2001 From: deen Date: Sat, 28 Jun 2008 10:04:50 +0000 Subject: [PATCH] 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 --- lustre/ChangeLog | 8 ++++++++ lustre/ldlm/ldlm_resource.c | 6 +++--- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/lustre/ChangeLog b/lustre/ChangeLog index 27df13c..be61455 100644 --- 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. diff --git a/lustre/ldlm/ldlm_resource.c b/lustre/ldlm/ldlm_resource.c index 87c53cb..84b72fd 100644 --- 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"); -- 1.8.3.1