From: Christoph Hellwig <hch@infradead.org> To: Mel Gorman <mgorman@suse.de> Cc: jack@suse.cz, xfs@oss.sgi.com, Christoph Hellwig <hch@infradead.org>, linux-mm@kvack.org, Wu Fengguang <fengguang.wu@intel.com>, Johannes Weiner <jweiner@redhat.com> Subject: Re: [PATCH 03/27] xfs: use write_cache_pages for writeback clustering Date: Fri, 1 Jul 2011 11:15:09 -0400 [thread overview] Message-ID: <20110701151509.GA30620@infradead.org> (raw) In-Reply-To: <20110701145935.GB29530@suse.de> On Fri, Jul 01, 2011 at 03:59:35PM +0100, Mel Gorman wrote: > On Fri, Jul 01, 2011 at 05:33:05AM -0400, Christoph Hellwig wrote: > > Johannes, Mel, Wu, > > Am adding Jan Kara as he has been working on writeback efficiency > recently as well. > > > Dave has been stressing some XFS patches of mine that remove the XFS > > internal writeback clustering in favour of using write_cache_pages. > > > > Against what kernel? 2.6.38 was a disaster for reclaim I've been > finding out this week. I don't know about 2.6.38.8. 2.6.39 was better. The patch series is against current 3.0-rc, I assume that's what Dave tested as well. > I'm assuming "test 180" is from xfstests which was not one of the tests > I used previously. To run with 1000 files instead of 100, was the file > "180" simply editted to make it look like this loop instead? Yes. to both questions. _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs
WARNING: multiple messages have this Message-ID (diff)
From: Christoph Hellwig <hch@infradead.org> To: Mel Gorman <mgorman@suse.de> Cc: Christoph Hellwig <hch@infradead.org>, Johannes Weiner <jweiner@redhat.com>, Wu Fengguang <fengguang.wu@intel.com>, Dave Chinner <david@fromorbit.com>, xfs@oss.sgi.com, jack@suse.cz, linux-mm@kvack.org Subject: Re: [PATCH 03/27] xfs: use write_cache_pages for writeback clustering Date: Fri, 1 Jul 2011 11:15:09 -0400 [thread overview] Message-ID: <20110701151509.GA30620@infradead.org> (raw) In-Reply-To: <20110701145935.GB29530@suse.de> On Fri, Jul 01, 2011 at 03:59:35PM +0100, Mel Gorman wrote: > On Fri, Jul 01, 2011 at 05:33:05AM -0400, Christoph Hellwig wrote: > > Johannes, Mel, Wu, > > Am adding Jan Kara as he has been working on writeback efficiency > recently as well. > > > Dave has been stressing some XFS patches of mine that remove the XFS > > internal writeback clustering in favour of using write_cache_pages. > > > > Against what kernel? 2.6.38 was a disaster for reclaim I've been > finding out this week. I don't know about 2.6.38.8. 2.6.39 was better. The patch series is against current 3.0-rc, I assume that's what Dave tested as well. > I'm assuming "test 180" is from xfstests which was not one of the tests > I used previously. To run with 1000 files instead of 100, was the file > "180" simply editted to make it look like this loop instead? Yes. to both questions. -- 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>
next prev parent reply other threads:[~2011-07-01 15:15 UTC|newest] Thread overview: 100+ messages / expand[flat|nested] mbox.gz Atom feed top 2011-06-29 14:01 [PATCH 00/27] patch queue for Linux 3.1 Christoph Hellwig 2011-06-29 14:01 ` [PATCH 01/27] xfs: PF_FSTRANS should never be set in ->writepage Christoph Hellwig 2011-06-30 1:34 ` Dave Chinner 2011-06-29 14:01 ` [PATCH 02/27] xfs: remove the unused ilock_nowait codepath in writepage Christoph Hellwig 2011-06-30 0:15 ` Dave Chinner 2011-06-30 1:26 ` Dave Chinner 2011-06-30 6:55 ` Christoph Hellwig 2011-06-29 14:01 ` [PATCH 03/27] xfs: use write_cache_pages for writeback clustering Christoph Hellwig 2011-06-30 2:00 ` Dave Chinner 2011-06-30 2:48 ` Dave Chinner 2011-06-30 6:57 ` Christoph Hellwig 2011-07-01 2:22 ` Dave Chinner 2011-07-01 4:18 ` Dave Chinner 2011-07-01 8:59 ` Christoph Hellwig 2011-07-01 9:20 ` Dave Chinner 2011-07-01 9:33 ` Christoph Hellwig 2011-07-01 9:33 ` Christoph Hellwig 2011-07-01 14:59 ` Mel Gorman 2011-07-01 14:59 ` Mel Gorman 2011-07-01 15:15 ` Christoph Hellwig [this message] 2011-07-01 15:15 ` Christoph Hellwig 2011-07-02 2:42 ` Dave Chinner 2011-07-02 2:42 ` Dave Chinner 2011-07-05 14:10 ` Mel Gorman 2011-07-05 14:10 ` Mel Gorman 2011-07-05 15:55 ` Dave Chinner 2011-07-05 15:55 ` Dave Chinner 2011-07-11 10:26 ` Christoph Hellwig 2011-07-11 10:26 ` Christoph Hellwig 2011-07-01 15:41 ` Wu Fengguang 2011-07-01 15:41 ` Wu Fengguang 2011-07-04 3:25 ` Dave Chinner 2011-07-04 3:25 ` Dave Chinner 2011-07-05 14:34 ` Mel Gorman 2011-07-05 14:34 ` Mel Gorman 2011-07-06 1:23 ` Dave Chinner 2011-07-06 1:23 ` Dave Chinner 2011-07-11 11:10 ` Christoph Hellwig 2011-07-11 11:10 ` Christoph Hellwig 2011-07-06 4:53 ` Wu Fengguang 2011-07-06 4:53 ` Wu Fengguang 2011-07-06 6:47 ` Minchan Kim 2011-07-06 6:47 ` Minchan Kim 2011-07-06 7:17 ` Dave Chinner 2011-07-06 7:17 ` Dave Chinner 2011-07-06 15:12 ` Johannes Weiner 2011-07-06 15:12 ` Johannes Weiner 2011-07-08 9:54 ` Dave Chinner 2011-07-08 9:54 ` Dave Chinner 2011-07-11 17:20 ` Johannes Weiner 2011-07-11 17:20 ` Johannes Weiner 2011-07-11 17:24 ` Christoph Hellwig 2011-07-11 17:24 ` Christoph Hellwig 2011-07-11 19:09 ` Rik van Riel 2011-07-11 19:09 ` Rik van Riel 2011-07-01 8:51 ` Christoph Hellwig 2011-06-29 14:01 ` [PATCH 04/27] xfs: cleanup xfs_add_to_ioend Christoph Hellwig 2011-06-29 22:13 ` Alex Elder 2011-06-30 2:00 ` Dave Chinner 2011-06-29 14:01 ` [PATCH 05/27] xfs: work around bogus gcc warning in xfs_allocbt_init_cursor Christoph Hellwig 2011-06-29 22:13 ` Alex Elder 2011-06-29 14:01 ` [PATCH 06/27] xfs: split xfs_setattr Christoph Hellwig 2011-06-29 22:13 ` Alex Elder 2011-06-30 7:03 ` Christoph Hellwig 2011-06-30 12:28 ` Alex Elder 2011-06-30 2:11 ` Dave Chinner 2011-06-29 14:01 ` [PATCH 08/27] xfs: kill xfs_itruncate_start Christoph Hellwig 2011-06-29 22:13 ` Alex Elder 2011-06-29 14:01 ` [PATCH 09/27] xfs: split xfs_itruncate_finish Christoph Hellwig 2011-06-30 2:44 ` Dave Chinner 2011-06-30 7:18 ` Christoph Hellwig 2011-06-29 14:01 ` [PATCH 10/27] xfs: improve sync behaviour in the fact of aggressive dirtying Christoph Hellwig 2011-06-30 2:52 ` Dave Chinner 2011-06-29 14:01 ` [PATCH 11/27] xfs: fix filesystsem freeze race in xfs_trans_alloc Christoph Hellwig 2011-06-30 2:59 ` Dave Chinner 2011-06-29 14:01 ` [PATCH 12/27] xfs: remove i_transp Christoph Hellwig 2011-06-30 3:00 ` Dave Chinner 2011-06-29 14:01 ` [PATCH 13/27] xfs: factor out xfs_dir2_leaf_find_entry Christoph Hellwig 2011-06-30 6:11 ` Dave Chinner 2011-06-30 7:34 ` Christoph Hellwig 2011-06-29 14:01 ` [PATCH 14/27] xfs: cleanup shortform directory inode number handling Christoph Hellwig 2011-06-30 6:35 ` Dave Chinner 2011-06-30 7:39 ` Christoph Hellwig 2011-06-29 14:01 ` [PATCH 15/27] xfs: kill struct xfs_dir2_sf Christoph Hellwig 2011-06-30 7:04 ` Dave Chinner 2011-06-30 7:09 ` Christoph Hellwig 2011-06-29 14:01 ` [PATCH 16/27] xfs: cleanup the defintion of struct xfs_dir2_sf_entry Christoph Hellwig 2011-06-29 14:01 ` [PATCH 17/27] xfs: avoid usage of struct xfs_dir2_block Christoph Hellwig 2011-06-29 14:01 ` [PATCH 18/27] xfs: kill " Christoph Hellwig 2011-06-29 14:01 ` [PATCH 19/27] xfs: avoid usage of struct xfs_dir2_data Christoph Hellwig 2011-06-29 14:01 ` [PATCH 20/27] xfs: kill " Christoph Hellwig 2011-06-29 14:01 ` [PATCH 21/27] xfs: cleanup the defintion of struct xfs_dir2_data_entry Christoph Hellwig 2011-06-29 14:01 ` [PATCH 22/27] xfs: cleanup struct xfs_dir2_leaf Christoph Hellwig 2011-06-29 14:01 ` [PATCH 23/27] xfs: remove the unused xfs_bufhash structure Christoph Hellwig 2011-06-29 14:01 ` [PATCH 24/27] xfs: clean up buffer locking helpers Christoph Hellwig 2011-06-29 14:01 ` [PATCH 25/27] xfs: return the buffer locked from xfs_buf_get_uncached Christoph Hellwig 2011-06-29 14:01 ` [PATCH 26/27] xfs: cleanup I/O-related buffer flags Christoph Hellwig 2011-06-29 14:01 ` [PATCH 27/27] xfs: avoid a few disk cache flushes Christoph Hellwig 2011-06-30 6:36 ` [PATCH 00/27] patch queue for Linux 3.1 Dave Chinner 2011-06-30 6:50 ` Christoph Hellwig
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=20110701151509.GA30620@infradead.org \ --to=hch@infradead.org \ --cc=fengguang.wu@intel.com \ --cc=jack@suse.cz \ --cc=jweiner@redhat.com \ --cc=linux-mm@kvack.org \ --cc=mgorman@suse.de \ --cc=xfs@oss.sgi.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: linkBe 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.