diff options
author | Christoph Hellwig <hch@infradead.org> | 2010-10-31 08:35:31 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-31 09:35:15 -0400 |
commit | bb8430a2c8fe2b726033017daadf73c69b0348ea (patch) | |
tree | feb6063cab812816bc67bca4e509995b5ad9217d | |
parent | 51ee4b84f5c86935b438d6636f34b523edb415a8 (diff) |
locks: remove fl_copy_lock lock_manager operation
This one was only used for a nasty hack in nfsd, which has recently
been removed.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | Documentation/filesystems/Locking | 2 | ||||
-rw-r--r-- | fs/locks.c | 5 | ||||
-rw-r--r-- | include/linux/fs.h | 1 |
3 files changed, 1 insertions, 7 deletions
diff --git a/Documentation/filesystems/Locking b/Documentation/filesystems/Locking index 8a817f656f0..a91f3089001 100644 --- a/Documentation/filesystems/Locking +++ b/Documentation/filesystems/Locking | |||
@@ -322,7 +322,6 @@ fl_release_private: yes yes | |||
322 | prototypes: | 322 | prototypes: |
323 | int (*fl_compare_owner)(struct file_lock *, struct file_lock *); | 323 | int (*fl_compare_owner)(struct file_lock *, struct file_lock *); |
324 | void (*fl_notify)(struct file_lock *); /* unblock callback */ | 324 | void (*fl_notify)(struct file_lock *); /* unblock callback */ |
325 | void (*fl_copy_lock)(struct file_lock *, struct file_lock *); | ||
326 | void (*fl_release_private)(struct file_lock *); | 325 | void (*fl_release_private)(struct file_lock *); |
327 | void (*fl_break)(struct file_lock *); /* break_lease callback */ | 326 | void (*fl_break)(struct file_lock *); /* break_lease callback */ |
328 | 327 | ||
@@ -330,7 +329,6 @@ locking rules: | |||
330 | BKL may block | 329 | BKL may block |
331 | fl_compare_owner: yes no | 330 | fl_compare_owner: yes no |
332 | fl_notify: yes no | 331 | fl_notify: yes no |
333 | fl_copy_lock: yes no | ||
334 | fl_release_private: yes yes | 332 | fl_release_private: yes yes |
335 | fl_break: yes no | 333 | fl_break: yes no |
336 | 334 | ||
diff --git a/fs/locks.c b/fs/locks.c index a2ab790471b..65765cb6afe 100644 --- a/fs/locks.c +++ b/fs/locks.c | |||
@@ -235,11 +235,8 @@ static void locks_copy_private(struct file_lock *new, struct file_lock *fl) | |||
235 | fl->fl_ops->fl_copy_lock(new, fl); | 235 | fl->fl_ops->fl_copy_lock(new, fl); |
236 | new->fl_ops = fl->fl_ops; | 236 | new->fl_ops = fl->fl_ops; |
237 | } | 237 | } |
238 | if (fl->fl_lmops) { | 238 | if (fl->fl_lmops) |
239 | if (fl->fl_lmops->fl_copy_lock) | ||
240 | fl->fl_lmops->fl_copy_lock(new, fl); | ||
241 | new->fl_lmops = fl->fl_lmops; | 239 | new->fl_lmops = fl->fl_lmops; |
242 | } | ||
243 | } | 240 | } |
244 | 241 | ||
245 | /* | 242 | /* |
diff --git a/include/linux/fs.h b/include/linux/fs.h index 1eb29399a4f..334d68a1710 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h | |||
@@ -1056,7 +1056,6 @@ struct lock_manager_operations { | |||
1056 | int (*fl_compare_owner)(struct file_lock *, struct file_lock *); | 1056 | int (*fl_compare_owner)(struct file_lock *, struct file_lock *); |
1057 | void (*fl_notify)(struct file_lock *); /* unblock callback */ | 1057 | void (*fl_notify)(struct file_lock *); /* unblock callback */ |
1058 | int (*fl_grant)(struct file_lock *, struct file_lock *, int); | 1058 | int (*fl_grant)(struct file_lock *, struct file_lock *, int); |
1059 | void (*fl_copy_lock)(struct file_lock *, struct file_lock *); | ||
1060 | void (*fl_release_private)(struct file_lock *); | 1059 | void (*fl_release_private)(struct file_lock *); |
1061 | void (*fl_break)(struct file_lock *); | 1060 | void (*fl_break)(struct file_lock *); |
1062 | int (*fl_mylease)(struct file_lock *, struct file_lock *); | 1061 | int (*fl_mylease)(struct file_lock *, struct file_lock *); |