LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: ira.weiny@intel.com
To: lsf-pc@lists.linux-foundation.org
Cc: linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-mm@kvack.org, "Dan Williams" <dan.j.williams@intel.com>,
"Jan Kara" <jack@suse.cz>, "Jérôme Glisse" <jglisse@redhat.com>,
"John Hubbard" <jhubbard@nvidia.com>,
"Michal Hocko" <mhocko@suse.com>,
"Ira Weiny" <ira.weiny@intel.com>
Subject: [RFC PATCH 06/10] fs/locks: Add longterm lease traces
Date: Sun, 28 Apr 2019 21:53:55 -0700 [thread overview]
Message-ID: <20190429045359.8923-7-ira.weiny@intel.com> (raw)
In-Reply-To: <20190429045359.8923-1-ira.weiny@intel.com>
From: Ira Weiny <ira.weiny@intel.com>
---
fs/locks.c | 5 +++++
include/trace/events/filelock.h | 37 ++++++++++++++++++++++++++++++++-
2 files changed, 41 insertions(+), 1 deletion(-)
diff --git a/fs/locks.c b/fs/locks.c
index ae508d192223..58c6d7a411b6 100644
--- a/fs/locks.c
+++ b/fs/locks.c
@@ -2136,6 +2136,8 @@ static int do_fcntl_add_lease(unsigned int fd, struct file *filp, long arg)
}
new->fa_fd = fd;
+ trace_take_longterm_lease(fl);
+
error = vfs_setlease(filp, arg, &fl, (void **)&new);
if (fl)
locks_free_lock(fl);
@@ -3118,6 +3120,8 @@ bool page_set_longterm_lease(struct page *page)
kref_get(&existing_fl->gup_ref);
}
+ trace_take_longterm_lease(existing_fl);
+
spin_unlock(&ctx->flc_lock);
percpu_up_read(&file_rwsem);
@@ -3153,6 +3157,7 @@ void page_remove_longterm_lease(struct page *page)
percpu_down_read(&file_rwsem);
spin_lock(&ctx->flc_lock);
found = find_longterm_lease(inode);
+ trace_release_longterm_lease(found);
if (found)
kref_put(&found->gup_ref, release_longterm_lease);
spin_unlock(&ctx->flc_lock);
diff --git a/include/trace/events/filelock.h b/include/trace/events/filelock.h
index 4b735923f2ff..c6f39f03cb8b 100644
--- a/include/trace/events/filelock.h
+++ b/include/trace/events/filelock.h
@@ -27,7 +27,8 @@
{ FL_SLEEP, "FL_SLEEP" }, \
{ FL_DOWNGRADE_PENDING, "FL_DOWNGRADE_PENDING" }, \
{ FL_UNLOCK_PENDING, "FL_UNLOCK_PENDING" }, \
- { FL_OFDLCK, "FL_OFDLCK" })
+ { FL_OFDLCK, "FL_OFDLCK" }, \
+ { FL_LONGTERM, "FL_LONGTERM" })
#define show_fl_type(val) \
__print_symbolic(val, \
@@ -238,6 +239,40 @@ TRACE_EVENT(leases_conflict,
show_fl_type(__entry->b_fl_type))
);
+DECLARE_EVENT_CLASS(longterm_lease,
+ TP_PROTO(struct file_lock *fl),
+
+ TP_ARGS(fl),
+
+ TP_STRUCT__entry(
+ __field(void *, fl)
+ __field(void *, owner)
+ __field(unsigned int, fl_flags)
+ __field(unsigned int, cnt)
+ __field(unsigned char, fl_type)
+ ),
+
+ TP_fast_assign(
+ __entry->fl = fl;
+ __entry->owner = fl ? fl->fl_owner : NULL;
+ __entry->fl_flags = fl ? fl->fl_flags : 0;
+ __entry->cnt = fl ? kref_read(&fl->gup_ref) : 0;
+ __entry->fl_type = fl ? fl->fl_type : 0;
+ ),
+
+ TP_printk("owner=0x%p fl=%p(%d) fl_flags=%s fl_type=%s",
+ __entry->owner, __entry->fl, __entry->cnt,
+ show_fl_flags(__entry->fl_flags),
+ show_fl_type(__entry->fl_type))
+);
+DEFINE_EVENT(longterm_lease, take_longterm_lease,
+ TP_PROTO(struct file_lock *fl),
+ TP_ARGS(fl));
+DEFINE_EVENT(longterm_lease, release_longterm_lease,
+ TP_PROTO(struct file_lock *fl),
+ TP_ARGS(fl));
+
+
#endif /* _TRACE_FILELOCK_H */
/* This part must be outside protection */
--
2.20.1
next prev parent reply other threads:[~2019-04-29 4:55 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-29 4:53 [RFC PATCH 00/10] RDMA/FS DAX "LONGTERM" lease proposal ira.weiny
2019-04-29 4:53 ` [RFC PATCH 01/10] fs/locks: Add trace_leases_conflict ira.weiny
2019-04-29 4:53 ` [RFC PATCH 02/10] fs/locks: Introduce FL_LONGTERM file lease ira.weiny
2019-04-29 4:53 ` [RFC PATCH 03/10] mm/gup: Pass flags down to __gup_device_huge* calls ira.weiny
2019-04-29 4:53 ` [RFC PATCH 04/10] WIP: mm/gup: Ensure F_LONGTERM lease is held on GUP pages ira.weiny
2019-04-29 4:53 ` [RFC PATCH 05/10] mm/gup: Take FL_LONGTERM lease if not set by user ira.weiny
2019-04-29 4:53 ` ira.weiny [this message]
2019-04-29 4:53 ` [RFC PATCH 07/10] fs/dax: Create function dax_mapping_is_dax() ira.weiny
2019-04-29 4:53 ` [RFC PATCH 08/10] mm/gup: fs: Send SIGBUS on truncate of active file ira.weiny
2019-04-29 4:53 ` [RFC PATCH 09/10] fs/locks: Add tracepoint for SIGBUS on LONGTERM expiration ira.weiny
2019-04-29 4:53 ` [RFC PATCH 10/10] mm/gup: Remove FOLL_LONGTERM DAX exclusion ira.weiny
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190429045359.8923-7-ira.weiny@intel.com \
--to=ira.weiny@intel.com \
--cc=dan.j.williams@intel.com \
--cc=jack@suse.cz \
--cc=jglisse@redhat.com \
--cc=jhubbard@nvidia.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lsf-pc@lists.linux-foundation.org \
--cc=mhocko@suse.com \
--subject='Re: [RFC PATCH 06/10] fs/locks: Add longterm lease traces' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).