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/super.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/super.c')
-rw-r--r-- | fs/ocfs2/super.c | 32 |
1 files changed, 6 insertions, 26 deletions
diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c index 36c423fb0635..21c4bfdfd739 100644 --- a/fs/ocfs2/super.c +++ b/fs/ocfs2/super.c | |||
@@ -441,8 +441,6 @@ static int ocfs2_init_global_system_inodes(struct ocfs2_super *osb) | |||
441 | int status = 0; | 441 | int status = 0; |
442 | int i; | 442 | int i; |
443 | 443 | ||
444 | mlog_entry_void(); | ||
445 | |||
446 | new = ocfs2_iget(osb, osb->root_blkno, OCFS2_FI_FLAG_SYSFILE, 0); | 444 | new = ocfs2_iget(osb, osb->root_blkno, OCFS2_FI_FLAG_SYSFILE, 0); |
447 | if (IS_ERR(new)) { | 445 | if (IS_ERR(new)) { |
448 | status = PTR_ERR(new); | 446 | status = PTR_ERR(new); |
@@ -488,8 +486,6 @@ static int ocfs2_init_local_system_inodes(struct ocfs2_super *osb) | |||
488 | int status = 0; | 486 | int status = 0; |
489 | int i; | 487 | int i; |
490 | 488 | ||
491 | mlog_entry_void(); | ||
492 | |||
493 | for (i = OCFS2_LAST_GLOBAL_SYSTEM_INODE + 1; | 489 | for (i = OCFS2_LAST_GLOBAL_SYSTEM_INODE + 1; |
494 | i < NUM_SYSTEM_INODES; | 490 | i < NUM_SYSTEM_INODES; |
495 | i++) { | 491 | i++) { |
@@ -517,8 +513,6 @@ static void ocfs2_release_system_inodes(struct ocfs2_super *osb) | |||
517 | int i; | 513 | int i; |
518 | struct inode *inode; | 514 | struct inode *inode; |
519 | 515 | ||
520 | mlog_entry_void(); | ||
521 | |||
522 | for (i = 0; i < NUM_GLOBAL_SYSTEM_INODES; i++) { | 516 | for (i = 0; i < NUM_GLOBAL_SYSTEM_INODES; i++) { |
523 | inode = osb->global_system_inodes[i]; | 517 | inode = osb->global_system_inodes[i]; |
524 | if (inode) { | 518 | if (inode) { |
@@ -1032,7 +1026,7 @@ static int ocfs2_fill_super(struct super_block *sb, void *data, int silent) | |||
1032 | char nodestr[8]; | 1026 | char nodestr[8]; |
1033 | struct ocfs2_blockcheck_stats stats; | 1027 | struct ocfs2_blockcheck_stats stats; |
1034 | 1028 | ||
1035 | mlog_entry("%p, %p, %i", sb, data, silent); | 1029 | mlog(0, "%p, %p, %i", sb, data, silent); |
1036 | 1030 | ||
1037 | if (!ocfs2_parse_options(sb, data, &parsed_options, 0)) { | 1031 | if (!ocfs2_parse_options(sb, data, &parsed_options, 0)) { |
1038 | status = -EINVAL; | 1032 | status = -EINVAL; |
@@ -1320,8 +1314,8 @@ static int ocfs2_parse_options(struct super_block *sb, | |||
1320 | char *p; | 1314 | char *p; |
1321 | u32 tmp; | 1315 | u32 tmp; |
1322 | 1316 | ||
1323 | mlog_entry("remount: %d, options: \"%s\"\n", is_remount, | 1317 | mlog(0, "remount: %d, options: \"%s\"\n", is_remount, |
1324 | options ? options : "(none)"); | 1318 | options ? options : "(none)"); |
1325 | 1319 | ||
1326 | mopt->commit_interval = 0; | 1320 | mopt->commit_interval = 0; |
1327 | mopt->mount_opt = OCFS2_MOUNT_NOINTR; | 1321 | mopt->mount_opt = OCFS2_MOUNT_NOINTR; |
@@ -1629,8 +1623,6 @@ static int __init ocfs2_init(void) | |||
1629 | { | 1623 | { |
1630 | int status; | 1624 | int status; |
1631 | 1625 | ||
1632 | mlog_entry_void(); | ||
1633 | |||
1634 | ocfs2_print_version(); | 1626 | ocfs2_print_version(); |
1635 | 1627 | ||
1636 | status = init_ocfs2_uptodate_cache(); | 1628 | status = init_ocfs2_uptodate_cache(); |
@@ -1681,8 +1673,6 @@ leave: | |||
1681 | 1673 | ||
1682 | static void __exit ocfs2_exit(void) | 1674 | static void __exit ocfs2_exit(void) |
1683 | { | 1675 | { |
1684 | mlog_entry_void(); | ||
1685 | |||
1686 | ocfs2_quota_shutdown(); | 1676 | ocfs2_quota_shutdown(); |
1687 | 1677 | ||
1688 | if (ocfs2_wq) { | 1678 | if (ocfs2_wq) { |
@@ -1705,7 +1695,7 @@ static void __exit ocfs2_exit(void) | |||
1705 | 1695 | ||
1706 | static void ocfs2_put_super(struct super_block *sb) | 1696 | static void ocfs2_put_super(struct super_block *sb) |
1707 | { | 1697 | { |
1708 | mlog_entry("(0x%p)\n", sb); | 1698 | mlog(0, "(0x%p)\n", sb); |
1709 | 1699 | ||
1710 | ocfs2_sync_blockdev(sb); | 1700 | ocfs2_sync_blockdev(sb); |
1711 | ocfs2_dismount_volume(sb, 0); | 1701 | ocfs2_dismount_volume(sb, 0); |
@@ -1722,7 +1712,7 @@ static int ocfs2_statfs(struct dentry *dentry, struct kstatfs *buf) | |||
1722 | struct buffer_head *bh = NULL; | 1712 | struct buffer_head *bh = NULL; |
1723 | struct inode *inode = NULL; | 1713 | struct inode *inode = NULL; |
1724 | 1714 | ||
1725 | mlog_entry("(%p, %p)\n", dentry->d_sb, buf); | 1715 | mlog(0, "(%p, %p)\n", dentry->d_sb, buf); |
1726 | 1716 | ||
1727 | osb = OCFS2_SB(dentry->d_sb); | 1717 | osb = OCFS2_SB(dentry->d_sb); |
1728 | 1718 | ||
@@ -1889,8 +1879,6 @@ static int ocfs2_mount_volume(struct super_block *sb) | |||
1889 | int unlock_super = 0; | 1879 | int unlock_super = 0; |
1890 | struct ocfs2_super *osb = OCFS2_SB(sb); | 1880 | struct ocfs2_super *osb = OCFS2_SB(sb); |
1891 | 1881 | ||
1892 | mlog_entry_void(); | ||
1893 | |||
1894 | if (ocfs2_is_hard_readonly(osb)) | 1882 | if (ocfs2_is_hard_readonly(osb)) |
1895 | goto leave; | 1883 | goto leave; |
1896 | 1884 | ||
@@ -1945,7 +1933,7 @@ static void ocfs2_dismount_volume(struct super_block *sb, int mnt_err) | |||
1945 | struct ocfs2_super *osb = NULL; | 1933 | struct ocfs2_super *osb = NULL; |
1946 | char nodestr[8]; | 1934 | char nodestr[8]; |
1947 | 1935 | ||
1948 | mlog_entry("(0x%p)\n", sb); | 1936 | mlog(0, "(0x%p)\n", sb); |
1949 | 1937 | ||
1950 | BUG_ON(!sb); | 1938 | BUG_ON(!sb); |
1951 | osb = OCFS2_SB(sb); | 1939 | osb = OCFS2_SB(sb); |
@@ -2097,8 +2085,6 @@ static int ocfs2_initialize_super(struct super_block *sb, | |||
2097 | struct ocfs2_super *osb; | 2085 | struct ocfs2_super *osb; |
2098 | u64 total_blocks; | 2086 | u64 total_blocks; |
2099 | 2087 | ||
2100 | mlog_entry_void(); | ||
2101 | |||
2102 | osb = kzalloc(sizeof(struct ocfs2_super), GFP_KERNEL); | 2088 | osb = kzalloc(sizeof(struct ocfs2_super), GFP_KERNEL); |
2103 | if (!osb) { | 2089 | if (!osb) { |
2104 | status = -ENOMEM; | 2090 | status = -ENOMEM; |
@@ -2403,8 +2389,6 @@ static int ocfs2_verify_volume(struct ocfs2_dinode *di, | |||
2403 | { | 2389 | { |
2404 | int status = -EAGAIN; | 2390 | int status = -EAGAIN; |
2405 | 2391 | ||
2406 | mlog_entry_void(); | ||
2407 | |||
2408 | if (memcmp(di->i_signature, OCFS2_SUPER_BLOCK_SIGNATURE, | 2392 | if (memcmp(di->i_signature, OCFS2_SUPER_BLOCK_SIGNATURE, |
2409 | strlen(OCFS2_SUPER_BLOCK_SIGNATURE)) == 0) { | 2393 | strlen(OCFS2_SUPER_BLOCK_SIGNATURE)) == 0) { |
2410 | /* We have to do a raw check of the feature here */ | 2394 | /* We have to do a raw check of the feature here */ |
@@ -2472,8 +2456,6 @@ static int ocfs2_check_volume(struct ocfs2_super *osb) | |||
2472 | * recover | 2456 | * recover |
2473 | * ourselves. */ | 2457 | * ourselves. */ |
2474 | 2458 | ||
2475 | mlog_entry_void(); | ||
2476 | |||
2477 | /* Init our journal object. */ | 2459 | /* Init our journal object. */ |
2478 | status = ocfs2_journal_init(osb->journal, &dirty); | 2460 | status = ocfs2_journal_init(osb->journal, &dirty); |
2479 | if (status < 0) { | 2461 | if (status < 0) { |
@@ -2568,8 +2550,6 @@ finally: | |||
2568 | */ | 2550 | */ |
2569 | static void ocfs2_delete_osb(struct ocfs2_super *osb) | 2551 | static void ocfs2_delete_osb(struct ocfs2_super *osb) |
2570 | { | 2552 | { |
2571 | mlog_entry_void(); | ||
2572 | |||
2573 | /* This function assumes that the caller has the main osb resource */ | 2553 | /* This function assumes that the caller has the main osb resource */ |
2574 | 2554 | ||
2575 | ocfs2_free_slot_info(osb); | 2555 | ocfs2_free_slot_info(osb); |