2004-03-15 22:57:22

by Lever, Charles

[permalink] [raw]
Subject: RE: df hangs on down nfs server mounted with hard,intr,can't kill

hi steve-

downloaded your patches from 30 dec '03.

zeroing the RPC counters is a little outside the scope of my
patch, as i'm providing a whole new interface that is outside
the /proc/net/rpc/* hierarchy. this is work that might/should
go in separately from mine? (my patch will allow zeroing the
stats that i'm providing, which are different than the ones
already provided in /proc/net/rpc/nfs).

but one question: why do you use a hex value for this interface?
i thought /proc was supposed to be human-readable. why not use
"z" "zc" "zs" just like your nfsstat arguments so that one can
just as easily do:

echo "zs" > /proc/net/rpc/nfs

as well as

nfsstat -zs

> -----Original Message-----
> From: Steve Dickson [mailto:[email protected]]=20
> Sent: Monday, March 15, 2004 1:16 PM
> To: Trond Myklebust
> Cc: Yusuf Goolamabbas; [email protected]
> Subject: Re: [NFS] df hangs on down nfs server mounted with=20
> hard,intr, can't kill
>=20
>=20
> Trond Myklebust wrote:
>=20
> >P=E5 m=E5 , 15/03/2004 klokka 00:21, skreiv Yusuf Goolamabbas:
> > =20
> >
> >>Also, what's the status with the nfszerostats patch from=20
> Steve Dickson=20
> >>of Redhat
> >>
> >>http://people.redhat.com/steved/NFS/nfszerostats/
> >> =20
> >>
> >
> >Chuck is currently working on some enhanced NFS statistics=20
> patches. I=20
> >imagine he will take steps to include Steve's patches in his=20
> work, and=20
> >then push them to me.
> > =20
> >
> FYI... The nfs-utils code to support nfszerostats is already
> in the FC2 rawhides... I'm hopeful Neil will include this
> patch in the next nfs-utils release....
>=20
> SteveD.
>=20
>=20
>=20
>=20
> -------------------------------------------------------
> This SF.Net email is sponsored by: IBM Linux Tutorials
> Free Linux tutorial presented by Daniel Robbins, President=20
> and CEO of GenToo technologies. Learn everything from=20
> fundamentals to system=20
> administration.http://ads.osdn.com/?ad_id=1470&alloc_id638&op=3Dick
> _______________________________________________
> NFS maillist - [email protected]=20
> https://lists.sourceforge.net/lists/listinfo/n> fs
>=20


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs


2004-03-16 02:26:56

by Steve Dickson

[permalink] [raw]
Subject: Re: df hangs on down nfs server mounted with hard,intr,can't kill

Lever, Charles wrote:

>zeroing the RPC counters is a little outside the scope of my
>patch, as i'm providing a whole new interface that is outside
>the /proc/net/rpc/* hierarchy. this is work that might/should
>go in separately from mine?
>
Yes.. I agree... they are separate... and probably should stay separate.

>but one question: why do you use a hex value for this interface?
>
>
Two reasons, one the debug interface uses a numeral interface and
two, I figured since nfsstat is already converting the command line
arguments into 'flag bits', reusing those flags bits to tell the kernel
which stats should be zeroed would be fairly clean and non intrusive
way to do it.

>i thought /proc was supposed to be human-readable. why not use
>"z" "zc" "zs" just like your nfsstat arguments so that one can
>just as easily do:
>
> echo "zs" > /proc/net/rpc/nfs
>
>
> nfsstat -zs
>
>
>
Good point... I guess I just assumed the nfsstat interface would be used.

SteveD.


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
NFS maillist - [email protected]
https://lists.sourceforge.net/lists/listinfo/nfs