All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wu Fengguang <fengguang.wu@intel.com>
To: Christian Ehrhardt <ehrhardt@linux.vnet.ibm.com>
Cc: Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.cz>,
	Christoph Hellwig <hch@infradead.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Memory Management List <linux-mm@kvack.org>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	Ankit Jain <radical@gmail.com>, Rik van Riel <riel@redhat.com>,
	Nikanth Karthikesan <knikanth@suse.de>,
	LKML <linux-kernel@vger.kernel.org>,
	Andi Kleen <andi@firstfloor.org>
Subject: Re: [PATCH 2/8] readahead: make default readahead size a kernel parameter
Date: Wed, 30 Nov 2011 21:29:28 +0800	[thread overview]
Message-ID: <20111130132928.GA31589@localhost> (raw)
In-Reply-To: <4ED629CB.401@linux.vnet.ibm.com>

On Wed, Nov 30, 2011 at 09:04:11PM +0800, Christian Ehrhardt wrote:
> 
> 
> On 11/28/2011 03:39 AM, Wu Fengguang wrote:
> > On Fri, Nov 25, 2011 at 08:36:33AM +0800, Dave Chinner wrote:
> >> On Thu, Nov 24, 2011 at 11:28:22PM +0100, Jan Kara wrote:
> >>> On Mon 21-11-11 19:35:40, Wu Fengguang wrote:
> >>>> On Mon, Nov 21, 2011 at 06:01:37PM +0800, Christoph Hellwig wrote:
> >>>>> On Mon, Nov 21, 2011 at 05:18:21PM +0800, Wu Fengguang wrote:
> >>>>>> From: Nikanth Karthikesan<knikanth@suse.de>
> >>>>>>
> [...]
> 
> >>
> >> And one that has already been in use for exactly this purpose for
> >> years. Indeed, it's far more flexible because you can give different
> >> types of devices different default readahead settings quite easily,
> >> and it you can set different defaults for just about any tunable
> >> parameter (e.g. readahead, ctq depth, max IO sizes, etc) in the same
> >> way.
> >
> > I'm interested in this usage, too. Would you share some of your rules?
> >
> 
> FYI - This is an example of a rules Suse delivers in SLES @ s390 for a 
> while now. With little modifications it could be used for all Dave 
> mentioned above.

It's a really good example, thank you!

> cat /etc/udev/rules.d/60-readahead.rules
> # 
>  
>  
> 
> # Rules to set an increased default max readahead size for s390 disk 
> devices 
>  
> 
> # This file should be installed in /etc/udev/rules.d 
>  
>  
> 
> # 
>  
> SUBSYSTEM!="block", GOTO="ra_end" 
> 
> ACTION!="add", GOTO="ra_end" 
> 
> # on device add set initial readahead to 512 (instead of in kernel 128) 
> 
> KERNEL=="sd*[!0-9]", ATTR{queue/read_ahead_kb}="512" 
> 
> KERNEL=="dasd*[!0-9]", ATTR{queue/read_ahead_kb}="512" 

So SLES (@s390 and maybe more) is already shipping with 512kb
readahead size? Good to know this!

Thanks,
Fengguang

>  
> 
> 
> LABEL="ra_end"
> 
> -- 
> 
> Grüsse / regards, Christian Ehrhardt
> IBM Linux Technology Center, System z Linux Performance

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Wu Fengguang <fengguang.wu@intel.com>
To: Christian Ehrhardt <ehrhardt@linux.vnet.ibm.com>
Cc: Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.cz>,
	Christoph Hellwig <hch@infradead.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Memory Management List <linux-mm@kvack.org>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	Ankit Jain <radical@gmail.com>, Rik van Riel <riel@redhat.com>,
	Nikanth Karthikesan <knikanth@suse.de>,
	LKML <linux-kernel@vger.kernel.org>,
	Andi Kleen <andi@firstfloor.org>
Subject: Re: [PATCH 2/8] readahead: make default readahead size a kernel parameter
Date: Wed, 30 Nov 2011 21:29:28 +0800	[thread overview]
Message-ID: <20111130132928.GA31589@localhost> (raw)
In-Reply-To: <4ED629CB.401@linux.vnet.ibm.com>

On Wed, Nov 30, 2011 at 09:04:11PM +0800, Christian Ehrhardt wrote:
> 
> 
> On 11/28/2011 03:39 AM, Wu Fengguang wrote:
> > On Fri, Nov 25, 2011 at 08:36:33AM +0800, Dave Chinner wrote:
> >> On Thu, Nov 24, 2011 at 11:28:22PM +0100, Jan Kara wrote:
> >>> On Mon 21-11-11 19:35:40, Wu Fengguang wrote:
> >>>> On Mon, Nov 21, 2011 at 06:01:37PM +0800, Christoph Hellwig wrote:
> >>>>> On Mon, Nov 21, 2011 at 05:18:21PM +0800, Wu Fengguang wrote:
> >>>>>> From: Nikanth Karthikesan<knikanth@suse.de>
> >>>>>>
> [...]
> 
> >>
> >> And one that has already been in use for exactly this purpose for
> >> years. Indeed, it's far more flexible because you can give different
> >> types of devices different default readahead settings quite easily,
> >> and it you can set different defaults for just about any tunable
> >> parameter (e.g. readahead, ctq depth, max IO sizes, etc) in the same
> >> way.
> >
> > I'm interested in this usage, too. Would you share some of your rules?
> >
> 
> FYI - This is an example of a rules Suse delivers in SLES @ s390 for a 
> while now. With little modifications it could be used for all Dave 
> mentioned above.

It's a really good example, thank you!

> cat /etc/udev/rules.d/60-readahead.rules
> # 
>  
>  
> 
> # Rules to set an increased default max readahead size for s390 disk 
> devices 
>  
> 
> # This file should be installed in /etc/udev/rules.d 
>  
>  
> 
> # 
>  
> SUBSYSTEM!="block", GOTO="ra_end" 
> 
> ACTION!="add", GOTO="ra_end" 
> 
> # on device add set initial readahead to 512 (instead of in kernel 128) 
> 
> KERNEL=="sd*[!0-9]", ATTR{queue/read_ahead_kb}="512" 
> 
> KERNEL=="dasd*[!0-9]", ATTR{queue/read_ahead_kb}="512" 

So SLES (@s390 and maybe more) is already shipping with 512kb
readahead size? Good to know this!

Thanks,
Fengguang

>  
> 
> 
> LABEL="ra_end"
> 
> -- 
> 
> GrA 1/4 sse / regards, Christian Ehrhardt
> IBM Linux Technology Center, System z Linux Performance

--
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/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Wu Fengguang <fengguang.wu@intel.com>
To: Christian Ehrhardt <ehrhardt@linux.vnet.ibm.com>
Cc: Dave Chinner <david@fromorbit.com>, Jan Kara <jack@suse.cz>,
	Christoph Hellwig <hch@infradead.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Memory Management List <linux-mm@kvack.org>,
	"linux-fsdevel@vger.kernel.org" <linux-fsdevel@vger.kernel.org>,
	Ankit Jain <radical@gmail.com>, Rik van Riel <riel@redhat.com>,
	Nikanth Karthikesan <knikanth@suse.de>,
	LKML <linux-kernel@vger.kernel.org>,
	Andi Kleen <andi@firstfloor.org>
Subject: Re: [PATCH 2/8] readahead: make default readahead size a kernel parameter
Date: Wed, 30 Nov 2011 21:29:28 +0800	[thread overview]
Message-ID: <20111130132928.GA31589@localhost> (raw)
In-Reply-To: <4ED629CB.401@linux.vnet.ibm.com>

On Wed, Nov 30, 2011 at 09:04:11PM +0800, Christian Ehrhardt wrote:
> 
> 
> On 11/28/2011 03:39 AM, Wu Fengguang wrote:
> > On Fri, Nov 25, 2011 at 08:36:33AM +0800, Dave Chinner wrote:
> >> On Thu, Nov 24, 2011 at 11:28:22PM +0100, Jan Kara wrote:
> >>> On Mon 21-11-11 19:35:40, Wu Fengguang wrote:
> >>>> On Mon, Nov 21, 2011 at 06:01:37PM +0800, Christoph Hellwig wrote:
> >>>>> On Mon, Nov 21, 2011 at 05:18:21PM +0800, Wu Fengguang wrote:
> >>>>>> From: Nikanth Karthikesan<knikanth@suse.de>
> >>>>>>
> [...]
> 
> >>
> >> And one that has already been in use for exactly this purpose for
> >> years. Indeed, it's far more flexible because you can give different
> >> types of devices different default readahead settings quite easily,
> >> and it you can set different defaults for just about any tunable
> >> parameter (e.g. readahead, ctq depth, max IO sizes, etc) in the same
> >> way.
> >
> > I'm interested in this usage, too. Would you share some of your rules?
> >
> 
> FYI - This is an example of a rules Suse delivers in SLES @ s390 for a 
> while now. With little modifications it could be used for all Dave 
> mentioned above.

It's a really good example, thank you!

> cat /etc/udev/rules.d/60-readahead.rules
> # 
>  
>  
> 
> # Rules to set an increased default max readahead size for s390 disk 
> devices 
>  
> 
> # This file should be installed in /etc/udev/rules.d 
>  
>  
> 
> # 
>  
> SUBSYSTEM!="block", GOTO="ra_end" 
> 
> ACTION!="add", GOTO="ra_end" 
> 
> # on device add set initial readahead to 512 (instead of in kernel 128) 
> 
> KERNEL=="sd*[!0-9]", ATTR{queue/read_ahead_kb}="512" 
> 
> KERNEL=="dasd*[!0-9]", ATTR{queue/read_ahead_kb}="512" 

So SLES (@s390 and maybe more) is already shipping with 512kb
readahead size? Good to know this!

Thanks,
Fengguang

>  
> 
> 
> LABEL="ra_end"
> 
> -- 
> 
> Grüsse / regards, Christian Ehrhardt
> IBM Linux Technology Center, System z Linux Performance

  reply	other threads:[~2011-11-30 13:29 UTC|newest]

Thread overview: 116+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-21  9:18 [PATCH 0/8] readahead stats/tracing, backwards prefetching and more Wu Fengguang
2011-11-21  9:18 ` Wu Fengguang
2011-11-21  9:18 ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 1/8] block: limit default readahead size for small devices Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 10:00   ` Christoph Hellwig
2011-11-21 10:00     ` Christoph Hellwig
2011-11-21 11:24     ` Wu Fengguang
2011-11-21 11:24       ` Wu Fengguang
2011-11-21 12:47     ` Andi Kleen
2011-11-21 12:47       ` Andi Kleen
2011-11-21 14:46   ` Jeff Moyer
2011-11-21 14:46     ` Jeff Moyer
2011-11-21 14:46     ` Jeff Moyer
2011-11-21 22:52   ` Andrew Morton
2011-11-21 22:52     ` Andrew Morton
2011-11-21 22:52     ` Andrew Morton
2011-11-22 14:23     ` Jeff Moyer
2011-11-22 14:23       ` Jeff Moyer
2011-11-22 14:23       ` Jeff Moyer
2011-11-23 12:18     ` Wu Fengguang
2011-11-23 12:18       ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 2/8] readahead: make default readahead size a kernel parameter Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 10:01   ` Christoph Hellwig
2011-11-21 10:01     ` Christoph Hellwig
2011-11-21 11:35     ` Wu Fengguang
2011-11-21 11:35       ` Wu Fengguang
2011-11-24 22:28       ` Jan Kara
2011-11-24 22:28         ` Jan Kara
2011-11-25  0:36         ` Dave Chinner
2011-11-25  0:36           ` Dave Chinner
2011-11-28  2:39           ` Wu Fengguang
2011-11-28  2:39             ` Wu Fengguang
2011-11-30 13:04             ` Christian Ehrhardt
2011-11-30 13:04               ` Christian Ehrhardt
2011-11-30 13:04               ` Christian Ehrhardt
2011-11-30 13:29               ` Wu Fengguang [this message]
2011-11-30 13:29                 ` Wu Fengguang
2011-11-30 13:29                 ` Wu Fengguang
2011-11-30 16:09                 ` Jan Kara
2011-11-30 16:09                   ` Jan Kara
2011-11-21 13:16   ` Namhyung Kim
2011-11-21 13:16     ` Namhyung Kim
2011-11-21 13:24     ` Wu Fengguang
2011-11-21 13:24       ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 3/8] readahead: replace ra->mmap_miss with ra->ra_flags Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 11:04   ` Steven Whitehouse
2011-11-21 11:04     ` Steven Whitehouse
2011-11-21 11:42     ` Wu Fengguang
2011-11-21 11:42       ` Wu Fengguang
2011-11-21 23:01   ` Andrew Morton
2011-11-21 23:01     ` Andrew Morton
2011-11-21 23:01     ` Andrew Morton
2011-11-23 12:47     ` Wu Fengguang
2011-11-23 12:47       ` Wu Fengguang
2011-11-23 20:31       ` Andrew Morton
2011-11-23 20:31         ` Andrew Morton
2011-11-29  3:42         ` Wu Fengguang
2011-11-29  3:42           ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 4/8] readahead: record readahead patterns Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 23:19   ` Andrew Morton
2011-11-21 23:19     ` Andrew Morton
2011-11-21 23:19     ` Andrew Morton
2011-11-29  2:40     ` Wu Fengguang
2011-11-29  2:40       ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 5/8] readahead: add /debug/readahead/stats Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 14:17   ` Andi Kleen
2011-11-21 14:17     ` Andi Kleen
2011-11-22 14:14     ` Wu Fengguang
2011-11-22 14:14       ` Wu Fengguang
2011-11-21 23:29   ` Andrew Morton
2011-11-21 23:29     ` Andrew Morton
2011-11-21 23:32     ` Andi Kleen
2011-11-21 23:32       ` Andi Kleen
2011-11-29  3:23     ` Wu Fengguang
2011-11-29  3:23       ` Wu Fengguang
2011-11-29  4:49       ` Andrew Morton
2011-11-29  4:49         ` Andrew Morton
2011-11-29  6:41         ` Wu Fengguang
2011-11-29  6:41           ` Wu Fengguang
2011-11-29 12:29           ` Wu Fengguang
2011-11-29 12:29             ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 6/8] readahead: add debug tracing event Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 14:01   ` Steven Rostedt
2011-11-21 14:01     ` Steven Rostedt
2011-11-21  9:18 ` [PATCH 7/8] readahead: basic support for backwards prefetching Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 23:33   ` Andrew Morton
2011-11-21 23:33     ` Andrew Morton
2011-11-21 23:33     ` Andrew Morton
2011-11-29  3:08     ` Wu Fengguang
2011-11-29  3:08       ` Wu Fengguang
2011-11-21  9:18 ` [PATCH 8/8] readahead: dont do start-of-file readahead after lseek() Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21  9:18   ` Wu Fengguang
2011-11-21 23:36   ` Andrew Morton
2011-11-21 23:36     ` Andrew Morton
2011-11-21 23:36     ` Andrew Morton
2011-11-22 14:18     ` Wu Fengguang
2011-11-22 14:18       ` Wu Fengguang
2011-11-21  9:56 ` [PATCH 0/8] readahead stats/tracing, backwards prefetching and more Christoph Hellwig
2011-11-21  9:56   ` Christoph Hellwig
2011-11-21 12:00   ` Wu Fengguang
2011-11-21 12:00     ` 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=20111130132928.GA31589@localhost \
    --to=fengguang.wu@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=andi@firstfloor.org \
    --cc=david@fromorbit.com \
    --cc=ehrhardt@linux.vnet.ibm.com \
    --cc=hch@infradead.org \
    --cc=jack@suse.cz \
    --cc=knikanth@suse.de \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=radical@gmail.com \
    --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 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.