Re: [patch] drm/ttm: remove unnecessary save_flags and ttm_flag_masked in ttm_bo_util.c

[Thomas Hellstrom <thomas@shipmail.org>]

Yuan,
This looks like an old leftover from a previous cleanup.

Although, when I look at the code, it seems save_flags should be removed 
completely.
Can you take a look at that and respin the patch?

Thanks,
Thomas



Yuan, Shengquan wrote:
> commit d60326ac977a5e99047b44f9b313ff79cd3a14b4
> Author: Austin Yuan <shengquan.yuan@gmail.com>
> Date:   Tue Jan 19 11:45:24 2010 +0800
>
>     drm/ttm: remove unnecessary save_flags and ttm_flag_masked in ttm_bo_util.c
>
>     Signed-off-by: Austin Yuan <shengquan.yuan@gmail.com>
>
> diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c
> b/drivers/gpu/drm/ttm/ttm_bo_util.c
> index ceae52f..a6fca9e 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo_util.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo_util.c
> @@ -77,7 +77,7 @@ int ttm_bo_move_ttm(struct ttm_buffer_object *bo,
>
>  	*old_mem = *new_mem;
>  	new_mem->mm_node = NULL;
> -	ttm_flag_masked(&save_flags, new_mem->placement, TTM_PL_MASK_MEMTYPE);
> +
>  	return 0;
>  }
>  EXPORT_SYMBOL(ttm_bo_move_ttm);
> @@ -219,7 +219,6 @@ int ttm_bo_move_memcpy(struct ttm_buffer_object *bo,
>  	void *old_iomap;
>  	void *new_iomap;
>  	int ret;
> -	uint32_t save_flags = old_mem->placement;
>  	unsigned long i;
>  	unsigned long page;
>  	unsigned long add = 0;
> @@ -270,7 +269,6 @@ out2:
>
>  	*old_mem = *new_mem;
>  	new_mem->mm_node = NULL;
> -	ttm_flag_masked(&save_flags, new_mem->placement, TTM_PL_MASK_MEMTYPE);
>
>  	if ((man->flags & TTM_MEMTYPE_FLAG_FIXED) && (ttm != NULL)) {
>  		ttm_tt_unbind(ttm);
> @@ -537,7 +535,6 @@ int ttm_bo_move_accel_cleanup(struct ttm_buffer_object *bo,
>  	struct ttm_mem_type_manager *man = &bdev->man[new_mem->mem_type];
>  	struct ttm_mem_reg *old_mem = &bo->mem;
>  	int ret;
> -	uint32_t save_flags = old_mem->placement;
>  	struct ttm_buffer_object *ghost_obj;
>  	void *tmp_obj = NULL;
>
> @@ -598,7 +595,7 @@ int ttm_bo_move_accel_cleanup(struct ttm_buffer_object *bo,
>
>  	*old_mem = *new_mem;
>  	new_mem->mm_node = NULL;
> -	ttm_flag_masked(&save_flags, new_mem->placement, TTM_PL_MASK_MEMTYPE);
> +
>  	return 0;
>  }
>  EXPORT_SYMBOL(ttm_bo_move_accel_cleanup);
>
> ------------------------------------------------------------------------------
> Throughout its 18-year history, RSA Conference consistently attracts the
> world's best and brightest in the field, creating opportunities for Conference
> attendees to learn about information security's most important issues through
> interactions with peers, luminaries and emerging and established companies.
> http://p.sf.net/sfu/rsaconf-dev2dev
> --
> _______________________________________________
> Dri-devel mailing list
> Dri-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/dri-devel
>   


------------------------------------------------------------------------------
Throughout its 18-year history, RSA Conference consistently attracts the
world's best and brightest in the field, creating opportunities for Conference
attendees to learn about information security's most important issues through
interactions with peers, luminaries and emerging and established companies.
http://p.sf.net/sfu/rsaconf-dev2dev
--