From mboxrd@z Thu Jan 1 00:00:00 1970 From: Trond Myklebust Subject: Re: [PATCH] locks: provide a file lease method enabling cluster-coherent leases Date: Tue, 05 Jun 2007 19:40:28 -0400 Message-ID: <1181086828.6108.11.camel@heimdal.trondhjem.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: "J. Bruce Fields" , linux-fsdevel@vger.kernel.org, Robert Rappaport , David Teigland , nfs@lists.sourceforge.net To: Marc Eshel Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: nfs-bounces@lists.sourceforge.net Errors-To: nfs-bounces@lists.sourceforge.net List-Id: linux-fsdevel.vger.kernel.org On Tue, 2007-06-05 at 15:56 -0700, Marc Eshel wrote: > Hi Bruce, > The file system does need to keep the local state up to date, like it does > with posix locks, so it might need to call __setlease(). The why we had it > before was that the call to the file system was done from outside of > setlease() and the file system was able to call setlease() which is > exported. Now that the call to the fs moved into setlease() the file > system can not call it anymore so one possible solution would be to export > __setlease(). > Marc. Please just make a vfs_setlease() which has the ability to call down into the filesystem and leave the exported setlease() as a generic method that can continue to be called by the filesystems themselves (and acts as a fallback for vfs_setlease()). That would be closer to the VFS naming conventions. Trond ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ NFS maillist - NFS@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs