Merge PR ceph#58437 into main

batrick · batrick · commit b18b1ef232d0 · 2024-08-06T17:05:10.000-04:00
* refs/pull/58437/head:
	mds: remove the misleading and redundant logs

Reviewed-by: Patrick Donnelly &lt;pdonnell@redhat.com&gt;
diff --git a/src/mds/MDCache.cc b/src/mds/MDCache.cc
@@ -8524,7 +8524,6 @@ int MDCache::path_traverse(const MDRequestRef& mdr, MDSContextFactory& cf,
 	  lov.add_rdlock(&dn->lock);
 	}
 	if (!mds->locker->acquire_locks(mdr, lov)) {
-	  dout(10) << "traverse: failed to rdlock " << dn->lock << " " << *dn << dendl;
 	  return 1;
 	}
       } else if (!path_locked &&
@@ -8645,7 +8644,6 @@ int MDCache::path_traverse(const MDRequestRef& mdr, MDSContextFactory& cf,
 		lov.add_rdlock(&dn->lock);
 	      }
 	      if (!mds->locker->acquire_locks(mdr, lov)) {
-		dout(10) << "traverse: failed to rdlock " << dn->lock << " " << *dn << dendl;
 		return 1;
 	      }
 	    }

Original file line number	Diff line number	Diff line change
`@@ -8524,7 +8524,6 @@ int MDCache::path_traverse(const MDRequestRef& mdr, MDSContextFactory& cf,`
`8524`	`8524`	`lov.add_rdlock(&dn->lock);`
`8525`	`8525`	`}`
`8526`	`8526`	`if (!mds->locker->acquire_locks(mdr, lov)) {`
`8527`		`- dout(10) << "traverse: failed to rdlock " << dn->lock << " " << *dn << dendl;`
`8528`	`8527`	`return 1;`
`8529`	`8528`	`}`
`8530`	`8529`	`} else if (!path_locked &&`
`@@ -8645,7 +8644,6 @@ int MDCache::path_traverse(const MDRequestRef& mdr, MDSContextFactory& cf,`
`8645`	`8644`	`lov.add_rdlock(&dn->lock);`
`8646`	`8645`	`}`
`8647`	`8646`	`if (!mds->locker->acquire_locks(mdr, lov)) {`
`8648`		`- dout(10) << "traverse: failed to rdlock " << dn->lock << " " << *dn << dendl;`
`8649`	`8647`	`return 1;`
`8650`	`8648`	`}`
`8651`	`8649`	`}`