From patchwork Fri Aug 15 00:09:01 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kinglong Mee X-Patchwork-Id: 4725941 Return-Path: X-Original-To: patchwork-linux-nfs@patchwork.kernel.org Delivered-To: patchwork-parsemail@patchwork2.web.kernel.org Received: from mail.kernel.org (mail.kernel.org [198.145.19.201]) by patchwork2.web.kernel.org (Postfix) with ESMTP id 38347C0338 for ; Fri, 15 Aug 2014 00:09:17 +0000 (UTC) Received: from mail.kernel.org (localhost [127.0.0.1]) by mail.kernel.org (Postfix) with ESMTP id 56DA720142 for ; Fri, 15 Aug 2014 00:09:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 722D32012B for ; Fri, 15 Aug 2014 00:09:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751352AbaHOAJO (ORCPT ); Thu, 14 Aug 2014 20:09:14 -0400 Received: from mail-pa0-f48.google.com ([209.85.220.48]:57039 "EHLO mail-pa0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750916AbaHOAJN (ORCPT ); Thu, 14 Aug 2014 20:09:13 -0400 Received: by mail-pa0-f48.google.com with SMTP id et14so2518143pad.35 for ; Thu, 14 Aug 2014 17:09:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=message-id:date:from:user-agent:mime-version:to:cc:subject :references:in-reply-to:content-type:content-transfer-encoding; bh=LhDrclO+N1ITRPOGlCjersfB39+TGPelVUyec7f8LCs=; b=OmddagadTnmoc8McHpjqso0xeH4lonarpKGFabW8jIUbAM5+axkjv/EvgtT+53HS3R fNG31U/U4WKTiAHBCvYoeBRx9vJLPxseXe/YH/KrABvYmnZHNpLVv8V0oU8eslBk5B++ h0+wBtZtCBsHc/hHd5iXozV8KO1Wuf9ASslktnHyfdCLHld62bo3ELoT1YJVa7/dlLyu bRqCtsG/ZeJAbXzfNisgJ387tzF9cRBBBAlcCks3KUwOrU3MTt3cokKQF39aj09Ybekz 0twCRZUb0PhIsst5R8NL+yYSk1fiYGrF1AaWcwj4ZAxjSbsbYjowKM6SZ369SwWX/u3r KjZA== X-Received: by 10.70.132.162 with SMTP id ov2mr13761504pdb.118.1408061352768; Thu, 14 Aug 2014 17:09:12 -0700 (PDT) Received: from [192.168.0.100] ([171.208.179.144]) by mx.google.com with ESMTPSA id vy3sm6439298pbc.92.2014.08.14.17.09.08 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 14 Aug 2014 17:09:11 -0700 (PDT) Message-ID: <53ED4F9D.3070103@gmail.com> Date: Fri, 15 Aug 2014 08:09:01 +0800 From: Kinglong Mee User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.0 MIME-Version: 1.0 To: Jeff Layton CC: "J. Bruce Fields" , Linux NFS Mailing List , Trond Myklebust , linux-fsdevel@vger.kernel.org, kinglongmee@gmail.com Subject: [PATCH 3/5 v3] locks: New ops in file_lock_operations for copy/release owner References: <53BAAAC5.9000106@gmail.com> <53E22EA5.70708@gmail.com> <20140809065112.700e0ecc@tlielax.poochiereds.net> <53E791F1.40802@gmail.com> In-Reply-To: <53E791F1.40802@gmail.com> Sender: linux-nfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-nfs@vger.kernel.org X-Spam-Status: No, score=-7.5 required=5.0 tests=BAYES_00, DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED, FREEMAIL_FROM, RCVD_IN_DNSWL_HI, RP_MATCHES_RCVD, T_DKIM_INVALID, UNPARSEABLE_RELAY autolearn=ham version=3.3.1 X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on mail.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP NFSD or other lockmanager may increase the owner's reference, so adds two new options for copying and releasing owner. v3: same as v2, isn't exist in v1. Signed-off-by: Kinglong Mee --- fs/locks.c | 12 ++++++++++-- include/linux/fs.h | 2 ++ 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/fs/locks.c b/fs/locks.c index fe52abb..cbe9b82 100644 --- a/fs/locks.c +++ b/fs/locks.c @@ -230,8 +230,12 @@ void locks_release_private(struct file_lock *fl) fl->fl_ops->fl_release_private(fl); fl->fl_ops = NULL; } - fl->fl_lmops = NULL; + if (fl->fl_lmops) { + if (fl->fl_lmops->lm_release_owner) + fl->fl_lmops->lm_release_owner(fl); + fl->fl_lmops = NULL; + } } EXPORT_SYMBOL_GPL(locks_release_private); @@ -274,8 +278,12 @@ static void locks_copy_private(struct file_lock *new, struct file_lock *fl) fl->fl_ops->fl_copy_lock(new, fl); new->fl_ops = fl->fl_ops; } - if (fl->fl_lmops) + + if (fl->fl_lmops) { + if (fl->fl_lmops->lm_copy_owner) + fl->fl_lmops->lm_copy_owner(new, fl); new->fl_lmops = fl->fl_lmops; + } } /* diff --git a/include/linux/fs.h b/include/linux/fs.h index a383a30..54bf69d 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h @@ -868,6 +868,8 @@ struct file_lock_operations { struct lock_manager_operations { int (*lm_compare_owner)(struct file_lock *, struct file_lock *); unsigned long (*lm_owner_key)(struct file_lock *); + void (*lm_copy_owner)(struct file_lock *, struct file_lock *); + void (*lm_release_owner)(struct file_lock *); void (*lm_notify)(struct file_lock *); /* unblock callback */ int (*lm_grant)(struct file_lock *, int); void (*lm_break)(struct file_lock *);