Thanks for reply, this patch is mainly for performance reason. since TTM is to clear page for all non-fixed memory (GTT and CPU domains), which takes much more CPU overhead. The patch is to remove 'clear page' for GTT domain. btw: Micheal has rejected this patch.

Regards, David Zhou (Chunming)

On 2016年01月05日 22:32, Lucas Stach wrote:

Hi David,

why is this patch 2/4 if it's just a single patch?

Am Dienstag, den 05.01.2016, 11:55 +0800 schrieb david1.zhou@amd.com:

...

From: Chunming Zhou David1.Zhou@amd.com

Change-Id: I653d61e32faaa11c39bb978bf90301513a0c35b8

This change-id is just pure noise when you submit something upstream.

Please add a proper change log on why the change below is necessary/helpful.

Regards, Lucas

...

Signed-off-by: Chunming Zhou David1.Zhou@amd.com

drivers/gpu/drm/ttm/ttm_bo.c | 1 + 1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c index 745e996..0633b65 100644 --- a/drivers/gpu/drm/ttm/ttm_bo.c +++ b/drivers/gpu/drm/ttm/ttm_bo.c @@ -301,6 +301,7 @@ static int ttm_bo_handle_move_mem(struct ttm_buffer_object *bo, if (!(new_man->flags & TTM_MEMTYPE_FLAG_FIXED)) { if (bo->ttm == NULL) { bool zero = !(old_man->flags & TTM_MEMTYPE_FLAG_FIXED);

• 	zero = (mem->mem_type == TTM_PL_SYSTEM) ?
  

zero : false; ret = ttm_bo_add_ttm(bo, zero); if (ret) goto out_err;