From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [RFC PATCH 4/4] Directory listing support for union mounted directories. Date: Wed, 20 Jun 2007 18:02:21 +0100 Message-ID: <20070620170221.GA13237@infradead.org> References: <20070620055050.GB4267@in.ibm.com> <20070620055418.GF4267@in.ibm.com> <20070620120947.GA3973@infradead.org> <1182349348.6225.15.camel@heimdal.trondhjem.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Christoph Hellwig , Bharata B Rao , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Jan Blunck To: Trond Myklebust Return-path: Received: from pentafluge.infradead.org ([213.146.154.40]:53272 "EHLO pentafluge.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753570AbXFTRC2 (ORCPT ); Wed, 20 Jun 2007 13:02:28 -0400 Content-Disposition: inline In-Reply-To: <1182349348.6225.15.camel@heimdal.trondhjem.org> Sender: linux-fsdevel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Wed, Jun 20, 2007 at 10:22:28AM -0400, Trond Myklebust wrote: > No it shouldn't. The struct file contains other stateful information > from the open() call (such as authentication info) that needs to be > passed into readdir. Which is exactly that problem this tries to solve. Once you have union mounts you'll have a single open file descriptor for multiple actual directories. Beause of that you can't simply attach to the state to the struct file but have to keep it in a different way.