Re: [PATCH 1/2] drm/ttm: fix object deallocation to properly fill in the page pool.

On 09.07.2015 03:16, j.glisse@gmail.com wrote:

From: Jérôme Glisse jglisse@redhat.com

Current code never allowed the page pool to actualy fill in anyway. This fix it, so that we only start freeing page from the pool when we go over the pool size.

Signed-off-by: Jérôme Glisse jglisse@redhat.com

[...]

• if (pool->npages_free > _manager->options.max_size) {
  

• if (pool->npages_free > _manager->options.max_size)
  npages = pool->npages_free - _manager->options.max_size;
  

• 	/* free at least NUM_PAGES_TO_ALLOC number of pages
  

• 	 * to reduce calls to set_memory_wb */
  

• 	if (npages < NUM_PAGES_TO_ALLOC)
  

• 		npages = NUM_PAGES_TO_ALLOC;
  

• }
  

This should be part of patch 2. With that fixed, both patches are

Reviewed-and-Tested-by: Michel Dänzer michel.daenzer@amd.com