xen pvfb: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
authorKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Thu, 3 Dec 2009 15:31:57 +0000 (10:31 -0500)
committerJeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Fri, 4 Dec 2009 01:30:47 +0000 (17:30 -0800)
In Xen-paravirt mode, VM_IO flag signifies that the page frame number (PFN)
is actually a machine frame number (MFN). This is correct for memory backed by
PCI devices, but wrong for memory allocated from System RAM where the PFN
!= MFN.

During page faults, pages with VM_IO, get assigned to special domain I/O
domain and as said, the PFN is interpreted as MFN. When Xen hypervisor
modifies the PTE it interprets the PFN as the MFN, complains and
fails the PTE modification.

The end result is an infinitive page fault in the domain.

Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
drivers/video/xen-fbfront.c

index 54cd916101744ff7c5c1c21d68d4d4f6b028f066..91a68e9eb66d7ff7ad8fd866d1266bd2ebf6dd10 100644 (file)
@@ -440,7 +440,7 @@ static int __devinit xenfb_probe(struct xenbus_device *dev,
        fb_info->fix.type = FB_TYPE_PACKED_PIXELS;
        fb_info->fix.accel = FB_ACCEL_NONE;
 
-       fb_info->flags = FBINFO_FLAG_DEFAULT;
+       fb_info->flags = FBINFO_FLAG_DEFAULT | FBINFO_VIRTFB;
 
        ret = fb_alloc_cmap(&fb_info->cmap, 256, 0);
        if (ret < 0) {