diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-08-01 07:56:16 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-08-01 12:01:46 -0400 |
commit | 068535f1fef4c90aee23eb7b9b9a71c5b72d7cd0 (patch) | |
tree | ca71858417d881bc8f72f4a2a7b035d041f7a43c | |
parent | 2d534926205db9ffce4bbbde67cb9b2cee4b835c (diff) |
locks: remove unused lm_release_private
In commit 3b6e2723f32d ("locks: prevent side-effects of
locks_release_private before file_lock is initialized") we removed the
last user of lm_release_private without removing the field itself.
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | Documentation/filesystems/Locking | 2 | ||||
-rw-r--r-- | fs/locks.c | 6 | ||||
-rw-r--r-- | include/linux/fs.h | 1 |
3 files changed, 1 insertions, 8 deletions
diff --git a/Documentation/filesystems/Locking b/Documentation/filesystems/Locking index 2db1900d753..7f647e17830 100644 --- a/Documentation/filesystems/Locking +++ b/Documentation/filesystems/Locking | |||
@@ -359,7 +359,6 @@ prototypes: | |||
359 | int (*lm_compare_owner)(struct file_lock *, struct file_lock *); | 359 | int (*lm_compare_owner)(struct file_lock *, struct file_lock *); |
360 | void (*lm_notify)(struct file_lock *); /* unblock callback */ | 360 | void (*lm_notify)(struct file_lock *); /* unblock callback */ |
361 | int (*lm_grant)(struct file_lock *, struct file_lock *, int); | 361 | int (*lm_grant)(struct file_lock *, struct file_lock *, int); |
362 | void (*lm_release_private)(struct file_lock *); | ||
363 | void (*lm_break)(struct file_lock *); /* break_lease callback */ | 362 | void (*lm_break)(struct file_lock *); /* break_lease callback */ |
364 | int (*lm_change)(struct file_lock **, int); | 363 | int (*lm_change)(struct file_lock **, int); |
365 | 364 | ||
@@ -368,7 +367,6 @@ locking rules: | |||
368 | lm_compare_owner: yes no | 367 | lm_compare_owner: yes no |
369 | lm_notify: yes no | 368 | lm_notify: yes no |
370 | lm_grant: no no | 369 | lm_grant: no no |
371 | lm_release_private: maybe no | ||
372 | lm_break: yes no | 370 | lm_break: yes no |
373 | lm_change yes no | 371 | lm_change yes no |
374 | 372 | ||
diff --git a/fs/locks.c b/fs/locks.c index cdcf219a739..7e81bfc7516 100644 --- a/fs/locks.c +++ b/fs/locks.c | |||
@@ -200,11 +200,7 @@ void locks_release_private(struct file_lock *fl) | |||
200 | fl->fl_ops->fl_release_private(fl); | 200 | fl->fl_ops->fl_release_private(fl); |
201 | fl->fl_ops = NULL; | 201 | fl->fl_ops = NULL; |
202 | } | 202 | } |
203 | if (fl->fl_lmops) { | 203 | fl->fl_lmops = NULL; |
204 | if (fl->fl_lmops->lm_release_private) | ||
205 | fl->fl_lmops->lm_release_private(fl); | ||
206 | fl->fl_lmops = NULL; | ||
207 | } | ||
208 | 204 | ||
209 | } | 205 | } |
210 | EXPORT_SYMBOL_GPL(locks_release_private); | 206 | EXPORT_SYMBOL_GPL(locks_release_private); |
diff --git a/include/linux/fs.h b/include/linux/fs.h index d7eed5b98ae..4ba5c871552 100644 --- a/include/linux/fs.h +++ b/include/linux/fs.h | |||
@@ -1162,7 +1162,6 @@ struct lock_manager_operations { | |||
1162 | int (*lm_compare_owner)(struct file_lock *, struct file_lock *); | 1162 | int (*lm_compare_owner)(struct file_lock *, struct file_lock *); |
1163 | void (*lm_notify)(struct file_lock *); /* unblock callback */ | 1163 | void (*lm_notify)(struct file_lock *); /* unblock callback */ |
1164 | int (*lm_grant)(struct file_lock *, struct file_lock *, int); | 1164 | int (*lm_grant)(struct file_lock *, struct file_lock *, int); |
1165 | void (*lm_release_private)(struct file_lock *); | ||
1166 | void (*lm_break)(struct file_lock *); | 1165 | void (*lm_break)(struct file_lock *); |
1167 | int (*lm_change)(struct file_lock **, int); | 1166 | int (*lm_change)(struct file_lock **, int); |
1168 | }; | 1167 | }; |