Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757310AbZKKIeC (ORCPT ); Wed, 11 Nov 2009 03:34:02 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756911AbZKKIeB (ORCPT ); Wed, 11 Nov 2009 03:34:01 -0500 Received: from fxip-0047f.externet.hu ([88.209.222.127]:46658 "EHLO pomaz-ex.szeredi.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755719AbZKKIeA (ORCPT ); Wed, 11 Nov 2009 03:34:00 -0500 To: Trond Myklebust CC: miklos@szeredi.hu, jlayton@redhat.com, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-nfs@vger.kernel.org, ebiederm@xmission.com, adobriyan@gmail.com, viro@ZenIV.linux.org.uk, jamie@shareable.org In-reply-to: <1257927970.5138.24.camel@heimdal.trondhjem.org> (message from Trond Myklebust on Wed, 11 Nov 2009 17:26:10 +0900) Subject: Re: [PATCH 0/2] vfs: ensure that dentries are revalidated on open (try #2) References: <1257870456-31188-1-git-send-email-jlayton@redhat.com> <1257927970.5138.24.camel@heimdal.trondhjem.org> Message-Id: From: Miklos Szeredi Date: Wed, 11 Nov 2009 09:33:56 +0100 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 820 Lines: 20 On Wed, 11 Nov 2009, Trond Myklebust wrote: > > Having said that, doing revalidation for proc symlinks and bind mounts > > (and not just for opens) might make sense. This is something similar > > to FS_REVAL_DOT, so perhaps make it conditional on this flag (or a > > new, appropriately named one). > > Aren't both proc symlinks and bind mounts pretty much guaranteed to > point to a valid dentry? It could for example happen that client bind mounts a regular file, then the file is unlinked on the server. Then the bind mounted dentry is no longer valid. Thanks, Miklos -- 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/