From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754555Ab1CCIR4 (ORCPT ); Thu, 3 Mar 2011 03:17:56 -0500 Received: from mga01.intel.com ([192.55.52.88]:59999 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750910Ab1CCIRz (ORCPT ); Thu, 3 Mar 2011 03:17:55 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.62,257,1297065600"; d="scan'208";a="893290335" Message-Id: <20110303074949.419321686@intel.com> User-Agent: quilt/0.48-1 Date: Thu, 03 Mar 2011 14:45:11 +0800 From: Wu Fengguang To: Andrew Morton CC: Jan Kara , Chris Mason , Wu Fengguang CC: Christoph Hellwig CC: Trond Myklebust CC: Dave Chinner CC: "Theodore Ts'o" CC: Peter Zijlstra CC: Mel Gorman CC: Rik van Riel CC: KOSAKI Motohiro CC: Greg Thelen CC: Minchan Kim CC: Vivek Goyal CC: Andrea Righi CC: Balbir Singh Cc: linux-mm Cc: Cc: LKML Subject: [PATCH 06/27] btrfs: lower the dirty balance poll interval References: <20110303064505.718671603@intel.com> Content-Disposition: inline; filename=btrfs-limit-nr-dirtied.patch Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Call balance_dirty_pages_ratelimit_nr() on every 32 pages dirtied. Tests show that original larger intervals can easily make the bdi dirty limit exceeded on 100 concurrent dd. CC: Chris Mason Signed-off-by: Wu Fengguang --- fs/btrfs/file.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) --- linux-next.orig/fs/btrfs/file.c 2011-03-02 20:15:19.000000000 +0800 +++ linux-next/fs/btrfs/file.c 2011-03-02 20:35:07.000000000 +0800 @@ -949,9 +949,8 @@ static ssize_t btrfs_file_aio_write(stru } iov_iter_init(&i, iov, nr_segs, count, num_written); - nrptrs = min((iov_iter_count(&i) + PAGE_CACHE_SIZE - 1) / - PAGE_CACHE_SIZE, PAGE_CACHE_SIZE / - (sizeof(struct page *))); + nrptrs = min(DIV_ROUND_UP(iov_iter_count(&i), PAGE_CACHE_SIZE), + min(32UL, PAGE_CACHE_SIZE / sizeof(struct page *))); pages = kmalloc(nrptrs * sizeof(struct page *), GFP_KERNEL); if (!pages) { ret = -ENOMEM;