From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Morton Subject: Re: [PATCH 06/13] writeback: bdi write bandwidth estimation Date: Wed, 17 Nov 2010 15:08:37 -0800 Message-ID: <20101117150837.a18d56c1.akpm@linux-foundation.org> References: <20101117042720.033773013@intel.com> <20101117042850.002299964@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Jan Kara , Li Shaohua , Christoph Hellwig , Dave Chinner , "Theodore Ts'o" , Chris Mason , Peter Zijlstra , Mel Gorman , Rik van Riel , KOSAKI Motohiro , linux-mm , , LKML To: Wu Fengguang Return-path: In-Reply-To: <20101117042850.002299964@intel.com> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Wed, 17 Nov 2010 12:27:26 +0800 Wu Fengguang wrote: > + w = min(elapsed / (HZ/100), 128UL); I did try setting HZ=10 many years ago, and the kernel blew up. I do recall hearing of people who set HZ very low, perhaps because their huge machines were seeing performance prolems when the timer tick went off. Probably there's no need to do that any more. But still, we shouldn't hard-wire the (HZ >= 100) assumption if we don't absolutely need to, and I don't think it is absolutely needed here. -- 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 policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: email@kvack.org