Re: [Solved] Nouveau on dom0

[Jeremy Fitzhardinge <jeremy@goop.org>]

On 03/10/2010 04:50 AM, Arvind R wrote:
> On Mon, Mar 8, 2010 at 11:21 PM, Konrad Rzeszutek Wilk
> <konrad.wilk@oracle.com>  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@gmail.com>  wrote:
>>>        
>>>> On Sat, Mar 6, 2010 at 1:46 PM, Arvind R<arvino55@gmail.com>  wrote:
>>>>          
>>>>> On Sat, Mar 6, 2010 at 1:53 AM, Konrad Rzeszutek Wilk
>>>>> <konrad.wilk@oracle.com>  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@oracle.com>  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@oracle.com>  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