Date: Wed, 16 May 2007 10:15:35 -0700
From: Andrew Morton <akpm@linux-foundation.org>
To: =?ISO-8859-1?B?SvZybg==?= Engel <joern@lazybastard.org>
Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
       Dave Kleikamp <shaggy@linux.vnet.ibm.com>, David Chinner <dgc@sgi.com>,
       Anton Altaparmakov <aia21@cam.ac.uk>, Al Viro <viro@ftp.linux.org.uk>,
       Christoph Hellwig <hch@infradead.org>
Subject: Re: [PATCH resend] introduce I_SYNC
Message-Id: <20070516101535.94e44b3b.akpm@linux-foundation.org>
In-Reply-To: <20070516170110.GC8113@lazybastard.org>
References: <20070516170110.GC8113@lazybastard.org>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 730
Lines: 15

On Wed, 16 May 2007 19:01:14 +0200 J?rn Engel <joern@lazybastard.org> wrote:

> This patch introduces a new flag, I_SYNC and seperates out all sync()
> users of I_LOCK to use the new flag instead.

gack, you like sticking your head in dark and dusty holes.

If we're going to do this then please let's get some exhaustive commentary
in there so that others have a chance of understanding these flags without
having to do the amount of reverse-engineering which you've been put through.
-
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/