From mboxrd@z Thu Jan  1 00:00:00 1970
From: David Howells <dhowells@redhat.com>
Subject: Re: [-mm PATCH] ocfs2: Shared writeable mmap
Date: Tue, 20 Jun 2006 13:59:44 +0100
Message-ID: <9630.1150808384@warthog.cambridge.redhat.com>
References: <1150787267.28517.126.camel@lappy>  <20060619234643.GK3082@ca-server1.us.oracle.com> <20060619170736.65237ce7.akpm@osdl.org>
Cc: Andrew Morton <akpm@osdl.org>,
	Mark Fasheh <mark.fasheh@oracle.com>, dhowells@redhat.com,
	linux-fsdevel@vger.kernel.org, ocfs2-devel@oss.oracle.com
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from mx1.redhat.com ([66.187.233.31]:971 "EHLO mx1.redhat.com")
	by vger.kernel.org with ESMTP id S1750736AbWFTM7x (ORCPT
	<rfc822;linux-fsdevel@vger.kernel.org>);
	Tue, 20 Jun 2006 08:59:53 -0400
In-Reply-To: <1150787267.28517.126.camel@lappy> 
To: Peter Zijlstra <a.p.zijlstra@chello.nl>
Sender: linux-fsdevel-owner@vger.kernel.org
List-Id: linux-fsdevel.vger.kernel.org

Peter Zijlstra <a.p.zijlstra@chello.nl> wrote:

> -	if (unlikely(vma->vm_flags & VM_SHARED)) {
> +	if (unlikely(vma->vm_flags & (VM_SHARED|VM_WRITE) ==
> +				VM_SHARED|VM_WRITE) {

NAK!

"==" is higher priority than "|".  What you meant was:

-	if (unlikely(vma->vm_flags & VM_SHARED)) {
+	if (unlikely(vma->vm_flags & (VM_SHARED|VM_WRITE) ==
+				(VM_SHARED|VM_WRITE)) {

David