diff options
| author | Linus Torvalds <[email protected]> | 2009-03-19 18:32:05 +0000 |
|---|---|---|
| committer | Linus Torvalds <[email protected]> | 2009-03-19 18:32:05 +0000 |
| commit | a8e7d49aa7be728c4ae241a75a2a124cdcabc0c5 (patch) | |
| tree | fec49351496b886d8aa0e7b55e58c2f1394b051a /net/unix/af_unix.c | |
| parent | Add '-fwrapv' to gcc CFLAGS (diff) | |
| download | kernel-a8e7d49aa7be728c4ae241a75a2a124cdcabc0c5.tar.gz kernel-a8e7d49aa7be728c4ae241a75a2a124cdcabc0c5.zip | |
Fix race in create_empty_buffers() vs __set_page_dirty_buffers()
Nick Piggin noticed this (very unlikely) race between setting a page
dirty and creating the buffers for it - we need to hold the mapping
private_lock until we've set the page dirty bit in order to make sure
that create_empty_buffers() might not build up a set of buffers without
the dirty bits set when the page is dirty.
I doubt anybody has ever hit this race (and it didn't solve the issue
Nick was looking at), but as Nick says: "Still, it does appear to solve
a real race, which we should close."
Acked-by: Nick Piggin <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'net/unix/af_unix.c')
0 files changed, 0 insertions, 0 deletions