diff options
Diffstat (limited to 'fs/fcntl.c')
| -rw-r--r-- | fs/fcntl.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/fs/fcntl.c b/fs/fcntl.c index d865ca66ccba..cc8e4de2fee5 100644 --- a/fs/fcntl.c +++ b/fs/fcntl.c | |||
| @@ -531,6 +531,12 @@ int fasync_helper(int fd, struct file * filp, int on, struct fasync_struct **fap | |||
| 531 | if (!new) | 531 | if (!new) |
| 532 | return -ENOMEM; | 532 | return -ENOMEM; |
| 533 | } | 533 | } |
| 534 | |||
| 535 | /* | ||
| 536 | * We need to take f_lock first since it's not an IRQ-safe | ||
| 537 | * lock. | ||
| 538 | */ | ||
| 539 | spin_lock(&filp->f_lock); | ||
| 534 | write_lock_irq(&fasync_lock); | 540 | write_lock_irq(&fasync_lock); |
| 535 | for (fp = fapp; (fa = *fp) != NULL; fp = &fa->fa_next) { | 541 | for (fp = fapp; (fa = *fp) != NULL; fp = &fa->fa_next) { |
| 536 | if (fa->fa_file == filp) { | 542 | if (fa->fa_file == filp) { |
| @@ -555,14 +561,12 @@ int fasync_helper(int fd, struct file * filp, int on, struct fasync_struct **fap | |||
| 555 | result = 1; | 561 | result = 1; |
| 556 | } | 562 | } |
| 557 | out: | 563 | out: |
| 558 | /* Fix up FASYNC bit while still holding fasync_lock */ | ||
| 559 | spin_lock(&filp->f_lock); | ||
| 560 | if (on) | 564 | if (on) |
| 561 | filp->f_flags |= FASYNC; | 565 | filp->f_flags |= FASYNC; |
| 562 | else | 566 | else |
| 563 | filp->f_flags &= ~FASYNC; | 567 | filp->f_flags &= ~FASYNC; |
| 564 | spin_unlock(&filp->f_lock); | ||
| 565 | write_unlock_irq(&fasync_lock); | 568 | write_unlock_irq(&fasync_lock); |
| 569 | spin_unlock(&filp->f_lock); | ||
| 566 | return result; | 570 | return result; |
| 567 | } | 571 | } |
| 568 | 572 | ||
