From owner-freebsd-arch@FreeBSD.ORG Sat Apr 12 17:24:12 2008 Return-Path: Delivered-To: arch@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id E9A861065685 for ; Sat, 12 Apr 2008 17:24:12 +0000 (UTC) (envelope-from dfr@rabson.org) Received: from itchy.rabson.org (mail.rabson.org [IPv6:2002:50b1:e8f2:1::143]) by mx1.freebsd.org (Postfix) with ESMTP id 95BD58FC23 for ; Sat, 12 Apr 2008 17:24:12 +0000 (UTC) (envelope-from dfr@rabson.org) Received: from [IPv6:2002:50b1:e8f2:1:21b:63ff:feb8:5abc] (unknown [IPv6:2002:50b1:e8f2:1:21b:63ff:feb8:5abc]) by itchy.rabson.org (Postfix) with ESMTP id BC0603FA7; Sat, 12 Apr 2008 18:24:11 +0100 (BST) Message-Id: <41ED3941-E5E6-45F0-B880-C1B2861FDE32@rabson.org> From: Doug Rabson To: Kirk McKusick In-Reply-To: <200804121703.m3CH3StJ081660@chez.mckusick.com> Content-Type: text/plain; charset=US-ASCII; format=flowed; delsp=yes Content-Transfer-Encoding: 7bit Mime-Version: 1.0 (Apple Message framework v919.2) Date: Sat, 12 Apr 2008 18:24:11 +0100 References: <200804121703.m3CH3StJ081660@chez.mckusick.com> X-Mailer: Apple Mail (2.919.2) Cc: arch@freebsd.org Subject: Re: VOP_LEASE X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 12 Apr 2008 17:24:13 -0000 On 12 Apr 2008, at 18:03, Kirk McKusick wrote: >> Date: Sat, 12 Apr 2008 02:13:15 -1000 (HST) >> From: Jeff Roberson >> To: arch@freebsd.org >> Subject: VOP_LEASE >> >> As far as I can tell this has never been used. Unless someone can >> show me >> otherwise I'm going to go ahead and remove it. >> >> Thanks, >> Jeff > > VOP_LEASE is used by NQNFS and NFSv4. It notifies them when a file > is modified locally so that they know to update any outstanding > leases (e.g., evict any write lease for the file and do callbacks > for any read leases for the file). Deleting VOP_LEASE would break > NFS big time. I think our NQNFS support might have been removed some time ago - I can't see any calls to VOP_LEASE in the code right now. Something like VOP_LEASE would certainly be useful for a hypothetical future NFSv4 server. I believe that samba could use it too for its oplocks feature which appears to be similar to NQNFS's leases and NFSv4's delegations.