aboutsummaryrefslogtreecommitdiffstats
path: root/fs/fcntl.c
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2010-10-27 21:17:02 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2010-10-27 21:17:02 -0400
commit55f335a8857db2ee22c068e7ab7141fc79928296 (patch)
tree7e04667573f87a6970b6804940ef8fcb4bb6ca3b /fs/fcntl.c
parent7420a8c0de8d99b201aeeab6fed16ca95ebf55a5 (diff)
fasync: Fix placement of FASYNC flag comment
In commit f7347ce4ee7c ("fasync: re-organize fasync entry insertion to allow it under a spinlock") Arnd took an earlier patch of mine that had the comment about the FASYNC flag above the wrong function. When the fasync_add_entry() function was split to introduce the new fasync_insert_entry() helper function, the code that actually cares about the FASYNC bit moved to that new helper. So just move the comment to the right point. Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/fcntl.c')
-rw-r--r--fs/fcntl.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/fcntl.c b/fs/fcntl.c
index dcdbc6f5c33b..ecc8b3954ed6 100644
--- a/fs/fcntl.c
+++ b/fs/fcntl.c
@@ -684,6 +684,9 @@ void fasync_free(struct fasync_struct *new)
684/* 684/*
685 * Insert a new entry into the fasync list. Return the pointer to the 685 * Insert a new entry into the fasync list. Return the pointer to the
686 * old one if we didn't use the new one. 686 * old one if we didn't use the new one.
687 *
688 * NOTE! It is very important that the FASYNC flag always
689 * match the state "is the filp on a fasync list".
687 */ 690 */
688struct fasync_struct *fasync_insert_entry(int fd, struct file *filp, struct fasync_struct **fapp, struct fasync_struct *new) 691struct fasync_struct *fasync_insert_entry(int fd, struct file *filp, struct fasync_struct **fapp, struct fasync_struct *new)
689{ 692{
@@ -718,9 +721,6 @@ out:
718/* 721/*
719 * Add a fasync entry. Return negative on error, positive if 722 * Add a fasync entry. Return negative on error, positive if
720 * added, and zero if did nothing but change an existing one. 723 * added, and zero if did nothing but change an existing one.
721 *
722 * NOTE! It is very important that the FASYNC flag always
723 * match the state "is the filp on a fasync list".
724 */ 724 */
725static int fasync_add_entry(int fd, struct file *filp, struct fasync_struct **fapp) 725static int fasync_add_entry(int fd, struct file *filp, struct fasync_struct **fapp)
726{ 726{