Re: ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator V5

[Thomas Hellstrom <thellstrom@vmware.com>]

On 11/14/2011 07:54 PM, Jerome Glisse wrote:
> On Mon, Nov 14, 2011 at 05:06:42PM +0100, Thomas Hellstrom wrote:
>    
>> On 11/14/2011 05:02 PM, Jerome Glisse wrote:
>>      
>>> On Mon, Nov 14, 2011 at 9:49 AM, Thomas Hellstrom<thellstrom@vmware.com>   wrote:
>>>        
>>>> On 11/11/2011 11:47 PM, j.glisse@gmail.com wrote:
>>>>          
>>>>> So attached is updated serie of patch with fixes for swaping issue
>>>>> that also impacted memory accounting.
>>>>>            
>>>> Jerome,
>>>> Out of interest, what was the swapping issue?
>>>>
>>>> /Thomas
>>>>
>>>>
>>>>          
>>> The page list was NULL and some access ended up freaking the kernel
>>> and oom kicked in for some reason i still don't understand. Bottom
>>> line things were going south in a split second leaving me with no log
>>> or access, i thought it was accounting not working as it was what i
>>> tested at the time. I should have tested accouting without the
>>> patchset
>>>
>>> Cheers,
>>> Jerome
>>>        
>> Still, I think you have a point in the fact that *when* finally the
>> OOM killer kicks in,
>> it probably doesn't have a way to associate user-space created bos
>> with processes
>> and kill the right process.
>>
>> /Thomas
>>
>>      
> Btw can you review patch 14 as it touch vmwgfx, i am pretty sure
> i got it right.
>
> Cheers,
> Jerome
>    
Yes I will do a more thorough review tomorrow of all unreviewed stuff.

/Thomas