diff options
author | Davide Libenzi <[email protected]> | 2009-03-18 17:04:19 -0700 |
---|---|---|
committer | Linus Torvalds <[email protected]> | 2009-03-19 15:57:18 -0700 |
commit | 87c3a86e1c220121d0ced59d1a71e78ed9abc6dd (patch) | |
tree | f0460d0a5f34da0c3df3cc729b705fc93a69be4b /fs/jbd2/commit.c | |
parent | d0115552cdb0b4d4146975889fee2e9355515c4b (diff) |
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 <[email protected]>
Cc: Benjamin LaHaise <[email protected]>
Cc: Trond Myklebust <[email protected]>
Cc: Eric Dumazet <[email protected]>
Signed-off-by: Jeff Moyer <[email protected]>
Cc: Zach Brown <[email protected]>
Cc: <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'fs/jbd2/commit.c')
0 files changed, 0 insertions, 0 deletions