Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758164AbZKRRya (ORCPT ); Wed, 18 Nov 2009 12:54:30 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1758124AbZKRRy3 (ORCPT ); Wed, 18 Nov 2009 12:54:29 -0500 Received: from earthlight.etchedpixels.co.uk ([81.2.110.250]:38167 "EHLO www.etchedpixels.co.uk" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1758074AbZKRRy2 (ORCPT ); Wed, 18 Nov 2009 12:54:28 -0500 Date: Wed, 18 Nov 2009 17:55:17 +0000 From: Alan Cox To: Oliver Neukum Cc: Jamie Lokier , Jan Blunck , linux-fsdevel@vger.kernel.org, "Linux-Kernel Mailinglist" , Andrew Morton , jkacur@redhat.com, Thomas Gleixner , Arnd Bergmann , Christoph Hellwig , =?ISO-8859-14?B?RnLpZOlyaWM=?= Weisbecker , Alexander Viro Subject: Re: [PATCH 1/2] BKL: Remove BKL from default_llseek() Message-ID: <20091118175517.174ec906@lxorguk.ukuu.org.uk> In-Reply-To: <200911181835.55007.oliver@neukum.org> References: <1258560457-15129-1-git-send-email-jblunck@suse.de> <20091118171524.4d2f8cec@lxorguk.ukuu.org.uk> <20091118172730.GD28723@shareable.org> <200911181835.55007.oliver@neukum.org> X-Mailer: Claws Mail 3.7.3 (GTK+ 2.14.7; x86_64-redhat-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 851 Lines: 20 > > Of course, drivers shouldn't be using f_pos outside their llseek > > method, as they should all behave the same with pread/pwrite as with > > llseek+read/write. > > Might not a driver update f_pos after read/write? The driver updates the passed pointer to an offset. It has no idea how to lock that and that is isolated and handled higher up the stack. There are no obvious reasons to peer at f_pos. I've so far checked all of drivers/char and that is clean (as well as being the most likely suspect for old code). A sweep of the other driver subsystems should be enough to find any offenders Alan -- 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/