linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
  • [parent not found: <1400749779-24879-3-git-send-email-mgorman@suse.de>]
  • [parent not found: <1400749779-24879-4-git-send-email-mgorman@suse.de>]
  • * Re: [PATCH 0/3] Shrinkers and proportional reclaim
           [not found] <1400749779-24879-1-git-send-email-mgorman@suse.de>
                       ` (2 preceding siblings ...)
           [not found] ` <1400749779-24879-4-git-send-email-mgorman@suse.de>
    @ 2014-05-22 16:14 ` Yuanhan Liu
      2014-05-22 16:30   ` Mel Gorman
      2014-05-22 17:22 ` Tim Chen
      2014-05-26 21:44 ` Hugh Dickins
      5 siblings, 1 reply; 13+ messages in thread
    From: Yuanhan Liu @ 2014-05-22 16:14 UTC (permalink / raw)
      To: Mel Gorman
      Cc: Andrew Morton, Johannes Weiner, Hugh Dickins, Tim Chen,
    	Dave Chinner, Yuanhan Liu, Bob Liu, Jan Kara, Rik van Riel,
    	Mel Gorman, Linux Kernel, Linux-MM, Linux-FSDevel
    
    On Thu, May 22, 2014 at 10:09:36AM +0100, Mel Gorman wrote:
    > This series is aimed at regressions noticed during reclaim activity. The
    > first two patches are shrinker patches that were posted ages ago but never
    > merged for reasons that are unclear to me. I'm posting them again to see if
    > there was a reason they were dropped or if they just got lost. Dave?  Time?
    > The last patch adjusts proportional reclaim. Yuanhan Liu, can you retest
    > the vm scalability test cases on a larger machine? Hugh, does this work
    > for you on the memcg test cases?
    
    Sure, and here is the result. I applied these 3 patches on v3.15-rc6,
    and head commit is 60c10afd. e82e0561 is the old commit that introduced
    the regression.  The testserver has 512G memory and 120 CPU.
    
    It's a simple result; if you need more data, I can gather them and send
    it to you tomorrow:
    
    e82e0561        v3.15-rc6       60c10afd
    ----------------------------------------
    18560785        12232122        38868453
                    -34%            +109
    
    As you can see, the performance is back, and it is way much better ;)
    
            --yliu
    > 
    > Based on ext4, I get the following results but unfortunately my larger test
    > machines are all unavailable so this is based on a relatively small machine.
    > 
    > postmark
    >                                   3.15.0-rc5            3.15.0-rc5
    >                                      vanilla       proportion-v1r4
    > Ops/sec Transactions         21.00 (  0.00%)       25.00 ( 19.05%)
    > Ops/sec FilesCreate          39.00 (  0.00%)       45.00 ( 15.38%)
    > Ops/sec CreateTransact       10.00 (  0.00%)       12.00 ( 20.00%)
    > Ops/sec FilesDeleted       6202.00 (  0.00%)     6202.00 (  0.00%)
    > Ops/sec DeleteTransact       11.00 (  0.00%)       12.00 (  9.09%)
    > Ops/sec DataRead/MB          25.97 (  0.00%)       30.02 ( 15.59%)
    > Ops/sec DataWrite/MB         49.99 (  0.00%)       57.78 ( 15.58%)
    > 
    > ffsb (mail server simulator)
    >                                  3.15.0-rc5             3.15.0-rc5
    >                                     vanilla        proportion-v1r4
    > Ops/sec readall           9402.63 (  0.00%)      9805.74 (  4.29%)
    > Ops/sec create            4695.45 (  0.00%)      4781.39 (  1.83%)
    > Ops/sec delete             173.72 (  0.00%)       177.23 (  2.02%)
    > Ops/sec Transactions     14271.80 (  0.00%)     14764.37 (  3.45%)
    > Ops/sec Read                37.00 (  0.00%)        38.50 (  4.05%)
    > Ops/sec Write               18.20 (  0.00%)        18.50 (  1.65%)
    > 
    > dd of a large file
    >                                 3.15.0-rc5            3.15.0-rc5
    >                                    vanilla       proportion-v1r4
    > WallTime DownloadTar       75.00 (  0.00%)       61.00 ( 18.67%)
    > WallTime DD               423.00 (  0.00%)      401.00 (  5.20%)
    > WallTime Delete             2.00 (  0.00%)        5.00 (-150.00%)
    > 
    > stutter (times mmap latency during large amounts of IO)
    > 
    >                             3.15.0-rc5            3.15.0-rc5
    >                                vanilla       proportion-v1r4
    > Unit >5ms Delays  80252.0000 (  0.00%)  81523.0000 ( -1.58%)
    > Unit Mmap min         8.2118 (  0.00%)      8.3206 ( -1.33%)
    > Unit Mmap mean       17.4614 (  0.00%)     17.2868 (  1.00%)
    > Unit Mmap stddev     24.9059 (  0.00%)     34.6771 (-39.23%)
    > Unit Mmap max      2811.6433 (  0.00%)   2645.1398 (  5.92%)
    > Unit Mmap 90%        20.5098 (  0.00%)     18.3105 ( 10.72%)
    > Unit Mmap 93%        22.9180 (  0.00%)     20.1751 ( 11.97%)
    > Unit Mmap 95%        25.2114 (  0.00%)     22.4988 ( 10.76%)
    > Unit Mmap 99%        46.1430 (  0.00%)     43.5952 (  5.52%)
    > Unit Ideal  Tput     85.2623 (  0.00%)     78.8906 (  7.47%)
    > Unit Tput min        44.0666 (  0.00%)     43.9609 (  0.24%)
    > Unit Tput mean       45.5646 (  0.00%)     45.2009 (  0.80%)
    > Unit Tput stddev      0.9318 (  0.00%)      1.1084 (-18.95%)
    > Unit Tput max        46.7375 (  0.00%)     46.7539 ( -0.04%)
    > 
    >  fs/super.c  | 16 +++++++++-------
    >  mm/vmscan.c | 36 +++++++++++++++++++++++++-----------
    >  2 files changed, 34 insertions(+), 18 deletions(-)
    > 
    > -- 
    > 1.8.4.5
    
    ^ permalink raw reply	[flat|nested] 13+ messages in thread
  • * Re: [PATCH 0/3] Shrinkers and proportional reclaim
           [not found] <1400749779-24879-1-git-send-email-mgorman@suse.de>
                       ` (3 preceding siblings ...)
      2014-05-22 16:14 ` [PATCH 0/3] Shrinkers and proportional reclaim Yuanhan Liu
    @ 2014-05-22 17:22 ` Tim Chen
      2014-05-26 21:44 ` Hugh Dickins
      5 siblings, 0 replies; 13+ messages in thread
    From: Tim Chen @ 2014-05-22 17:22 UTC (permalink / raw)
      To: Mel Gorman
      Cc: Andrew Morton, Johannes Weiner, Hugh Dickins, Dave Chinner,
    	Yuanhan Liu, Bob Liu, Jan Kara, Rik van Riel, Linux Kernel,
    	Linux-MM, Linux-FSDevel
    
    On Thu, 2014-05-22 at 10:09 +0100, Mel Gorman wrote:
    > This series is aimed at regressions noticed during reclaim activity. The
    > first two patches are shrinker patches that were posted ages ago but never
    > merged for reasons that are unclear to me. I'm posting them again to see if
    > there was a reason they were dropped or if they just got lost. Dave?  Time?
    
    As far as I remembered, I think Dave was planning to merge this as part
    of his VFS scalability patch series.  Otherwise there wasn't any other
    issues.
    
    Thanks to Mel for looking at these patches and Yunhan for testing them.
    
    Tim
    
    
    ^ permalink raw reply	[flat|nested] 13+ messages in thread
  • * Re: [PATCH 0/3] Shrinkers and proportional reclaim
           [not found] <1400749779-24879-1-git-send-email-mgorman@suse.de>
                       ` (4 preceding siblings ...)
      2014-05-22 17:22 ` Tim Chen
    @ 2014-05-26 21:44 ` Hugh Dickins
      2014-05-27  2:37   ` Dave Chinner
      5 siblings, 1 reply; 13+ messages in thread
    From: Hugh Dickins @ 2014-05-26 21:44 UTC (permalink / raw)
      To: Mel Gorman
      Cc: Andrew Morton, Johannes Weiner, Tim Chen, Dave Chinner,
    	Yuanhan Liu, Bob Liu, Jan Kara, Rik van Riel, Linux Kernel,
    	Linux-MM, Linux-FSDevel
    
    On Thu, 22 May 2014, Mel Gorman wrote:
    
    > This series is aimed at regressions noticed during reclaim activity. The
    > first two patches are shrinker patches that were posted ages ago but never
    > merged for reasons that are unclear to me. I'm posting them again to see if
    > there was a reason they were dropped or if they just got lost. Dave?  Time?
    > The last patch adjusts proportional reclaim. Yuanhan Liu, can you retest
    > the vm scalability test cases on a larger machine? Hugh, does this work
    > for you on the memcg test cases?
    
    Yes it does, thank you.
    
    Though the situation is muddy, since on our current internal tree, I'm
    surprised to find that the memcg test case no longer fails reliably
    without our workaround and without your fix.
    
    "Something must have changed"; but it would take a long time to work
    out what.  If I travel back in time with git, to where we first applied
    the "vindictive" patch, then yes that test case convincingly fails
    without either (my or your) patch, and passes with either patch.
    
    And you have something that satisfies Yuanhan too, that's great.
    
    I'm also pleased to see Dave and Tim reduce the contention in
    grab_super_passive(): that's a familiar symbol from livelock dumps.
    
    You might want to add this little 4/3, that we've had in for a
    while; but with grab_super_passive() out of super_cache_count(),
    it will have much less importance.
    
    
    [PATCH 4/3] fs/superblock: Avoid counting without __GFP_FS
    
    Don't waste time counting objects in super_cache_count() if no __GFP_FS:
    super_cache_scan() would only back out with SHRINK_STOP in that case.
    
    Signed-off-by: Hugh Dickins <hughd@google.com>
    ---
    
     fs/super.c |    6 ++++++
     1 file changed, 6 insertions(+)
    
    --- melgo/fs/super.c	2014-05-26 13:39:33.000131904 -0700
    +++ linux/fs/super.c	2014-05-26 13:56:19.012155813 -0700
    @@ -110,6 +110,12 @@ static unsigned long super_cache_count(s
     	struct super_block *sb;
     	long	total_objects = 0;
     
    +	/*
    +	 * None can be freed without __GFP_FS, so don't waste time counting.
    +	 */
    +	if (!(sc->gfp_mask & __GFP_FS))
    +		return 0;
    +
     	sb = container_of(shrink, struct super_block, s_shrink);
     
     	/*
    
    ^ permalink raw reply	[flat|nested] 13+ messages in thread

  • end of thread, other threads:[~2014-05-28  1:38 UTC | newest]
    
    Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
    -- links below jump to the message on this page --
         [not found] <1400749779-24879-1-git-send-email-mgorman@suse.de>
         [not found] ` <1400749779-24879-2-git-send-email-mgorman@suse.de>
    2014-05-22 15:52   ` [PATCH 1/3] fs/superblock: Unregister sb shrinker before ->kill_sb() Rik van Riel
         [not found] ` <1400749779-24879-3-git-send-email-mgorman@suse.de>
    2014-05-22 15:59   ` [PATCH 2/3] fs/superblock: Avoid locking counting inodes and dentries before reclaiming them Rik van Riel
         [not found] ` <1400749779-24879-4-git-send-email-mgorman@suse.de>
    2014-05-22 16:04   ` [PATCH 3/3] mm: vmscan: Use proportional scanning during direct reclaim and full scan at DEF_PRIORITY Rik van Riel
    2014-05-22 16:14 ` [PATCH 0/3] Shrinkers and proportional reclaim Yuanhan Liu
    2014-05-22 16:30   ` Mel Gorman
    2014-05-23  2:43     ` Yuanhan Liu
    2014-05-22 17:22 ` Tim Chen
    2014-05-26 21:44 ` Hugh Dickins
    2014-05-27  2:37   ` Dave Chinner
    2014-05-27 21:17     ` Hugh Dickins
    2014-05-27 23:02       ` Konstantin Khlebnikov
    2014-05-27 23:19         ` Hugh Dickins
    2014-05-28  1:37           ` Dave Chinner
    

    This is a public inbox, see mirroring instructions
    for how to clone and mirror all data and code used for this inbox;
    as well as URLs for NNTP newsgroup(s).