From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Hellstrom <thellstrom@vmware.com>
Subject: Re: ttm: merge ttm_backend & ttm_tt, introduce ttm dma allocator
 V5
Date: Mon, 14 Nov 2011 20:25:54 +0100
Message-ID: <4EC16B42.3070000@vmware.com>
References: <1321051669-16675-1-git-send-email-j.glisse@gmail.com>
	<4EC12A95.7080808@vmware.com>
	<CAH3drwarw4MqsDB8-KoaH=o-Q0iwOkOFukB25NeLsJcyeusiUQ@mail.gmail.com>
	<4EC13C92.5060105@vmware.com>
	<20111114185427.GA5300@homer.localdomain>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org>
Received: from smtp-outbound-1.vmware.com (smtp-outbound-1.vmware.com
	[65.115.85.69])
	by gabe.freedesktop.org (Postfix) with ESMTP id 0E8B79F316
	for <dri-devel@lists.freedesktop.org>;
	Mon, 14 Nov 2011 11:28:36 -0800 (PST)
In-Reply-To: <20111114185427.GA5300@homer.localdomain>
List-Unsubscribe: <http://lists.freedesktop.org/mailman/options/dri-devel>,
	<mailto:dri-devel-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <http://lists.freedesktop.org/archives/dri-devel>
List-Post: <mailto:dri-devel@lists.freedesktop.org>
List-Help: <mailto:dri-devel-request@lists.freedesktop.org?subject=help>
List-Subscribe: <http://lists.freedesktop.org/mailman/listinfo/dri-devel>,
	<mailto:dri-devel-request@lists.freedesktop.org?subject=subscribe>
Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org
Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org
To: Jerome Glisse <j.glisse@gmail.com>
Cc: dri-devel@lists.freedesktop.org
List-Id: dri-devel@lists.freedesktop.org

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