Whamcloud - gitweb
LU-9716 osc: osc_extent_tree_dump0() implementation is suboptimal 35/28235/2
authorAndrew Perepechko <andrew.perepechko@seagate.com>
Wed, 28 Jun 2017 09:24:26 +0000 (12:24 +0300)
committerJohn L. Hammond <john.hammond@intel.com>
Mon, 7 Aug 2017 18:50:34 +0000 (18:50 +0000)
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 <andrew.perepechko@seagate.com>
Reviewed-by: Alexander Boyko <alexander.boyko@seagate.com>
Reviewed-by: Alexander Zarochentsev <alexander.zarochentsev@seagate.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Signed-off-by: Minh Diep <minh.diep@intel.com>
Reviewed-on: https://review.whamcloud.com/28235
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Reviewed-by: John L. Hammond <john.hammond@intel.com>
lustre/osc/osc_cache.c

index 43c4b6f..3e9e29e 100644 (file)
@@ -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);