From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: [PATCH 3/5] mm: Implement IO-less balance_dirty_pages() Date: Fri, 04 Feb 2011 14:09:15 +0100 Message-ID: <1296824955.26581.645.camel@laptop> References: <1296783534-11585-1-git-send-email-jack@suse.cz> <1296783534-11585-4-git-send-email-jack@suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, linux-mm@kvack.org, Andrew Morton , Christoph Hellwig , Dave Chinner , Wu Fengguang To: Jan Kara Return-path: Received: from casper.infradead.org ([85.118.1.10]:47884 "EHLO casper.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752568Ab1BDNPy (ORCPT ); Fri, 4 Feb 2011 08:15:54 -0500 Received: from j77219.upc-j.chello.nl ([24.132.77.219] helo=dyad.programming.kicks-ass.net) by casper.infradead.org with esmtpsa (Exim 4.72 #1 (Red Hat Linux)) id 1PlLVx-0003cE-KI for linux-fsdevel@vger.kernel.org; Fri, 04 Feb 2011 13:15:53 +0000 In-Reply-To: <1296783534-11585-4-git-send-email-jack@suse.cz> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Fri, 2011-02-04 at 02:38 +0100, Jan Kara wrote: > +struct balance_waiter { > + struct list_head bw_list; > + unsigned long bw_to_write; /* Number of pages to wait for to > + get written */ That names somehow rubs me the wrong way.. the name suggests we need to do the writing, whereas we only wait for them to be written. > + struct task_struct *bw_task; /* Task waiting for IO */ > +};