From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Hellstrom Subject: Re: [patch] drm/ttm: remove unnecessary save_flags and ttm_flag_masked in ttm_bo_util.c Date: Wed, 20 Jan 2010 11:48:02 +0100 Message-ID: <4B56DF62.4030306@shipmail.org> References: Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.sourceforge.net To: "Yuan, Shengquan" Cc: dri-devel@lists.sf.net List-Id: dri-devel@lists.freedesktop.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 > 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 > > 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 --