From fcaff5d7cc2cd4d6fecbf641b8a3ce4f09490e3d Mon Sep 17 00:00:00 2001 From: Andrew Perepechko Date: Wed, 28 Jun 2017 12:24:26 +0300 Subject: [PATCH] LU-9716 osc: osc_extent_tree_dump0() implementation is suboptimal Avoid looping in osc_extent_tree_dump() if debugging is disabled. This helps us save some cpu ticks. Lustre-change: https://review.whamcloud.com/27866 Lustre-commit: 97115ccd159e4503ca16cb7f68ee7479c780f1cf Change-Id: I492429d8a6de79f67b5923895ffa58b7fe3a100d Seagate-bug-id: MRP-4469 Signed-off-by: Andrew Perepechko Reviewed-by: Alexander Boyko Reviewed-by: Alexander Zarochentsev Reviewed-by: Andreas Dilger Signed-off-by: Minh Diep Reviewed-on: https://review.whamcloud.com/28235 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Mike Pershin Reviewed-by: John L. Hammond --- lustre/osc/osc_cache.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lustre/osc/osc_cache.c b/lustre/osc/osc_cache.c index 43c4b6f..3e9e29e 100644 --- a/lustre/osc/osc_cache.c +++ b/lustre/osc/osc_cache.c @@ -1226,6 +1226,9 @@ static void osc_extent_tree_dump0(int level, struct osc_object *obj, struct osc_extent *ext; int cnt; + if (!cfs_cdebug_show(level, DEBUG_SUBSYSTEM)) + return; + CDEBUG(level, "Dump object %p extents at %s:%d, mppr: %u.\n", obj, func, line, osc_cli(obj)->cl_max_pages_per_rpc); -- 1.8.3.1