* [Ocfs2-devel] [PATCH] ocfs2: Add some trace log for orphan scan.
@ 2010-06-01 5:58 Tao Ma
2010-06-01 17:23 ` Sunil Mushran
2010-08-12 0:31 ` Joel Becker
0 siblings, 2 replies; 3+ messages in thread
From: Tao Ma @ 2010-06-01 5:58 UTC (permalink / raw)
To: ocfs2-devel
Now orphan scan worker has no trace log, so it is
very hard to tell whether it is finished or blocked.
So add 2 mlog trace log so that we can tell whether
the current orphan scan worker is blocked or not.
It does help when I analyzed a orphan scan bug.
Signed-off-by: Tao Ma <tao.ma@oracle.com>
---
fs/ocfs2/journal.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/fs/ocfs2/journal.c b/fs/ocfs2/journal.c
index 47878cf..776ab2f 100644
--- a/fs/ocfs2/journal.c
+++ b/fs/ocfs2/journal.c
@@ -1888,6 +1888,8 @@ void ocfs2_queue_orphan_scan(struct ocfs2_super *osb)
os = &osb->osb_orphan_scan;
+ mlog(0, "Begin orphan scan\n");
+
if (atomic_read(&os->os_state) == ORPHAN_SCAN_INACTIVE)
goto out;
@@ -1920,6 +1922,7 @@ void ocfs2_queue_orphan_scan(struct ocfs2_super *osb)
unlock:
ocfs2_orphan_scan_unlock(osb, seqno);
out:
+ mlog(0, "Orphan scan completed\n");
return;
}
--
1.5.5
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Ocfs2-devel] [PATCH] ocfs2: Add some trace log for orphan scan.
2010-06-01 5:58 [Ocfs2-devel] [PATCH] ocfs2: Add some trace log for orphan scan Tao Ma
@ 2010-06-01 17:23 ` Sunil Mushran
2010-08-12 0:31 ` Joel Becker
1 sibling, 0 replies; 3+ messages in thread
From: Sunil Mushran @ 2010-06-01 17:23 UTC (permalink / raw)
To: ocfs2-devel
Signed-off-by: Sunil Mushran<sunil.mushran@oracle.com>
On 05/31/2010 10:58 PM, Tao Ma wrote:
> Now orphan scan worker has no trace log, so it is
> very hard to tell whether it is finished or blocked.
> So add 2 mlog trace log so that we can tell whether
> the current orphan scan worker is blocked or not.
> It does help when I analyzed a orphan scan bug.
>
> Signed-off-by: Tao Ma<tao.ma@oracle.com>
> ---
> fs/ocfs2/journal.c | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/fs/ocfs2/journal.c b/fs/ocfs2/journal.c
> index 47878cf..776ab2f 100644
> --- a/fs/ocfs2/journal.c
> +++ b/fs/ocfs2/journal.c
> @@ -1888,6 +1888,8 @@ void ocfs2_queue_orphan_scan(struct ocfs2_super *osb)
>
> os =&osb->osb_orphan_scan;
>
> + mlog(0, "Begin orphan scan\n");
> +
> if (atomic_read(&os->os_state) == ORPHAN_SCAN_INACTIVE)
> goto out;
>
> @@ -1920,6 +1922,7 @@ void ocfs2_queue_orphan_scan(struct ocfs2_super *osb)
> unlock:
> ocfs2_orphan_scan_unlock(osb, seqno);
> out:
> + mlog(0, "Orphan scan completed\n");
> return;
> }
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Ocfs2-devel] [PATCH] ocfs2: Add some trace log for orphan scan.
2010-06-01 5:58 [Ocfs2-devel] [PATCH] ocfs2: Add some trace log for orphan scan Tao Ma
2010-06-01 17:23 ` Sunil Mushran
@ 2010-08-12 0:31 ` Joel Becker
1 sibling, 0 replies; 3+ messages in thread
From: Joel Becker @ 2010-08-12 0:31 UTC (permalink / raw)
To: ocfs2-devel
On Tue, Jun 01, 2010 at 01:58:13PM +0800, Tao Ma wrote:
> Now orphan scan worker has no trace log, so it is
> very hard to tell whether it is finished or blocked.
> So add 2 mlog trace log so that we can tell whether
> the current orphan scan worker is blocked or not.
> It does help when I analyzed a orphan scan bug.
>
> Signed-off-by: Tao Ma <tao.ma@oracle.com>
This patch is now in the merge-window branch of ocfs2.git.
Joel
--
"The nice thing about egotists is that they don't talk about other
people."
- Lucille S. Harper
Joel Becker
Consulting Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-08-12 0:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-06-01 5:58 [Ocfs2-devel] [PATCH] ocfs2: Add some trace log for orphan scan Tao Ma
2010-06-01 17:23 ` Sunil Mushran
2010-08-12 0:31 ` Joel Becker
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).