diff mbox

[2/4] drm/ttm: zero allocation only for CPU domain

Message ID 1451966143-27368-1-git-send-email-david1.zhou@amd.com (mailing list archive)
State New, archived
Headers show

Commit Message

Chunming Zhou Jan. 5, 2016, 3:55 a.m. UTC 
From: Chunming Zhou <David1.Zhou@amd.com>

Change-Id: I653d61e32faaa11c39bb978bf90301513a0c35b8
Signed-off-by: Chunming Zhou <David1.Zhou@amd.com>
---
 drivers/gpu/drm/ttm/ttm_bo.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Lucas Stach Jan. 5, 2016, 2:32 p.m. UTC | #1 
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;
Chunming Zhou Jan. 6, 2016, 6:44 a.m. UTC | #2 
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;
diff mbox

Patch

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;