From mboxrd@z Thu Jan 1 00:00:00 1970 From: "J. Bruce Fields" Subject: Re: [PATCH 1/10] lockd: add new export operation for nfsv4/lockd locking Date: Sun, 10 Dec 2006 13:31:01 -0500 Message-ID: <20061210183101.GA23387@fieldses.org> References: <11653832602203-git-send-email-bfields@fieldses.org> <1165383260627-git-send-email-bfields@fieldses.org> <457A4F52.1020709@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: linux-fsdevel@vger.kernel.org, nfs@lists.sourceforge.net, Marc Eshel Return-path: To: Wendy Cheng Content-Disposition: inline In-Reply-To: <457A4F52.1020709@redhat.com> 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 Sat, Dec 09, 2006 at 12:53:22AM -0500, Wendy Cheng wrote: > Starting to read these patches .. I see these new APIs are placed in > lockd directory. Was there any discussion before to make these generic > vfs layer functions ? I'm just curious. Yeah--see the introductory email. The new stuff is used only by lockd and nfsd, and there was an understandable desire not to touch the common locking code, but I agree that it'd be more elegant to share a common lock method. So that's probably what we'll do next time around.... --b. ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys - and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ NFS maillist - NFS@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs