diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-07-27 16:18:00 -0400 |
---|---|---|
committer | J. Bruce Fields <bfields@redhat.com> | 2012-07-27 16:18:00 -0400 |
commit | 96d6d59ceaeaacba4088862f3c57fcd011f52832 (patch) | |
tree | fee86db726235ee3aeb0efdc7ae790f299d6de9d | |
parent | 3b6e2723f32de42028617f2c99b244ccd72cd959 (diff) | |
download | op-kernel-dev-96d6d59ceaeaacba4088862f3c57fcd011f52832.zip op-kernel-dev-96d6d59ceaeaacba4088862f3c57fcd011f52832.tar.gz |
locks: move lease-specific code out of locks_delete_lock
No point putting something only used by one caller into common code.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
-rw-r--r-- | fs/locks.c | 11 |
1 files changed, 5 insertions, 6 deletions
@@ -570,12 +570,6 @@ static void locks_delete_lock(struct file_lock **thisfl_p) fl->fl_next = NULL; list_del_init(&fl->fl_link); - fasync_helper(0, fl->fl_file, 0, &fl->fl_fasync); - if (fl->fl_fasync != NULL) { - printk(KERN_ERR "locks_delete_lock: fasync == %p\n", fl->fl_fasync); - fl->fl_fasync = NULL; - } - if (fl->fl_nspid) { put_pid(fl->fl_nspid); fl->fl_nspid = NULL; @@ -1150,6 +1144,11 @@ int lease_modify(struct file_lock **before, int arg) f_delown(filp); filp->f_owner.signum = 0; + fasync_helper(0, fl->fl_file, 0, &fl->fl_fasync); + if (fl->fl_fasync != NULL) { + printk(KERN_ERR "locks_delete_lock: fasync == %p\n", fl->fl_fasync); + fl->fl_fasync = NULL; + } locks_delete_lock(before); } return 0; |