diff options
author | Tao Ma <boyu.mt@taobao.com> | 2011-02-20 22:10:44 -0500 |
---|---|---|
committer | Tao Ma <boyu.mt@taobao.com> | 2011-02-20 22:10:44 -0500 |
commit | ef6b689b63b9f5227ccee6f16dd9ee3faf58a464 (patch) | |
tree | ebe3416fcc221e152f2625e8e8794e92e3d8c979 /fs/ocfs2/quota_global.c | |
parent | 422e6c4bc4b48c15b3cb57a1ca71431abfc57e54 (diff) |
ocfs2: Remove ENTRY from masklog.
ENTRY is used to record the entry of a function.
But because it is added in so many functions, if we enable it,
the system logs get filled up quickly and cause too much I/O.
So actually no one can open it for a production system or even
for a test.
So for mlog_entry_void, we just remove it.
for mlog_entry(...), we replace it with mlog(0,...), and they
will be replace by trace event later.
Signed-off-by: Tao Ma <boyu.mt@taobao.com>
Diffstat (limited to 'fs/ocfs2/quota_global.c')
-rw-r--r-- | fs/ocfs2/quota_global.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/fs/ocfs2/quota_global.c b/fs/ocfs2/quota_global.c index 4607923eb24c..33397bd7e1bc 100644 --- a/fs/ocfs2/quota_global.c +++ b/fs/ocfs2/quota_global.c | |||
@@ -343,8 +343,6 @@ int ocfs2_global_read_info(struct super_block *sb, int type) | |||
343 | u64 pcount; | 343 | u64 pcount; |
344 | int status; | 344 | int status; |
345 | 345 | ||
346 | mlog_entry_void(); | ||
347 | |||
348 | /* Read global header */ | 346 | /* Read global header */ |
349 | gqinode = ocfs2_get_system_file_inode(OCFS2_SB(sb), ino[type], | 347 | gqinode = ocfs2_get_system_file_inode(OCFS2_SB(sb), ino[type], |
350 | OCFS2_INVALID_SLOT); | 348 | OCFS2_INVALID_SLOT); |
@@ -596,7 +594,7 @@ static int ocfs2_sync_dquot_helper(struct dquot *dquot, unsigned long type) | |||
596 | struct ocfs2_super *osb = OCFS2_SB(sb); | 594 | struct ocfs2_super *osb = OCFS2_SB(sb); |
597 | int status = 0; | 595 | int status = 0; |
598 | 596 | ||
599 | mlog_entry("id=%u qtype=%u type=%lu device=%s\n", dquot->dq_id, | 597 | mlog(0, "id=%u qtype=%u type=%lu device=%s\n", dquot->dq_id, |
600 | dquot->dq_type, type, sb->s_id); | 598 | dquot->dq_type, type, sb->s_id); |
601 | if (type != dquot->dq_type) | 599 | if (type != dquot->dq_type) |
602 | goto out; | 600 | goto out; |
@@ -649,7 +647,7 @@ static int ocfs2_write_dquot(struct dquot *dquot) | |||
649 | struct ocfs2_super *osb = OCFS2_SB(dquot->dq_sb); | 647 | struct ocfs2_super *osb = OCFS2_SB(dquot->dq_sb); |
650 | int status = 0; | 648 | int status = 0; |
651 | 649 | ||
652 | mlog_entry("id=%u, type=%d", dquot->dq_id, dquot->dq_type); | 650 | mlog(0, "id=%u, type=%d", dquot->dq_id, dquot->dq_type); |
653 | 651 | ||
654 | handle = ocfs2_start_trans(osb, OCFS2_QWRITE_CREDITS); | 652 | handle = ocfs2_start_trans(osb, OCFS2_QWRITE_CREDITS); |
655 | if (IS_ERR(handle)) { | 653 | if (IS_ERR(handle)) { |
@@ -688,7 +686,7 @@ static int ocfs2_release_dquot(struct dquot *dquot) | |||
688 | struct ocfs2_super *osb = OCFS2_SB(dquot->dq_sb); | 686 | struct ocfs2_super *osb = OCFS2_SB(dquot->dq_sb); |
689 | int status = 0; | 687 | int status = 0; |
690 | 688 | ||
691 | mlog_entry("id=%u, type=%d", dquot->dq_id, dquot->dq_type); | 689 | mlog(0, "id=%u, type=%d", dquot->dq_id, dquot->dq_type); |
692 | 690 | ||
693 | mutex_lock(&dquot->dq_lock); | 691 | mutex_lock(&dquot->dq_lock); |
694 | /* Check whether we are not racing with some other dqget() */ | 692 | /* Check whether we are not racing with some other dqget() */ |
@@ -745,7 +743,7 @@ static int ocfs2_acquire_dquot(struct dquot *dquot) | |||
745 | int need_alloc = ocfs2_global_qinit_alloc(sb, type); | 743 | int need_alloc = ocfs2_global_qinit_alloc(sb, type); |
746 | handle_t *handle; | 744 | handle_t *handle; |
747 | 745 | ||
748 | mlog_entry("id=%u, type=%d", dquot->dq_id, type); | 746 | mlog(0, "id=%u, type=%d", dquot->dq_id, type); |
749 | mutex_lock(&dquot->dq_lock); | 747 | mutex_lock(&dquot->dq_lock); |
750 | /* | 748 | /* |
751 | * We need an exclusive lock, because we're going to update use count | 749 | * We need an exclusive lock, because we're going to update use count |
@@ -831,7 +829,7 @@ static int ocfs2_mark_dquot_dirty(struct dquot *dquot) | |||
831 | handle_t *handle; | 829 | handle_t *handle; |
832 | struct ocfs2_super *osb = OCFS2_SB(sb); | 830 | struct ocfs2_super *osb = OCFS2_SB(sb); |
833 | 831 | ||
834 | mlog_entry("id=%u, type=%d", dquot->dq_id, type); | 832 | mlog(0, "id=%u, type=%d", dquot->dq_id, type); |
835 | 833 | ||
836 | /* In case user set some limits, sync dquot immediately to global | 834 | /* In case user set some limits, sync dquot immediately to global |
837 | * quota file so that information propagates quicker */ | 835 | * quota file so that information propagates quicker */ |
@@ -879,8 +877,6 @@ static int ocfs2_write_info(struct super_block *sb, int type) | |||
879 | int status = 0; | 877 | int status = 0; |
880 | struct ocfs2_mem_dqinfo *oinfo = sb_dqinfo(sb, type)->dqi_priv; | 878 | struct ocfs2_mem_dqinfo *oinfo = sb_dqinfo(sb, type)->dqi_priv; |
881 | 879 | ||
882 | mlog_entry_void(); | ||
883 | |||
884 | status = ocfs2_lock_global_qf(oinfo, 1); | 880 | status = ocfs2_lock_global_qf(oinfo, 1); |
885 | if (status < 0) | 881 | if (status < 0) |
886 | goto out; | 882 | goto out; |