All of lore.kernel.org
 help / color / mirror / Atom feed
From: Zheng Liu <gnehzuil.liu@gmail.com>
To: Jan Kara <jack@suse.cz>
Cc: linux-ext4@vger.kernel.org, Theodore Ts'o <tytso@mit.edu>,
	Andreas Dilger <adilger.kernel@dilger.ca>,
	Zheng Liu <wenqing.lz@taobao.com>
Subject: Re: [PATCH v3 2/6] ext4: track extent status tree shrinker delay statictics
Date: Thu, 4 Sep 2014 20:10:48 +0800	[thread overview]
Message-ID: <20140904121048.GA13193@gmail.com> (raw)
In-Reply-To: <20140827132607.GA22211@quack.suse.cz>

On Wed, Aug 27, 2014 at 03:26:07PM +0200, Jan Kara wrote:
> On Thu 07-08-14 11:35:49, Zheng Liu wrote:
> > From: Zheng Liu <wenqing.lz@taobao.com>
> > 
> > This commit adds some statictics in extent status tree shrinker.  The
> > purpose to add these is that we want to collect more details when we
> > encounter a stall caused by extent status tree shrinker.  Here we count
> > the following statictics:
> >   stats:
> >     the number of all objects on all extent status trees
> >     the number of reclaimable objects on lru list
> >     cache hits/misses
> >     the last sorted interval
> >     the number of inodes on lru list
> >   average:
> >     scan time for shrinking some objects
> >     the number of shrunk objects
> >   maximum:
> >     the inode that has max nr. of objects on lru list
> >     the maximum scan time for shrinking some objects
> > 
> > The output looks like below:
> >   $ cat /proc/fs/ext4/sda1/es_shrinker_info
> >   stats:
> >     28228 objects
> >     6341 reclaimable objects
> >     5281/631 cache hits/misses
> >     586 ms last sorted interval
> >     250 inodes on lru list
> >   average:
> >     153 us scan time
> >     128 shrunk objects
> >   maximum:
> >     255 inode (255 objects, 198 reclaimable)
> >     125723 us max scan time
>   When reading through this again, I realized that we probably don't want
> this file in /proc/fs/ext4 but rather in /sys/kernel/debug/ext4 because
> it's really a debugging interface and we don't want any tools to start
> using it.

Fair enough.  I will fix it in next version.

Thanks,
                                                - Zheng

  reply	other threads:[~2014-09-04 12:01 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-07  3:35 [PATCH v3 0/6] ext4: extents status tree shrinker improvement Zheng Liu
2014-08-07  3:35 ` [PATCH v3 1/6] ext4: improve extents status tree trace point Zheng Liu
2014-09-02  2:25   ` Theodore Ts'o
2014-08-07  3:35 ` [PATCH v3 2/6] ext4: track extent status tree shrinker delay statictics Zheng Liu
2014-08-27 13:26   ` Jan Kara
2014-09-04 12:10     ` Zheng Liu [this message]
2014-09-04 15:49       ` Theodore Ts'o
2014-08-07  3:35 ` [PATCH v3 3/6] ext4: cache extent hole in extent status tree for ext4_da_map_blocks() Zheng Liu
2014-08-27 13:55   ` Jan Kara
2014-09-04 13:05     ` Zheng Liu
2014-09-02  2:43   ` Theodore Ts'o
2014-09-04 13:04     ` Zheng Liu
2014-09-04 15:54       ` Theodore Ts'o
2014-08-07  3:35 ` [PATCH v3 4/6] ext4: change lru to round-robin in extent status tree shrinker Zheng Liu
2014-08-27 15:01   ` Jan Kara
2014-09-03  3:37     ` Theodore Ts'o
2014-09-03 15:31       ` Jan Kara
2014-09-03 20:00         ` Theodore Ts'o
2014-09-03 22:14           ` Jan Kara
2014-09-03 22:38             ` Theodore Ts'o
     [not found]               ` <20140904071553.GA26930@quack.suse.cz>
2014-09-04 15:44                 ` Theodore Ts'o
2014-09-08 15:47                   ` Jan Kara
2014-08-07  3:35 ` [PATCH v3 5/6] ext4: use a list to track all reclaimable objects for extent status tree Zheng Liu
2014-08-27 15:13   ` Jan Kara
2014-09-03  3:44     ` Theodore Ts'o
2014-08-07  3:35 ` [PATCH v3 6/6] ext4: use a garbage collection algorithm to manage object Zheng Liu
2014-08-27 15:24   ` Jan Kara
2014-10-20 14:48 ` [PATCH v3 0/6] ext4: extents status tree shrinker improvement Theodore Ts'o
2014-10-21 10:22   ` Jan Kara
2014-10-21 15:58     ` 刘峥(文卿)
2014-11-03 16:10       ` Jan Kara
2014-11-07  2:38         ` Zheng Liu
2014-11-13 23:40         ` Theodore Ts'o

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=20140904121048.GA13193@gmail.com \
    --to=gnehzuil.liu@gmail.com \
    --cc=adilger.kernel@dilger.ca \
    --cc=jack@suse.cz \
    --cc=linux-ext4@vger.kernel.org \
    --cc=tytso@mit.edu \
    --cc=wenqing.lz@taobao.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.