[GIT PULL] pv/fbfront

[GIT PULL] pv/fbfront

[Konrad Rzeszutek Wilk]

Hey Jeremy,

Please pull from:

git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git pv/fbfront

It has fixes that made it in v2.6.33 for the Xen Frame buffer:

 xen pvfb: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
 fb-defio: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
 fb-defio: If FBINFO_VIRTFB is defined, do not set VM_IO flag.

 drivers/video/broadsheetfb.c |    2 +-
 drivers/video/fb_defio.c     |    4 +++-
 drivers/video/hecubafb.c     |    2 +-
 drivers/video/metronomefb.c  |    2 +-
 drivers/video/xen-fbfront.c  |    2 +-
 include/linux/fb.h           |    1 +
 6 files changed, 8 insertions(+), 5 deletions(-)

Or perhaps we should ask Greg KH to put those fixes on his stable tree
and then you would merge them from him?

Re: [GIT PULL] pv/fbfront

[Jeremy Fitzhardinge]

On 03/01/2010 07:01 PM, Konrad Rzeszutek Wilk wrote:
> Hey Jeremy,
>
> Please pull from:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git pv/fbfront
>
> It has fixes that made it in v2.6.33 for the Xen Frame buffer:
>
>   xen pvfb: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
>   fb-defio: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
>   fb-defio: If FBINFO_VIRTFB is defined, do not set VM_IO flag.
>
>   drivers/video/broadsheetfb.c |    2 +-
>   drivers/video/fb_defio.c     |    4 +++-
>   drivers/video/hecubafb.c     |    2 +-
>   drivers/video/metronomefb.c  |    2 +-
>   drivers/video/xen-fbfront.c  |    2 +-
>   include/linux/fb.h           |    1 +
>   6 files changed, 8 insertions(+), 5 deletions(-)
>
> Or perhaps we should ask Greg KH to put those fixes on his stable tree
> and then you would merge them from him?
>    

I've merged it anyway.  I'm not sure they're stable material because the 
bug they fix only manifests when there's dom0 support in the kernel, I 
think.  Does plain 2.6.32 show problems with pvfb?

BTW, I can't get absolute (tablet) mode working with pvfb any more.  
Have you noticed that?

     J

Re: Re: [GIT PULL] pv/fbfront

[Konrad Rzeszutek Wilk]

On Tue, Mar 02, 2010 at 12:03:03PM -0800, Jeremy Fitzhardinge wrote:
> On 03/01/2010 07:01 PM, Konrad Rzeszutek Wilk wrote:
>> Hey Jeremy,
>>
>> Please pull from:
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git pv/fbfront
>>
>> It has fixes that made it in v2.6.33 for the Xen Frame buffer:
>>
>>   xen pvfb: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
>>   fb-defio: Inhibit VM_IO flag to be set on vmalloc-ed framebuffers.
>>   fb-defio: If FBINFO_VIRTFB is defined, do not set VM_IO flag.
>>
>>   drivers/video/broadsheetfb.c |    2 +-
>>   drivers/video/fb_defio.c     |    4 +++-
>>   drivers/video/hecubafb.c     |    2 +-
>>   drivers/video/metronomefb.c  |    2 +-
>>   drivers/video/xen-fbfront.c  |    2 +-
>>   include/linux/fb.h           |    1 +
>>   6 files changed, 8 insertions(+), 5 deletions(-)
>>
>> Or perhaps we should ask Greg KH to put those fixes on his stable tree
>> and then you would merge them from him?
>>    
>
> I've merged it anyway.  I'm not sure they're stable material because the  
> bug they fix only manifests when there's dom0 support in the kernel, I  
> think.  Does plain 2.6.32 show problems with pvfb?

No. But xen/stable does. Without those patches virgin xen/stable boots,
but on Dom0 console I see:

(XEN) mm.c:859:d1008 Error getting mfn 1c1f5 (pfn 5555555555555555) from
L1 entry 800000001c1f5467 for l1e_owner=1008, pg_owner=3273

these forever.
>
> BTW, I can't get absolute (tablet) mode working with pvfb any more.   
> Have you noticed that?

Noted. Haven't played with that - I just boot up tiny guests (20MB or
so) that can simple stuff. I will look in this.

Re: Re: [GIT PULL] pv/fbfront

[Jeremy Fitzhardinge]

On 03/02/2010 01:56 PM, Konrad Rzeszutek Wilk wrote:
> No. But xen/stable does. Without those patches virgin xen/stable boots,
> but on Dom0 console I see:
>
> (XEN) mm.c:859:d1008 Error getting mfn 1c1f5 (pfn 5555555555555555) from
> L1 entry 800000001c1f5467 for l1e_owner=1008, pg_owner=3273
>
> these forever.
>    

I haven't seen that, but I've merged xen/next into stable now, so it 
should have picked up those changes.

>> BTW, I can't get absolute (tablet) mode working with pvfb any more.
>> Have you noticed that?
>>      
> Noted. Haven't played with that - I just boot up tiny guests (20MB or
> so) that can simple stuff. I will look in this.
>    

I've been running 512MB domains with (fairly) clean F12 installs in them 
(32 and 64b, pv and hvm).

     J