diff options
author | Jaya Kumar | 2008-07-12 22:47:51 +0200 |
---|---|---|
committer | Linus Torvalds | 2008-07-12 23:33:41 +0200 |
commit | f31ad92f34913043cf008d6e479e92dfbaf02df1 (patch) | |
tree | 5f57faa359a060c56bddac6f22c2993e2309d911 /drivers/video/q40fb.c | |
parent | drivers/isdn/i4l/isdn_common.c fix small resource leak (diff) | |
download | kernel-qcow2-linux-f31ad92f34913043cf008d6e479e92dfbaf02df1.tar.gz kernel-qcow2-linux-f31ad92f34913043cf008d6e479e92dfbaf02df1.tar.xz kernel-qcow2-linux-f31ad92f34913043cf008d6e479e92dfbaf02df1.zip |
fbdev: bugfix for multiprocess defio
This patch is a bugfix for how defio handles multiple processes manipulating
the same framebuffer.
Thanks to Bernard Blackham for identifying this bug.
It occurs when two applications mmap the same framebuffer and concurrently
write to the same page. Normally, this doesn't occur since only a single
process mmaps the framebuffer. The symptom of the bug is that the mapping
applications will hang. The cause is that defio incorrectly tries to add the
same page twice to the pagelist. The solution I have is to walk the pagelist
and check for a duplicate before adding. Since I needed to walk the pagelist,
I now also keep the pagelist in sorted order.
Signed-off-by: Jaya Kumar <jayakumar.lkml@gmail.com>
Cc: Bernard Blackham <bernard@largestprime.net>
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 'drivers/video/q40fb.c')
0 files changed, 0 insertions, 0 deletions