Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754233AbYJFNBT (ORCPT ); Mon, 6 Oct 2008 09:01:19 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753169AbYJFNBE (ORCPT ); Mon, 6 Oct 2008 09:01:04 -0400 Received: from mx1.redhat.com ([66.187.233.31]:34838 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753161AbYJFNBD (ORCPT ); Mon, 6 Oct 2008 09:01:03 -0400 Date: Mon, 6 Oct 2008 09:00:14 -0400 (EDT) From: Mikulas Patocka X-X-Sender: mpatocka@hs20-bc2-1.build.redhat.com To: Arjan van de Ven cc: Andrew Morton , linux-kernel@vger.kernel.org, agk@redhat.com, mbroz@redhat.com, chris@arachsys.com Subject: Re: [PATCH 2/3] Fix fsync livelock In-Reply-To: <20081005212031.63ad246a@infradead.org> Message-ID: References: <20080911101616.GA24064@agk.fab.redhat.com> <20080923154905.50d4b0fa.akpm@linux-foundation.org> <20080923164623.ce82c1c2.akpm@linux-foundation.org> <20081001225404.4e973465.akpm@linux-foundation.org> <20081005153306.7e644c9f@infradead.org> <20081005160724.54dd1a27@infradead.org> <20081005162847.7bf0ead1@infradead.org> <20081005173019.0a358b09@infradead.org> <20081005212031.63ad246a@infradead.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 984 Lines: 26 On Sun, 5 Oct 2008, Arjan van de Ven wrote: > On Sun, 5 Oct 2008 23:30:51 -0400 (EDT > > The point is that many fsync()s may run in parallel and you have just > > one inode and just one chain. And if you add two-word list_head to a > > page, to link it on this list, many developers will hate it for > > increasing its size. > > why to a page? > a list head in the inode and chain up the bios.... And if you want to wait for a bio submitted by a different process? There's no way you can find the bio from the page. > or not make an actual list but just a "is the previous one done" thing > it's not all that hard to get something that works on a per inode basis, > that gives "wait for all io upto this one". So code it :) Mikulas -- 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/