Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755772AbZDDXcb (ORCPT ); Sat, 4 Apr 2009 19:32:31 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753885AbZDDXcV (ORCPT ); Sat, 4 Apr 2009 19:32:21 -0400 Received: from casper.infradead.org ([85.118.1.10]:56983 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750830AbZDDXcU (ORCPT ); Sat, 4 Apr 2009 19:32:20 -0400 Date: Sat, 4 Apr 2009 16:33:49 -0700 From: Arjan van de Ven To: Theodore Tso Cc: Jens Axboe , Linus Torvalds , Linux Kernel Developers List , Ext4 Developers List Subject: Re: [GIT PULL] Ext3 latency fixes Message-ID: <20090404163349.20df1208@infradead.org> In-Reply-To: <20090404232222.GA7480@mit.edu> References: <1238742067-30814-1-git-send-email-tytso@mit.edu> <20090404135719.GA9812@mit.edu> <20090404151649.GE5178@kernel.dk> <20090404173412.GF5178@kernel.dk> <20090404180108.GH5178@kernel.dk> <20090404232222.GA7480@mit.edu> Organization: Intel X-Mailer: Claws Mail 3.7.0 (GTK+ 2.14.7; i386-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-SRS-Rewrite: SMTP reverse-path rewritten from by casper.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1188 Lines: 25 On Sat, 4 Apr 2009 19:22:22 -0400 Theodore Tso wrote: > > However, the full latency fixes all the writes are synchronous, so it > must be the case that the delays are caused by the fact that queue is > getting implicitly unplugged after the synchronous write, and the > problem is no longer the mixing of WRITE and WRITE_SYNC requests as > posted in the commit log for 78f707bf. If we remove the automatic > unplug for WRITE_SYNC requests, and add an explicit unplug where it is > needed, that should fix the performance regression for this particular > sqlite test case. removing the unplug is bound to be bad; after all we're waiting on the IO. But maybe it should be "make the unplug a REALLY short time". At least for rotating storage. For non-rotating .. I'd never wait. -- Arjan van de Ven Intel Open Source Technology Centre For development, discussion and tips for power savings, visit http://www.lesswatts.org -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/