Subject: Re: Relative atime (was Re: What's in ocfs2.git)
From: Arjan van de Ven <arjan@infradead.org>
To: Eric Dumazet <dada1@cosmosbay.com>
Cc: Andrew Morton <akpm@osdl.org>, Valerie Henson <val_henson@linux.intel.com>,
       mark.fasheh@oracle.com, steve@chygwyn.com, linux-kernel@vger.kernel.org,
       ocfs2-devel@oss.oracle.com, linux-fsdevel@vger.kernel.org,
       viro@ftp.linux.org.uk
In-Reply-To: <200612061042.58595.dada1@cosmosbay.com>
References: <20061203203149.GC19617@ca-server1.us.oracle.com>
	 <20061205003619.GC8482@goober> <20061205205802.92b91ce1.akpm@osdl.org>
	 <200612061042.58595.dada1@cosmosbay.com>
Content-Type: text/plain
Organization: Intel International BV
Date: Wed, 06 Dec 2006 13:48:21 +0100
Message-Id: <1165409301.3233.443.camel@laptopd505.fenrus.org>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 599
Lines: 22


> 
> > 	if (inode->i_sb->s_flags & MS_NOATIME)
> > 		return;
> So that that one can be deleted.


Hi,

I would mostly expect the compiler to be relatively smart about this and
group a bunch of these tests together... so I rather see readable code
than optimized code for something the compiler should be doing for us ;)

Greetings,
   Arjan van de Ven

-
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/