linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Wu Fengguang <fengguang.wu@intel.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>,
	Johannes Weiner <hannes@cmpxchg.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Rik van Riel <riel@redhat.com>,
	Hugh Dickins <hugh.dickins@tiscali.co.uk>,
	Andi Kleen <andi@firstfloor.org>,
	Minchan Kim <minchan.kim@gmail.com>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Barnes, Jesse" <jesse.barnes@intel.com>
Subject: Re: [patch v3] swap: virtual swap readahead
Date: Wed, 10 Jun 2009 17:59:50 +0800	[thread overview]
Message-ID: <20090610095950.GA514@localhost> (raw)
In-Reply-To: <1244626976.13761.11593.camel@twins>

On Wed, Jun 10, 2009 at 05:42:56PM +0800, Peter Zijlstra wrote:
> On Wed, 2009-06-10 at 16:56 +0800, Wu Fengguang wrote:
> > 
> > Yes it worked!  But then I run into page allocation failures:
> > 
> > [  340.639803] Xorg: page allocation failure. order:4, mode:0x40d0
> > [  340.645744] Pid: 3258, comm: Xorg Not tainted 2.6.30-rc8-mm1 #303
> > [  340.651839] Call Trace:
> > [  340.654289]  [<ffffffff810c8204>] __alloc_pages_nodemask+0x344/0x6c0
> > [  340.660645]  [<ffffffff810f7489>] __slab_alloc_page+0xb9/0x3b0
> > [  340.666472]  [<ffffffff810f8608>] __kmalloc+0x198/0x250
> > [  340.671786]  [<ffffffffa014bf9f>] ? i915_gem_execbuffer+0x17f/0x11e0 [i915]
> > [  340.678746]  [<ffffffffa014bf9f>] i915_gem_execbuffer+0x17f/0x11e0 [i915]
> 
> Jesse Barnes had a patch to add a vmalloc fallback to those largish kms
> allocs.
> 
> But order-4 allocs failing isn't really strange, but it might indicate
> this patch fragments stuff sooner, although I've seen these particular
> failues before.

Thanks for the tip. Where is it? I'd like to try it out :)

Despite of the xorg failures, the test was able to complete with the
listed timing. The numbers are the time each program is able to start:

  before       after
    0.02        0.01    N xeyes
    0.76        0.68    N firefox
    1.88        1.89    N nautilus
    3.17        3.25    N nautilus --browser
    4.89        4.98    N gthumb
    6.47        6.79    N gedit
    8.16        8.56    N xpdf /usr/share/doc/shared-mime-info/shared-mime-info-spec.pdf
   12.55       12.61    N xterm
   14.57       14.99    N mlterm
   17.06       17.16    N gnome-terminal
   18.90       19.60    N urxvt
   23.48       24.26    N gnome-system-monitor
   26.52       27.13    N gnome-help
   29.65       30.29    N gnome-dictionary
   36.12       36.93    N /usr/games/sol
   39.27       39.21    N /usr/games/gnometris
   42.56       43.61    N /usr/games/gnect
   47.03       47.40    N /usr/games/gtali
   52.05       51.41    N /usr/games/iagno
   55.42       56.21    N /usr/games/gnotravex
   61.47       60.58    N /usr/games/mahjongg
   67.11       64.68    N /usr/games/gnome-sudoku
   75.15       72.42    N /usr/games/glines
   79.70       78.61    N /usr/games/glchess
   88.48       87.01    N /usr/games/gnomine
   96.51       95.03    N /usr/games/gnotski
  102.19      100.50    N /usr/games/gnibbles
  114.93      108.97    N /usr/games/gnobots2
  125.02      120.09    N /usr/games/blackjack
  135.11      134.39    N /usr/games/same-gnome
  154.50      159.99    N /usr/bin/gnome-window-properties
  162.09      176.04    N /usr/bin/gnome-default-applications-properties
  173.29      197.12    N /usr/bin/gnome-at-properties
  188.21      221.15    N /usr/bin/gnome-typing-monitor
  199.93      249.38    N /usr/bin/gnome-at-visual
  206.95      272.87    N /usr/bin/gnome-sound-properties
  224.49      302.03    N /usr/bin/gnome-at-mobility
  234.11      325.73    N /usr/bin/gnome-keybinding-properties
  248.59      358.64    N /usr/bin/gnome-about-me
  276.27      402.30    N /usr/bin/gnome-display-properties
  304.39      439.35    N /usr/bin/gnome-network-preferences
  342.01      482.78    N /usr/bin/gnome-mouse-properties
  388.58      528.54    N /usr/bin/gnome-appearance-properties
  508.47      653.12    N /usr/bin/gnome-control-center
  587.57      769.65    N /usr/bin/gnome-keyboard-properties
  758.16     1021.65    N : oocalc
  830.03     1124.14    N : oodraw
  900.03     1246.52    N : ooimpress
  993.91     1370.35    N : oomath
 1081.89     1478.34    N : ooweb
 1161.99     1595.85    N : oowriter

It's slower with the patch. Maybe we shall give it another run with
the vmalloc patch.

Thanks,
Fengguang

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2009-06-10  9:58 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-09 19:01 [patch v3] swap: virtual swap readahead Johannes Weiner
2009-06-09 19:37 ` Johannes Weiner
2009-06-10  5:03   ` Wu Fengguang
2009-06-10  7:45     ` Johannes Weiner
2009-06-10  8:11       ` Wu Fengguang
2009-06-10  8:32         ` KAMEZAWA Hiroyuki
2009-06-10  8:56           ` Wu Fengguang
2009-06-10  9:42             ` Peter Zijlstra
2009-06-10  9:59               ` Wu Fengguang [this message]
2009-06-10 10:05                 ` Peter Zijlstra
2009-06-10 11:32                   ` Wu Fengguang
2009-06-10 17:25                     ` Jesse Barnes
2009-06-11  5:22                       ` Wu Fengguang
2009-06-11 10:17                         ` Johannes Weiner
2009-06-12  1:59                           ` Wu Fengguang
2009-06-15 18:22                             ` Johannes Weiner
2009-06-18  9:19                               ` Wu Fengguang
2009-06-18 13:01                                 ` Johannes Weiner
2009-06-19  3:30                                   ` Wu Fengguang
2009-06-21 18:07                                   ` Hugh Dickins
2009-06-21 18:37                                     ` Johannes Weiner
2009-06-10  9:30           ` Johannes Weiner
2009-06-10  6:39   ` KAMEZAWA Hiroyuki
2009-06-11  5:31 ` KAMEZAWA Hiroyuki
2009-06-17 22:41   ` Johannes Weiner
2009-06-18  9:29     ` Wu Fengguang
2009-06-18 13:09       ` Johannes Weiner
2009-06-19  3:17         ` Wu Fengguang

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=20090610095950.GA514@localhost \
    --to=fengguang.wu@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=andi@firstfloor.org \
    --cc=hannes@cmpxchg.org \
    --cc=hugh.dickins@tiscali.co.uk \
    --cc=jesse.barnes@intel.com \
    --cc=kamezawa.hiroyu@jp.fujitsu.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=minchan.kim@gmail.com \
    --cc=peterz@infradead.org \
    --cc=riel@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;
as well as URLs for NNTP newsgroup(s).