Skip to content
Snippets Groups Projects
Commit f8412db9 authored by deen's avatar deen
Browse files

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
parent 895a50d6
No related branches found
No related tags found
No related merge requests found
......@@ -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>
......
......@@ -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");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment