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 <[email protected]> > 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 <[email protected]> > > 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 > [email protected] > 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 -- _______________________________________________ Dri-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dri-devel
