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

Michel Dänzer michel at daenzer.net
Wed Jul 8 19:41:51 PDT 2015


On 09.07.2015 03:16, j.glisse at gmail.com wrote:
> From: Jérôme Glisse <jglisse at 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 at 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 at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the dri-devel mailing list