diff options
| author | Davide Libenzi <davidel@xmailserver.org> | 2009-03-19 01:04:19 +0100 |
|---|---|---|
| committer | Linus Torvalds <torvalds@linux-foundation.org> | 2009-03-19 23:57:18 +0100 |
| commit | 87c3a86e1c220121d0ced59d1a71e78ed9abc6dd (patch) | |
| tree | f0460d0a5f34da0c3df3cc729b705fc93a69be4b /mm/pdflush.c | |
| parent | Move cc-option to below arch-specific setup (diff) | |
| download | linux-87c3a86e1c220121d0ced59d1a71e78ed9abc6dd.tar.xz linux-87c3a86e1c220121d0ced59d1a71e78ed9abc6dd.zip | |
eventfd: remove fput() call from possible IRQ context
Remove a source of fput() call from inside IRQ context. Myself, like Eric,
wasn't able to reproduce an fput() call from IRQ context, but Jeff said he was
able to, with the attached test program. Independently from this, the bug is
conceptually there, so we might be better off fixing it. This patch adds an
optimization similar to the one we already do on ->ki_filp, on ->ki_eventfd.
Playing with ->f_count directly is not pretty in general, but the alternative
here would be to add a brand new delayed fput() infrastructure, that I'm not
sure is worth it.
Signed-off-by: Davide Libenzi <davidel@xmailserver.org>
Cc: Benjamin LaHaise <bcrl@kvack.org>
Cc: Trond Myklebust <trond.myklebust@fys.uio.no>
Cc: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: Jeff Moyer <jmoyer@redhat.com>
Cc: Zach Brown <zach.brown@oracle.com>
Cc: <stable@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm/pdflush.c')
0 files changed, 0 insertions, 0 deletions