From mboxrd@z Thu Jan 1 00:00:00 1970 From: Al Viro Subject: Re: [RESEND][PATCH] lseek: remove i_mutex Date: Thu, 25 Jun 2009 05:54:52 +0100 Message-ID: <20090625045452.GA8633@ZenIV.linux.org.uk> References: <6.0.0.20.2.20090625115833.05a9ce10@172.19.0.2> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Andrew Morton , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, Christoph Hellwig To: Hisashi Hifumi Return-path: Received: from zeniv.linux.org.uk ([195.92.253.2]:56330 "EHLO ZenIV.linux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750755AbZFYEyx (ORCPT ); Thu, 25 Jun 2009 00:54:53 -0400 Content-Disposition: inline In-Reply-To: <6.0.0.20.2.20090625115833.05a9ce10@172.19.0.2> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, Jun 25, 2009 at 01:41:10PM +0900, Hisashi Hifumi wrote: > Hi, > > Following patch removes i_mutex from generic_file_llseek. > I think the reason of protecting lseek with i_mutex is just > touching i_size (and f_pos) atomically. NAK. There's also a lot of fun with directories - readdir() vs. llseek().