Mingming Cao wrote:
> [PATCH] jbd2 stats through procfs
>
> The patch below updates the jbd stats patch to 2.6.20/jbd2.
> The initial patch was posted by Alex Tomas in December 2005
> (http://marc.info/?l=linux-ext4&m=113538565128617&w=2).
> It provides statistics via procfs such as transaction lifetime and size.
>
> [ This probably should be rewritten to use debugfs? -- Ted]
>
> Signed-off-by: Johann Lombardi <[email protected]>
I've started going through this one to clean it up to the point where it
can go forward. It's been sitting at the top of the unstable portion of
the patch queue for long enough, I think :)
I've converted the msecs to jiffies until the user boundary, changed the
union #defines as suggested by Andrew, and various other little issues etc.
Remaining to do is a generic time-difference calculator (instead of
jbd2_time_diff), and looking into whether it should be made a config
option; I tend to think it should, but it's fairly well sprinkled
through the code, so I'll see how well that works.
Also did we ever decided if this should go to debugfs?
Thanks,
-Eric
On Fri, 2007-11-30 at 17:08 -0600, Eric Sandeen wrote:
> Mingming Cao wrote:
> > [PATCH] jbd2 stats through procfs
> >
> > The patch below updates the jbd stats patch to 2.6.20/jbd2.
> > The initial patch was posted by Alex Tomas in December 2005
> > (http://marc.info/?l=linux-ext4&m=113538565128617&w=2).
> > It provides statistics via procfs such as transaction lifetime and size.
> >
> > [ This probably should be rewritten to use debugfs? -- Ted]
> >
> > Signed-off-by: Johann Lombardi <[email protected]>
>
> I've started going through this one to clean it up to the point where it
> can go forward. It's been sitting at the top of the unstable portion of
> the patch queue for long enough, I think :)
>
Thanks!
> I've converted the msecs to jiffies until the user boundary, changed the
> union #defines as suggested by Andrew, and various other little issues etc.
>
> Remaining to do is a generic time-difference calculator (instead of
> jbd2_time_diff), and looking into whether it should be made a config
> option; I tend to think it should, but it's fairly well sprinkled
> through the code, so I'll see how well that works.
>
> Also did we ever decided if this should go to debugfs?
>
I thought it was decided to keep it on procfs as debugfs is not always
on...
> Thanks,
>
> -Eric
> -
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html