Whamcloud - gitweb
LU-1095 ofd: Silence 'slow create' warnings 94/10394/3
authorBrian Behlendorf <behlendorf1@llnl.gov>
Tue, 20 May 2014 22:31:39 +0000 (15:31 -0700)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 23 Jun 2014 00:03:06 +0000 (00:03 +0000)
These messages were originally added to analyze a performance issue.
That issue is now much better understand and no longer needs to
be logged so verbosely.  They probably should have been disabled
prior to Lustre 2.4 being released but this was overlooked.

Change-Id: I395de36cdccc82afd4ed7b9c424d12d17b881f54
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Reviewed-on: http://review.whamcloud.com/10394
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ofd/ofd_dev.c
lustre/osp/osp_precreate.c

index 6351855..25debd1 100644 (file)
@@ -1379,11 +1379,10 @@ static int ofd_create_hdl(struct tgt_session_info *tsi)
                               count, seq, next_id);
 
                        if (cfs_time_after(jiffies, enough_time)) {
-                               LCONSOLE_WARN("%s: Slow creates, %d/%d objects"
-                                             " created at a rate of %d/s\n",
-                                             ofd_name(ofd), created,
-                                             diff + created,
-                                             created / DISK_TIMEOUT);
+                               CDEBUG(D_HA, "%s: Slow creates, %d/%d objects"
+                                     " created at a rate of %d/s\n",
+                                     ofd_name(ofd), created, diff + created,
+                                     created / DISK_TIMEOUT);
                                break;
                        }
 
index 219d90f..e6f81cd 100644 (file)
@@ -1017,13 +1017,13 @@ static int osp_precreate_timeout_condition(void *data)
 {
        struct osp_device *d = data;
 
-       LCONSOLE_WARN("%s: slow creates, last="DFID", next="DFID", "
-                     "reserved="LPU64", syn_changes=%lu, "
-                     "syn_rpc_in_progress=%d, status=%d\n",
-                     d->opd_obd->obd_name, PFID(&d->opd_pre_last_created_fid),
-                     PFID(&d->opd_pre_used_fid), d->opd_pre_reserved,
-                     d->opd_syn_changes, d->opd_syn_rpc_in_progress,
-                     d->opd_pre_status);
+       CDEBUG(D_HA, "%s: slow creates, last="DFID", next="DFID", "
+             "reserved="LPU64", syn_changes=%lu, "
+             "syn_rpc_in_progress=%d, status=%d\n",
+             d->opd_obd->obd_name, PFID(&d->opd_pre_last_created_fid),
+             PFID(&d->opd_pre_used_fid), d->opd_pre_reserved,
+             d->opd_syn_changes, d->opd_syn_rpc_in_progress,
+             d->opd_pre_status);
 
        return 1;
 }