public inbox for linux-nfs@vger.kernel.org
 help / color / mirror / Atom feed
* Too man programs calls nfsiostat.
@ 2013-04-09 23:46 NeilBrown
  2014-01-21  3:00 ` Too many " NeilBrown
  2014-01-21  3:45 ` NeilBrown
  0 siblings, 2 replies; 13+ messages in thread
From: NeilBrown @ 2013-04-09 23:46 UTC (permalink / raw)
  To: chuck.level, sysstat; +Cc: linux-nfs

[-- Attachment #1: Type: text/plain, Size: 562 bytes --]


Hi all,
 it seems that in 2010, a program called nfsiostat was added to the sysstat
 package (http://sebastien.godard.pagesperso-orange.fr/)

 This is unfortunate because in 2008 a program called nfsiostat was added to
 the nfs-utils package.
    (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)

 The default install locations seem to be different (/usr/bin vs /usr/sbin),
 so packagers would not immediately notice.

 This seems like something that should be "fixed".

 Any suggestions what a good "fix" should look like?

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs calls nfsiostat.
  2013-04-09 23:46 Too man programs calls nfsiostat NeilBrown
@ 2014-01-21  3:00 ` NeilBrown
  2014-01-21  3:45 ` NeilBrown
  1 sibling, 0 replies; 13+ messages in thread
From: NeilBrown @ 2014-01-21  3:00 UTC (permalink / raw)
  To: chuck.level, varekova; +Cc: sysstat, linux-nfs

[-- Attachment #1: Type: text/plain, Size: 984 bytes --]


Not quite a year ago I wrote:

On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:

> 
> Hi all,
>  it seems that in 2010, a program called nfsiostat was added to the sysstat
>  package (http://sebastien.godard.pagesperso-orange.fr/)
> 
>  This is unfortunate because in 2008 a program called nfsiostat was added to
>  the nfs-utils package.
>     (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
> 
>  The default install locations seem to be different (/usr/bin vs /usr/sbin),
>  so packagers would not immediately notice.
> 
>  This seems like something that should be "fixed".
> 
>  Any suggestions what a good "fix" should look like?
> 
> NeilBrown


No suggestions were forthcoming.
So I have a suggestion.
I propose we rename the "nfsiostat" in the "sysstat" package to
"nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
"nfs-utils-iostat".

Does anyone object?  If not I'll send off patches shortly :-)

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs calls nfsiostat.
  2013-04-09 23:46 Too man programs calls nfsiostat NeilBrown
  2014-01-21  3:00 ` Too many " NeilBrown
@ 2014-01-21  3:45 ` NeilBrown
  2014-01-21 15:40   ` Chuck Lever
  1 sibling, 1 reply; 13+ messages in thread
From: NeilBrown @ 2014-01-21  3:45 UTC (permalink / raw)
  To: chuck.lever, varekova; +Cc: sysstat, linux-nfs

[-- Attachment #1: Type: text/plain, Size: 1044 bytes --]


(This time without the typo in Chuck's address - sorry).

Not quite a year ago I wrote:

On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:

> 
> Hi all,
>  it seems that in 2010, a program called nfsiostat was added to the sysstat
>  package (http://sebastien.godard.pagesperso-orange.fr/)
> 
>  This is unfortunate because in 2008 a program called nfsiostat was added to
>  the nfs-utils package.
>     (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
> 
>  The default install locations seem to be different (/usr/bin vs /usr/sbin),
>  so packagers would not immediately notice.
> 
>  This seems like something that should be "fixed".
> 
>  Any suggestions what a good "fix" should look like?
> 
> NeilBrown


No suggestions were forthcoming.
So I have a suggestion.
I propose we rename the "nfsiostat" in the "sysstat" package to
"nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
"nfs-utils-iostat".

Does anyone object?  If not I'll send off patches shortly :-)

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs calls nfsiostat.
  2014-01-21  3:45 ` NeilBrown
@ 2014-01-21 15:40   ` Chuck Lever
  2014-01-22  0:24     ` Too many programs called nfsiostat NeilBrown
  0 siblings, 1 reply; 13+ messages in thread
From: Chuck Lever @ 2014-01-21 15:40 UTC (permalink / raw)
  To: Neil Brown; +Cc: varekova, sysstat, Linux NFS Mailing List


On Jan 20, 2014, at 10:45 PM, NeilBrown <neilb@suse.de> wrote:

> 
> (This time without the typo in Chuck's address - sorry).
> 
> Not quite a year ago I wrote:
> 
> On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:
> 
>> 
>> Hi all,
>> it seems that in 2010, a program called nfsiostat was added to the sysstat
>> package (http://sebastien.godard.pagesperso-orange.fr/)
>> 
>> This is unfortunate because in 2008 a program called nfsiostat was added to
>> the nfs-utils package.
>>    (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
>> 
>> The default install locations seem to be different (/usr/bin vs /usr/sbin),
>> so packagers would not immediately notice.
>> 
>> This seems like something that should be "fixed".
>> 
>> Any suggestions what a good "fix" should look like?
>> 
>> NeilBrown
> 
> 
> No suggestions were forthcoming.
> So I have a suggestion.
> I propose we rename the "nfsiostat" in the "sysstat" package to
> "nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
> "nfs-utils-iostat".
> 
> Does anyone object?  If not I'll send off patches shortly :-)

What does the sysstat version do?  do we need two programs, or can they be merged?  Or can one be dropped?

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com




^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-21 15:40   ` Chuck Lever
@ 2014-01-22  0:24     ` NeilBrown
  2014-01-22 13:12       ` Steve Dickson
  0 siblings, 1 reply; 13+ messages in thread
From: NeilBrown @ 2014-01-22  0:24 UTC (permalink / raw)
  To: Chuck Lever; +Cc: varekova, sysstat, Linux NFS Mailing List

[-- Attachment #1: Type: text/plain, Size: 2422 bytes --]

On Tue, 21 Jan 2014 10:40:01 -0500 Chuck Lever <chuck.lever@oracle.com> wrote:

> 
> On Jan 20, 2014, at 10:45 PM, NeilBrown <neilb@suse.de> wrote:
> 
> > 
> > (This time without the typo in Chuck's address - sorry).
> > 
> > Not quite a year ago I wrote:
> > 
> > On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:
> > 
> >> 
> >> Hi all,
> >> it seems that in 2010, a program called nfsiostat was added to the sysstat
> >> package (http://sebastien.godard.pagesperso-orange.fr/)
> >> 
> >> This is unfortunate because in 2008 a program called nfsiostat was added to
> >> the nfs-utils package.
> >>    (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
> >> 
> >> The default install locations seem to be different (/usr/bin vs /usr/sbin),
> >> so packagers would not immediately notice.
> >> 
> >> This seems like something that should be "fixed".
> >> 
> >> Any suggestions what a good "fix" should look like?
> >> 
> >> NeilBrown
> > 
> > 
> > No suggestions were forthcoming.
> > So I have a suggestion.
> > I propose we rename the "nfsiostat" in the "sysstat" package to
> > "nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
> > "nfs-utils-iostat".
> > 
> > Does anyone object?  If not I'll send off patches shortly :-)
> 
> What does the sysstat version do?  do we need two programs, or can they be merged?  Or can one be dropped?
> 

It reads /proc/self/mountstats and reports some numbers, not unlike the
nfs-utils version.

$ /usr/sbin/nfsiostat 

eli:/home mounted on /mnt:

   op/s		rpc bklog
   3.55	   0.00
read:            ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
write:           ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
$ /usr/bin/nfsiostat 
Linux 3.11.6-4-desktop (notabene.brown) 	01/22/2014 	_x86_64_	(8 CPU)

Filesystem:               rkB_nor/s    wkB_nor/s    rkB_dir/s    wkB_dir/s    rkB_svr/s    wkB_svr/s     ops/s    rops/s    wops/s
eli:/home                      0.00         0.00         0.00         0.00         0.00         0.00      0.00      0.00      0.00


The man page for the sysstat version is:

http://www.makelinux.com/man/1/N/nfsiostat

while man page for nfs-utils version is:

http://linux.die.net/man/8/nfsiostat

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-22  0:24     ` Too many programs called nfsiostat NeilBrown
@ 2014-01-22 13:12       ` Steve Dickson
  2014-01-22 21:41         ` Sebastien Godard
  2014-01-22 21:52         ` NeilBrown
  0 siblings, 2 replies; 13+ messages in thread
From: Steve Dickson @ 2014-01-22 13:12 UTC (permalink / raw)
  To: NeilBrown, Chuck Lever; +Cc: varekova, sysstat, Linux NFS Mailing List



On 21/01/14 19:24, NeilBrown wrote:
> On Tue, 21 Jan 2014 10:40:01 -0500 Chuck Lever <chuck.lever@oracle.com> wrote:
> 
>>
>> On Jan 20, 2014, at 10:45 PM, NeilBrown <neilb@suse.de> wrote:
>>
>>>
>>> (This time without the typo in Chuck's address - sorry).
>>>
>>> Not quite a year ago I wrote:
>>>
>>> On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:
>>>
>>>>
>>>> Hi all,
>>>> it seems that in 2010, a program called nfsiostat was added to the sysstat
>>>> package (http://sebastien.godard.pagesperso-orange.fr/)
>>>>
>>>> This is unfortunate because in 2008 a program called nfsiostat was added to
>>>> the nfs-utils package.
>>>>    (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
>>>>
>>>> The default install locations seem to be different (/usr/bin vs /usr/sbin),
>>>> so packagers would not immediately notice.
>>>>
>>>> This seems like something that should be "fixed".
>>>>
>>>> Any suggestions what a good "fix" should look like?
>>>>
>>>> NeilBrown
>>>
>>>
>>> No suggestions were forthcoming.
>>> So I have a suggestion.
>>> I propose we rename the "nfsiostat" in the "sysstat" package to
>>> "nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
>>> "nfs-utils-iostat".
>>>
>>> Does anyone object?  If not I'll send off patches shortly :-)
>>
>> What does the sysstat version do?  do we need two programs, or can they be merged?  Or can one be dropped?
>>
> 
> It reads /proc/self/mountstats and reports some numbers, not unlike the
> nfs-utils version.
> 
> $ /usr/sbin/nfsiostat 
> 
> eli:/home mounted on /mnt:
> 
>    op/s		rpc bklog
>    3.55	   0.00
> read:            ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
> 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
> write:           ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
> 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
> $ /usr/bin/nfsiostat 
> Linux 3.11.6-4-desktop (notabene.brown) 	01/22/2014 	_x86_64_	(8 CPU)
> 
> Filesystem:               rkB_nor/s    wkB_nor/s    rkB_dir/s    wkB_dir/s    rkB_svr/s    wkB_svr/s     ops/s    rops/s    wops/s
> eli:/home                      0.00         0.00         0.00         0.00         0.00         0.00      0.00      0.00      0.00
> 
> 
> The man page for the sysstat version is:
> 
> http://www.makelinux.com/man/1/N/nfsiostat
Is this version still actively being maintain? 

steved.

> 
> while man page for nfs-utils version is:
> 
> http://linux.die.net/man/8/nfsiostat
> 
> NeilBrown
> 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-22 13:12       ` Steve Dickson
@ 2014-01-22 21:41         ` Sebastien Godard
  2014-01-22 21:52         ` NeilBrown
  1 sibling, 0 replies; 13+ messages in thread
From: Sebastien Godard @ 2014-01-22 21:41 UTC (permalink / raw)
  To: Steve Dickson, NeilBrown, Chuck Lever
  Cc: varekova, Linux NFS Mailing List, Peter Schiffer

On 01/22/2014 02:12 PM, Steve Dickson wrote:
>
> On 21/01/14 19:24, NeilBrown wrote:
>> On Tue, 21 Jan 2014 10:40:01 -0500 Chuck Lever <chuck.lever@oracle.com> wrote:
>>
>>> On Jan 20, 2014, at 10:45 PM, NeilBrown <neilb@suse.de> wrote:
>>>
>>>> (This time without the typo in Chuck's address - sorry).
>>>>
>>>> Not quite a year ago I wrote:
>>>>
>>>> On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:
>>>>
>>>>> Hi all,
>>>>> it seems that in 2010, a program called nfsiostat was added to the sysstat
>>>>> package (http://sebastien.godard.pagesperso-orange.fr/)
>>>>>
>>>>> This is unfortunate because in 2008 a program called nfsiostat was added to
>>>>> the nfs-utils package.
>>>>>     (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
>>>>>
>>>>> The default install locations seem to be different (/usr/bin vs /usr/sbin),
>>>>> so packagers would not immediately notice.
>>>>>
>>>>> This seems like something that should be "fixed".
>>>>>
>>>>> Any suggestions what a good "fix" should look like?
>>>>>
>>>>> NeilBrown
>>>>
>>>> No suggestions were forthcoming.
>>>> So I have a suggestion.
>>>> I propose we rename the "nfsiostat" in the "sysstat" package to
>>>> "nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
>>>> "nfs-utils-iostat".
>>>>
>>>> Does anyone object?  If not I'll send off patches shortly :-)
>>> What does the sysstat version do?  do we need two programs, or can they be merged?  Or can one be dropped?
>>>
>> It reads /proc/self/mountstats and reports some numbers, not unlike the
>> nfs-utils version.
>>
>> $ /usr/sbin/nfsiostat
>>
>> eli:/home mounted on /mnt:
>>
>>     op/s		rpc bklog
>>     3.55	   0.00
>> read:            ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
>> 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
>> write:           ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
>> 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
>> $ /usr/bin/nfsiostat
>> Linux 3.11.6-4-desktop (notabene.brown) 	01/22/2014 	_x86_64_	(8 CPU)
>>
>> Filesystem:               rkB_nor/s    wkB_nor/s    rkB_dir/s    wkB_dir/s    rkB_svr/s    wkB_svr/s     ops/s    rops/s    wops/s
>> eli:/home                      0.00         0.00         0.00         0.00         0.00         0.00      0.00      0.00      0.00
>>
>>
>> The man page for the sysstat version is:
>>
>> http://www.makelinux.com/man/1/N/nfsiostat
> Is this version still actively being maintain?
>
> steved.
>
>> while man page for nfs-utils version is:
>>
>> http://linux.die.net/man/8/nfsiostat
>>
>> NeilBrown
>>
>
The nfsiostat command from the sysstat package is actually no longer 
actively being maintained.
So I think that it can be merged with that from nfs-utils package.

Regards,

-- 
Sebastien GODARD <sysstat [at] orange.fr>
http://sebastien.godard.pagesperso-orange.fr/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-22 13:12       ` Steve Dickson
  2014-01-22 21:41         ` Sebastien Godard
@ 2014-01-22 21:52         ` NeilBrown
  2014-01-23 17:42           ` Steve Dickson
  1 sibling, 1 reply; 13+ messages in thread
From: NeilBrown @ 2014-01-22 21:52 UTC (permalink / raw)
  To: Steve Dickson; +Cc: Chuck Lever, varekova, sysstat, Linux NFS Mailing List

[-- Attachment #1: Type: text/plain, Size: 2934 bytes --]

On Wed, 22 Jan 2014 08:12:35 -0500 Steve Dickson <SteveD@redhat.com> wrote:

> 
> 
> On 21/01/14 19:24, NeilBrown wrote:
> > On Tue, 21 Jan 2014 10:40:01 -0500 Chuck Lever <chuck.lever@oracle.com> wrote:
> > 
> >>
> >> On Jan 20, 2014, at 10:45 PM, NeilBrown <neilb@suse.de> wrote:
> >>
> >>>
> >>> (This time without the typo in Chuck's address - sorry).
> >>>
> >>> Not quite a year ago I wrote:
> >>>
> >>> On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:
> >>>
> >>>>
> >>>> Hi all,
> >>>> it seems that in 2010, a program called nfsiostat was added to the sysstat
> >>>> package (http://sebastien.godard.pagesperso-orange.fr/)
> >>>>
> >>>> This is unfortunate because in 2008 a program called nfsiostat was added to
> >>>> the nfs-utils package.
> >>>>    (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
> >>>>
> >>>> The default install locations seem to be different (/usr/bin vs /usr/sbin),
> >>>> so packagers would not immediately notice.
> >>>>
> >>>> This seems like something that should be "fixed".
> >>>>
> >>>> Any suggestions what a good "fix" should look like?
> >>>>
> >>>> NeilBrown
> >>>
> >>>
> >>> No suggestions were forthcoming.
> >>> So I have a suggestion.
> >>> I propose we rename the "nfsiostat" in the "sysstat" package to
> >>> "nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
> >>> "nfs-utils-iostat".
> >>>
> >>> Does anyone object?  If not I'll send off patches shortly :-)
> >>
> >> What does the sysstat version do?  do we need two programs, or can they be merged?  Or can one be dropped?
> >>
> > 
> > It reads /proc/self/mountstats and reports some numbers, not unlike the
> > nfs-utils version.
> > 
> > $ /usr/sbin/nfsiostat 
> > 
> > eli:/home mounted on /mnt:
> > 
> >    op/s		rpc bklog
> >    3.55	   0.00
> > read:            ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
> > 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
> > write:           ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
> > 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
> > $ /usr/bin/nfsiostat 
> > Linux 3.11.6-4-desktop (notabene.brown) 	01/22/2014 	_x86_64_	(8 CPU)
> > 
> > Filesystem:               rkB_nor/s    wkB_nor/s    rkB_dir/s    wkB_dir/s    rkB_svr/s    wkB_svr/s     ops/s    rops/s    wops/s
> > eli:/home                      0.00         0.00         0.00         0.00         0.00         0.00      0.00      0.00      0.00
> > 
> > 
> > The man page for the sysstat version is:
> > 
> > http://www.makelinux.com/man/1/N/nfsiostat
> Is this version still actively being maintain? 

Last change was 5 days ago.

https://github.com/sysstat/sysstat/commits/master/nfsiostat.c

NeilBrown


> 
> steved.
> 
> > 
> > while man page for nfs-utils version is:
> > 
> > http://linux.die.net/man/8/nfsiostat
> > 
> > NeilBrown
> > 


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-22 21:52         ` NeilBrown
@ 2014-01-23 17:42           ` Steve Dickson
  2014-01-23 20:12             ` Sebastien Godard
  0 siblings, 1 reply; 13+ messages in thread
From: Steve Dickson @ 2014-01-23 17:42 UTC (permalink / raw)
  To: NeilBrown; +Cc: Chuck Lever, varekova, sysstat, Linux NFS Mailing List



On 22/01/14 16:52, NeilBrown wrote:
> On Wed, 22 Jan 2014 08:12:35 -0500 Steve Dickson <SteveD@redhat.com> wrote:
> 
>>
>>
>> On 21/01/14 19:24, NeilBrown wrote:
>>> On Tue, 21 Jan 2014 10:40:01 -0500 Chuck Lever <chuck.lever@oracle.com> wrote:
>>>
>>>>
>>>> On Jan 20, 2014, at 10:45 PM, NeilBrown <neilb@suse.de> wrote:
>>>>
>>>>>
>>>>> (This time without the typo in Chuck's address - sorry).
>>>>>
>>>>> Not quite a year ago I wrote:
>>>>>
>>>>> On Wed, 10 Apr 2013 09:46:39 +1000 NeilBrown <neilb@suse.de> wrote:
>>>>>
>>>>>>
>>>>>> Hi all,
>>>>>> it seems that in 2010, a program called nfsiostat was added to the sysstat
>>>>>> package (http://sebastien.godard.pagesperso-orange.fr/)
>>>>>>
>>>>>> This is unfortunate because in 2008 a program called nfsiostat was added to
>>>>>> the nfs-utils package.
>>>>>>    (https://www.kernel.org/pub/linux/utils/nfs-utils/1.2.7/)
>>>>>>
>>>>>> The default install locations seem to be different (/usr/bin vs /usr/sbin),
>>>>>> so packagers would not immediately notice.
>>>>>>
>>>>>> This seems like something that should be "fixed".
>>>>>>
>>>>>> Any suggestions what a good "fix" should look like?
>>>>>>
>>>>>> NeilBrown
>>>>>
>>>>>
>>>>> No suggestions were forthcoming.
>>>>> So I have a suggestion.
>>>>> I propose we rename the "nfsiostat" in the "sysstat" package to
>>>>> "nfsio-sysstat" and the "nfsiostat" in "nfs-utils" to
>>>>> "nfs-utils-iostat".
>>>>>
>>>>> Does anyone object?  If not I'll send off patches shortly :-)
>>>>
>>>> What does the sysstat version do?  do we need two programs, or can they be merged?  Or can one be dropped?
>>>>
>>>
>>> It reads /proc/self/mountstats and reports some numbers, not unlike the
>>> nfs-utils version.
>>>
>>> $ /usr/sbin/nfsiostat 
>>>
>>> eli:/home mounted on /mnt:
>>>
>>>    op/s		rpc bklog
>>>    3.55	   0.00
>>> read:            ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
>>> 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
>>> write:           ops/s		   kB/s		  kB/op		retrans		avg RTT (ms)	avg exe (ms)
>>> 		  0.000	  0.000	  0.000       0 (0.0%)	  0.000	  0.000
>>> $ /usr/bin/nfsiostat 
>>> Linux 3.11.6-4-desktop (notabene.brown) 	01/22/2014 	_x86_64_	(8 CPU)
>>>
>>> Filesystem:               rkB_nor/s    wkB_nor/s    rkB_dir/s    wkB_dir/s    rkB_svr/s    wkB_svr/s     ops/s    rops/s    wops/s
>>> eli:/home                      0.00         0.00         0.00         0.00         0.00         0.00      0.00      0.00      0.00
>>>
>>>
>>> The man page for the sysstat version is:
>>>
>>> http://www.makelinux.com/man/1/N/nfsiostat
>> Is this version still actively being maintain? 
> 
> Last change was 5 days ago.
> 
> https://github.com/sysstat/sysstat/commits/master/nfsiostat.c
I guess the question now is do they want to continue maintaining
it or just merge the functionality into the nfs-utils version...

steved.
> 
> NeilBrown
> 
> 
>>
>> steved.
>>
>>>
>>> while man page for nfs-utils version is:
>>>
>>> http://linux.die.net/man/8/nfsiostat
>>>
>>> NeilBrown
>>>
> 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-23 17:42           ` Steve Dickson
@ 2014-01-23 20:12             ` Sebastien Godard
  2014-01-25 14:39               ` Steve Dickson
  0 siblings, 1 reply; 13+ messages in thread
From: Sebastien Godard @ 2014-01-23 20:12 UTC (permalink / raw)
  To: Steve Dickson, NeilBrown; +Cc: Chuck Lever, varekova, Linux NFS Mailing List

On 01/23/2014 06:42 PM, Steve Dickson wrote:
>
> On 22/01/14 16:52, NeilBrown wrote:
>> On Wed, 22 Jan 2014 08:12:35 -0500 Steve Dickson <SteveD@redhat.com> wrote:
>>
>>>
>>> On 21/01/14 19:24, NeilBrown wrote:
>>> Is this version still actively being maintain? 
>> Last change was 5 days ago.
>>
>> https://github.com/sysstat/sysstat/commits/master/nfsiostat.c
> I guess the question now is do they want to continue maintaining
> it or just merge the functionality into the nfs-utils version...
>

This is no longer a question as I already answered it. Please *read* my 
previous mail.

Regards,

-- 
Sebastien GODARD <sysstat [at] orange.fr>
http://sebastien.godard.pagesperso-orange.fr/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-23 20:12             ` Sebastien Godard
@ 2014-01-25 14:39               ` Steve Dickson
  2014-01-25 15:45                 ` Sebastien Godard
  0 siblings, 1 reply; 13+ messages in thread
From: Steve Dickson @ 2014-01-25 14:39 UTC (permalink / raw)
  To: Sebastien Godard, NeilBrown; +Cc: Chuck Lever, varekova, Linux NFS Mailing List



On 23/01/14 15:12, Sebastien Godard wrote:
> On 01/23/2014 06:42 PM, Steve Dickson wrote:
>>
>> On 22/01/14 16:52, NeilBrown wrote:
>>> On Wed, 22 Jan 2014 08:12:35 -0500 Steve Dickson <SteveD@redhat.com> wrote:
>>>
>>>>
>>>> On 21/01/14 19:24, NeilBrown wrote:
>>>> Is this version still actively being maintain? 
>>> Last change was 5 days ago.
>>>
>>> https://github.com/sysstat/sysstat/commits/master/nfsiostat.c
>> I guess the question now is do they want to continue maintaining
>> it or just merge the functionality into the nfs-utils version...
>>
> 
> This is no longer a question as I already answered it. Please *read* my previous mail.
I did... but thought Neil's mail was pointing out that some type of 
maintenance was happening... But if you say it's "no longer actively being maintained"
I'll believe you... :-) 

So what is the next step to deprecate the nfsiostat in sysstat? 

steved.

> 
> Regards,
> 

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-25 14:39               ` Steve Dickson
@ 2014-01-25 15:45                 ` Sebastien Godard
  2014-01-25 20:45                   ` NeilBrown
  0 siblings, 1 reply; 13+ messages in thread
From: Sebastien Godard @ 2014-01-25 15:45 UTC (permalink / raw)
  To: Steve Dickson, NeilBrown; +Cc: Chuck Lever, varekova, Linux NFS Mailing List

On 01/25/2014 03:39 PM, Steve Dickson wrote:
>
> On 23/01/14 15:12, Sebastien Godard wrote:
>> On 01/23/2014 06:42 PM, Steve Dickson wrote:
>>> On 22/01/14 16:52, NeilBrown wrote:
>>>> On Wed, 22 Jan 2014 08:12:35 -0500 Steve Dickson <SteveD@redhat.com> wrote:
>>>>
>>>>> On 21/01/14 19:24, NeilBrown wrote:
>>>>> Is this version still actively being maintain?
>>>> Last change was 5 days ago.
>>>>
>>>> https://github.com/sysstat/sysstat/commits/master/nfsiostat.c
>>> I guess the question now is do they want to continue maintaining
>>> it or just merge the functionality into the nfs-utils version...
>>>
>> This is no longer a question as I already answered it. Please *read* my previous mail.
> I did... but thought Neil's mail was pointing out that some type of
> maintenance was happening... But if you say it's "no longer actively being maintained"
> I'll believe you... :-)
>
> So what is the next step to deprecate the nfsiostat in sysstat?
>

In fact, the various patches applied to nfsiostat over the last few 
years were mainly cosmetic ones (changes in sysstat APIs, etc.)
So I propose now to simply remove nfsiostat code from sysstat: Next 
sysstat version will rename nfsiostat (eg. to nfsiostat-sysstat) and the 
man page will indicate it is now obsolete and point to the version from 
nfs-utils. Then nfsiostat will be definitively removed sometime before 
the end of this year.

Regards,

-- 
Sebastien GODARD <sysstat [at] orange.fr>
http://sebastien.godard.pagesperso-orange.fr/


^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: Too many programs called nfsiostat.
  2014-01-25 15:45                 ` Sebastien Godard
@ 2014-01-25 20:45                   ` NeilBrown
  0 siblings, 0 replies; 13+ messages in thread
From: NeilBrown @ 2014-01-25 20:45 UTC (permalink / raw)
  To: Sebastien Godard
  Cc: Steve Dickson, Chuck Lever, varekova, Linux NFS Mailing List

[-- Attachment #1: Type: text/plain, Size: 1685 bytes --]

On Sat, 25 Jan 2014 16:45:48 +0100 Sebastien Godard <sysstat@orange.fr> wrote:

> On 01/25/2014 03:39 PM, Steve Dickson wrote:
> >
> > On 23/01/14 15:12, Sebastien Godard wrote:
> >> On 01/23/2014 06:42 PM, Steve Dickson wrote:
> >>> On 22/01/14 16:52, NeilBrown wrote:
> >>>> On Wed, 22 Jan 2014 08:12:35 -0500 Steve Dickson <SteveD@redhat.com> wrote:
> >>>>
> >>>>> On 21/01/14 19:24, NeilBrown wrote:
> >>>>> Is this version still actively being maintain?
> >>>> Last change was 5 days ago.
> >>>>
> >>>> https://github.com/sysstat/sysstat/commits/master/nfsiostat.c
> >>> I guess the question now is do they want to continue maintaining
> >>> it or just merge the functionality into the nfs-utils version...
> >>>
> >> This is no longer a question as I already answered it. Please *read* my previous mail.
> > I did... but thought Neil's mail was pointing out that some type of
> > maintenance was happening... But if you say it's "no longer actively being maintained"
> > I'll believe you... :-)
> >
> > So what is the next step to deprecate the nfsiostat in sysstat?
> >
> 
> In fact, the various patches applied to nfsiostat over the last few 
> years were mainly cosmetic ones (changes in sysstat APIs, etc.)
> So I propose now to simply remove nfsiostat code from sysstat: Next 
> sysstat version will rename nfsiostat (eg. to nfsiostat-sysstat) and the 
> man page will indicate it is now obsolete and point to the version from 
> nfs-utils. Then nfsiostat will be definitively removed sometime before 
> the end of this year.
> 
> Regards,
> 

Fantastic - a nice clear resolution, just what I was hoping for.
Thanks a lot!

NeilBrown

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2014-01-25 20:46 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-09 23:46 Too man programs calls nfsiostat NeilBrown
2014-01-21  3:00 ` Too many " NeilBrown
2014-01-21  3:45 ` NeilBrown
2014-01-21 15:40   ` Chuck Lever
2014-01-22  0:24     ` Too many programs called nfsiostat NeilBrown
2014-01-22 13:12       ` Steve Dickson
2014-01-22 21:41         ` Sebastien Godard
2014-01-22 21:52         ` NeilBrown
2014-01-23 17:42           ` Steve Dickson
2014-01-23 20:12             ` Sebastien Godard
2014-01-25 14:39               ` Steve Dickson
2014-01-25 15:45                 ` Sebastien Godard
2014-01-25 20:45                   ` NeilBrown

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox