fb-defio: If FBINFO_VIRTFB is defined, do not set VM_IO flag.
authorKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Thu, 3 Dec 2009 15:31:56 +0000 (10:31 -0500)
committerJeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Fri, 4 Dec 2009 01:30:02 +0000 (17:30 -0800)
Most users (except sh_mobile_lcdcfb.c) get their framebuffer from
vmalloc. Setting VM_IO is not necessary as the memory obtained
from vmalloc is System RAM type and is not susceptible to PCI memory
constraints.

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Acked-by: Jaya Kumar <jayakumar.lkml@gmail.com>
drivers/video/fb_defio.c
include/linux/fb.h

index 0a7a6679ee6eb05e2be754ff816ba3547a0f7fdb..875d019bb2a48ae642af8495d131003d90b60139 100644 (file)
@@ -144,7 +144,9 @@ static const struct address_space_operations fb_deferred_io_aops = {
 static int fb_deferred_io_mmap(struct fb_info *info, struct vm_area_struct *vma)
 {
        vma->vm_ops = &fb_deferred_io_vm_ops;
-       vma->vm_flags |= ( VM_IO | VM_RESERVED | VM_DONTEXPAND );
+       vma->vm_flags |= ( VM_RESERVED | VM_DONTEXPAND );
+       if (!(info->flags & FBINFO_VIRTFB))
+               vma->vm_flags |= VM_IO;
        vma->vm_private_data = info;
        return 0;
 }
index f847df9e99b6f4fcd44991e757c1d5b7ca50681a..6e8ebf7e6523cb092f2449270df9dc5891d3fd31 100644 (file)
@@ -768,6 +768,7 @@ struct fb_tile_ops {
         *  takes over; acceleration engine should be in a quiescent state */
 
 /* hints */
+#define FBINFO_VIRTFB          0x0004 /* FB is System RAM, not device. */
 #define FBINFO_PARTIAL_PAN_OK  0x0040 /* otw use pan only for double-buffering */
 #define FBINFO_READS_FAST      0x0080 /* soft-copy faster than rendering */