Re: [Xen-devel] [Solved] Nouveau on dom0

[Jeremy Fitzhardinge <jeremy@xxxxxxxx>]

On 03/10/2010 04:50 AM, Arvind R wrote:
> On Mon, Mar 8, 2010 at 11:21 PM, Konrad Rzeszutek Wilk
> <konrad.wilk@xxxxxxxxxx>  wrote:
>    
> > On Sun, Mar 07, 2010 at 05:26:12AM +0530, Arvind R wrote:
> >      
> > > On Sun, Mar 7, 2010 at 2:29 AM, Arvind R<arvino55@xxxxxxxxx>  wrote:
> > >        
> > > > On Sat, Mar 6, 2010 at 1:46 PM, Arvind R<arvino55@xxxxxxxxx>  wrote:
> > > >          
> > > > > On Sat, Mar 6, 2010 at 1:53 AM, Konrad Rzeszutek Wilk
> > > > > <konrad.wilk@xxxxxxxxxx>  wrote:
> > > > >            
> > > > > > On Fri, Mar 05, 2010 at 01:16:13PM +0530, Arvind R wrote:
> > > > > >              
> > > > > > > On Thu, Mar 4, 2010 at 11:55 PM, Konrad Rzeszutek Wilk
> > > > > > > <konrad.wilk@xxxxxxxxxx>  wrote:
> > > > > > >                
> > > > > > > > On Thu, Mar 04, 2010 at 02:47:58PM +0530, Arvind R wrote:
> > > > > > > >                  
> > > > > > > > > On Wed, Mar 3, 2010 at 11:43 PM, Konrad Rzeszutek Wilk
> > > > > > > > > <konrad.wilk@xxxxxxxxxx>  wrote:
> > > > > > > > >                    
> > >        
> > > > > > (FYI, look at
> > > > > > http://git.kernel.org/?p=linux/kernel/git/konrad/xen.git;a=commit;h=e84db8b7136d1b4a393dbd982201d0c5a3794333)
> > > > > >              
> > > THAT SOLVED THE FAULTING; OUT_RING now completes under Xen.
> > >        
> > That is great! Thanks for doing all the hard-work in digging through the
> > code.
> >
> >
> > So this means you got graphics on the screen? Or at least that Kernel
> > Mode Setting and the DRM parts show fancy graphics during boot?
> >      
> AT LAST, yes! Patch: (after aboout 600 reboots!)
>
> diff -Naur nouveau-kernel.orig/drivers/gpu/drm/ttm/ttm_bo_vm.c
> nouveau-kernel.new/drivers/gpu/drm/ttm/ttm_bo_vm.c
> --- nouveau-kernel.orig/drivers/gpu/drm/ttm/ttm_bo_vm.c 2010-01-27
> 10:19:28.000000000 +0530
> +++ nouveau-kernel.new/drivers/gpu/drm/ttm/ttm_bo_vm.c  2010-03-10
> 17:28:59.000000000 +0530
> @@ -271,7 +271,10 @@
>           */
>
>          vma->vm_private_data = bo;
> -       vma->vm_flags |= VM_RESERVED | VM_IO | VM_MIXEDMAP | VM_DONTEXPAND;
> +       vma->vm_flags |= VM_RESERVED | VM_MIXEDMAP | VM_DONTEXPAND;
> +       if (!((bo->mem.placement&  TTM_PL_MASK_MEM)&  TTM_PL_FLAG_TT))
> +               vma->vm_flags |= VM_IO;
> +       vma->vm_page_prot = vma_get_vm_prot(vma->vm_flags);
>          return 0;
>   out_unref:
>          ttm_bo_unref(&bo);
>
>    
Cool, nice and simple.  Can you write it up as a proper patch for 
submission to upstream?
Thanks,
    J

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel