From mboxrd@z Thu Jan 1 00:00:00 1970 From: "J. Bruce Fields" Subject: Re: [PATCH v7 10/17] locks: make /proc/locks show IS_FILE_PVT locks with a P suffix Date: Sat, 29 Mar 2014 15:05:56 -0400 Message-ID: <20140329190556.GA11085@fieldses.org> References: <1395261961-10855-1-git-send-email-jlayton@redhat.com> <1395261961-10855-11-git-send-email-jlayton@redhat.com> <20140325002047.GF12979@fieldses.org> <20140324175711.73ad7310@ipyr.poochiereds.net> <20140325041854.GA23539@fieldses.org> <20140329101831.423918a2@tlielax.poochiereds.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: viro@ZenIV.linux.org.uk, linux-fsdevel@vger.kernel.org To: Jeff Layton Return-path: Received: from fieldses.org ([174.143.236.118]:44722 "EHLO fieldses.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752463AbaC2TF7 (ORCPT ); Sat, 29 Mar 2014 15:05:59 -0400 Content-Disposition: inline In-Reply-To: <20140329101831.423918a2@tlielax.poochiereds.net> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Sat, Mar 29, 2014 at 10:18:31AM -0400, Jeff Layton wrote: > Thanks for the ACK, but I think I'd like to modify this patch a bit. > Since POSIX hasn't accepted this yet, calling these POSIXP locks is > somewhat of a misnomer. How about this patch instead? > > -----------------------[snip]------------------------ > > [PATCH] locks: make /proc/locks show IS_FILE_PVT locks as type "FLPVT" Sounds like something Bill the Cat would say. ACK.--b. > > In a later patch, we'll be adding a new type of lock that's owned by > the struct file instead of the files_struct. Those sorts of locks > will be flagged with a new FL_FILE_PVT flag. > > Report these types of locks as "FLPVT" in /proc/locks to distinguish > them from "classic" POSIX locks. > > Signed-off-by: Jeff Layton > --- > fs/locks.c | 11 +++++++++-- > include/linux/fs.h | 1 + > 2 files changed, 10 insertions(+), 2 deletions(-) > > diff --git a/fs/locks.c b/fs/locks.c > index 4d4e790150e0..0e1f0df8de12 100644 > --- a/fs/locks.c > +++ b/fs/locks.c > @@ -135,6 +135,7 @@ > #define IS_POSIX(fl) (fl->fl_flags & FL_POSIX) > #define IS_FLOCK(fl) (fl->fl_flags & FL_FLOCK) > #define IS_LEASE(fl) (fl->fl_flags & (FL_LEASE|FL_DELEG)) > +#define IS_FILE_PVT(fl) (fl->fl_flags & FL_FILE_PVT) > > static bool lease_breaking(struct file_lock *fl) > { > @@ -2313,8 +2314,14 @@ static void lock_get_status(struct seq_file *f, struct file_lock *fl, > > seq_printf(f, "%lld:%s ", id, pfx); > if (IS_POSIX(fl)) { > - seq_printf(f, "%6s %s ", > - (fl->fl_flags & FL_ACCESS) ? "ACCESS" : "POSIX ", > + if (fl->fl_flags & FL_ACCESS) > + seq_printf(f, "ACCESS"); > + else if (IS_FILE_PVT(fl)) > + seq_printf(f, "FLPVT "); > + else > + seq_printf(f, "POSIX "); > + > + seq_printf(f, " %s ", > (inode == NULL) ? "*NOINODE*" : > mandatory_lock(inode) ? "MANDATORY" : "ADVISORY "); > } else if (IS_FLOCK(fl)) { > diff --git a/include/linux/fs.h b/include/linux/fs.h > index 7527d96913d3..5ddeb8de5e77 100644 > --- a/include/linux/fs.h > +++ b/include/linux/fs.h > @@ -888,6 +888,7 @@ static inline int file_check_writeable(struct file *filp) > #define FL_SLEEP 128 /* A blocking lock */ > #define FL_DOWNGRADE_PENDING 256 /* Lease is being downgraded */ > #define FL_UNLOCK_PENDING 512 /* Lease is being broken */ > +#define FL_FILE_PVT 1024 /* lock is private to the file */ > > /* > * Special return value from posix_lock_file() and vfs_lock_file() for > -- > 1.9.0 >