diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-27 21:17:02 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2010-10-27 21:17:02 -0400 |
commit | 55f335a8857db2ee22c068e7ab7141fc79928296 (patch) | |
tree | 7e04667573f87a6970b6804940ef8fcb4bb6ca3b /fs/fcntl.c | |
parent | 7420a8c0de8d99b201aeeab6fed16ca95ebf55a5 (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.c | 6 |
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 | */ |
688 | struct fasync_struct *fasync_insert_entry(int fd, struct file *filp, struct fasync_struct **fapp, struct fasync_struct *new) | 691 | struct 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 | */ |
725 | static int fasync_add_entry(int fd, struct file *filp, struct fasync_struct **fapp) | 725 | static int fasync_add_entry(int fd, struct file *filp, struct fasync_struct **fapp) |
726 | { | 726 | { |