All of lore.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.