On Wed, Sep 21, 2011 at 02:31:12PM -0400, Steve Dickson wrote:
>
>
> On 09/21/2011 02:25 PM, Bryan Schumaker wrote:
> > On 09/21/2011 02:16 PM, Steve Dickson wrote:
> >> Both parser of nfsiostat and mountstats depend on the
> >> the first word in /proc/self/mountstats to be 'device'
> >> for them to parse through all the devices.
> >>
> >> NFS mounts entries, in 3.1, no longer have start with
> >> the word 'device'. They start with the actual device,
> >> which confused both set of parsing routines.
> >
> > I sent in a patch to fix this on the kernel end almost a month ago, but it must have gotten lost with the kernel.org confusion. I included what I came up with below
> Interesting... Maybe we should still apply these patches until
> your patch shows up? They are definitely broken...
May as well fix the problem on both sides if possible.
--b.
>
> steved.
> .
> >
> > - Bryan
> >
> > [PATCH] VFS: Add "device" tag to /proc/self/mountstats
> >
> > nfsiostat was failing to find mounted filesystems on kernels after
> > 2.6.38 because of changes to show_vfsstat() by commit
> > c7f404b40a3665d9f4e9a927cc5c1ee0479ed8f9. This patch adds back the
> > "device" tag before the nfs server entry so scripts can parse the
> > mountstats file correctly.
> >
> > Signed-off-by: Bryan Schumaker <bjschuma@xxxxxxxxxx>
> > ---
> > fs/namespace.c | 1 +
> > 1 files changed, 1 insertions(+), 0 deletions(-)
> >
> > diff --git a/fs/namespace.c b/fs/namespace.c
> > index 22bfe82..1429114 100644
> > --- a/fs/namespace.c
> > +++ b/fs/namespace.c
> > @@ -1109,6 +1109,7 @@ static int show_vfsstat(struct seq_file *m, void *v)
> >
> > /* device */
> > if (mnt->mnt_sb->s_op->show_devname) {
> > + seq_puts(m, "device ");
> > err = mnt->mnt_sb->s_op->show_devname(m, mnt);
> > } else {
> > if (mnt->mnt_devname) {
> >
> >
> >>
> >> Steve Dickson (2):
> >> nfsiostat: Breaks on 3.1 kernels
> >> mountstats: Breaks on 3.1 kernels
> >>
> >> tools/mountstats/mountstats.py | 9 +++++++++
> >> tools/nfs-iostat/nfs-iostat.py | 11 +++++++++--
> >> 2 files changed, 18 insertions(+), 2 deletions(-)
> >>
> >
> --
> To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
