From mboxrd@z Thu Jan 1 00:00:00 1970 From: Artem Bityutskiy Subject: Re: [PATCH 0/11] Per-bdi writeback flusher threads v9 Date: Wed, 03 Jun 2009 14:12:02 +0300 Message-ID: <4A265A82.8060909@nokia.com> References: <1243511204-2328-1-git-send-email-jens.axboe@oracle.com> Reply-To: Artem.Bityutskiy@nokia.com Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, tytso@mit.edu, chris.mason@oracle.com, david@fromorbit.com, hch@infradead.org, akpm@linux-foundation.org, jack@suse.cz, yanmin_zhang@linux.intel.com, richard@rsk.demon.co.uk, damien.wyart@free.fr To: Jens Axboe Return-path: Received: from smtp.nokia.com ([192.100.122.230]:24336 "EHLO mgw-mx03.nokia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752458AbZFCLM4 (ORCPT ); Wed, 3 Jun 2009 07:12:56 -0400 In-Reply-To: <1243511204-2328-1-git-send-email-jens.axboe@oracle.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: Jens Axboe wrote: > Here's the 9th version of the writeback patches. Changes since v8: >=20 > - Fix a bdi_work on-stack allocation hang. I hope this fixes Ted's > issue. > - Get rid of the explicit wait queues, we can just use wake_up_proces= s() > since it's just for that one task. > - Add separate "sync_supers" thread that makes sure that the dirty > super blocks get written. We cannot safely do this from bdi_forker_= task(), > as that risks deadlocking on ->s_umount. Artem, I implemented this > by doing the wake ups from a timer so that it would be easier for y= ou > to just deactivate the timer when there are no super blocks. I wonder if you would consider to work on top of the latest VFS changes= : git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs-2.6.git for-next =46or me the problem is that my original patches were created against the VFS tree, and they do not apply nicely to your tree. So what I've tried to do - I applied your patches on top of the VFS tree. But they did not apply cleanly either. I'm currently working on merging them, but I thought it is better to ask if you already did this. = =20 --=20 Best Regards, Artem Bityutskiy (=D0=90=D1=80=D1=82=D1=91=D0=BC =D0=91=D0=B8=D1=82=D1=8E= =D1=86=D0=BA=D0=B8=D0=B9) -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel= " in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html