public inbox for linux-nfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Steve Dickson <SteveD@redhat.com>
To: NeilBrown <neilb@suse.de>
Cc: Chuck Lever <chuck.lever@oracle.com>,
	varekova@redhat.com, sysstat@orange.fr,
	Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: Too many programs called nfsiostat.
Date: Thu, 23 Jan 2014 12:42:45 -0500	[thread overview]
Message-ID: <52E15495.3050103@RedHat.com> (raw)
In-Reply-To: <20140123085226.72ebaf85@notabene.brown>



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
>>>
> 

  reply	other threads:[~2014-01-23 17:42 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
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

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52E15495.3050103@RedHat.com \
    --to=steved@redhat.com \
    --cc=chuck.lever@oracle.com \
    --cc=linux-nfs@vger.kernel.org \
    --cc=neilb@suse.de \
    --cc=sysstat@orange.fr \
    --cc=varekova@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox