From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bob Liu Subject: [PATCH 06/14] tmem: cleanup: remove unneed parameter for pgp_free() Date: Wed, 18 Dec 2013 14:52:33 +0800 Message-ID: <1387349561-27923-7-git-send-email-bob.liu@oracle.com> References: <1387349561-27923-1-git-send-email-bob.liu@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1VtB23-0006eg-4Q for xen-devel@lists.xenproject.org; Wed, 18 Dec 2013 06:54:59 +0000 Received: by mail-pb0-f48.google.com with SMTP id md12so8014168pbc.21 for ; Tue, 17 Dec 2013 22:54:55 -0800 (PST) In-Reply-To: <1387349561-27923-1-git-send-email-bob.liu@oracle.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xenproject.org Cc: james.harper@bendigoit.com.au, keir@xen.org, ian.campbell@citrix.com, andrew.cooper3@citrix.com, JBeulich@suse.com List-Id: xen-devel@lists.xenproject.org The only difference of the from_delete parameter in pgp_free() is one line ASSERT(), this patch remove it to make code more clean. Signed-off-by: Bob Liu --- xen/common/tmem.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/xen/common/tmem.c b/xen/common/tmem.c index 8a6ee84..9cfbca3 100644 --- a/xen/common/tmem.c +++ b/xen/common/tmem.c @@ -654,15 +654,14 @@ static void pgp_free_data(struct tmem_page_descriptor *pgp, struct tmem_pool *po pgp->size = -1; } -static void pgp_free(struct tmem_page_descriptor *pgp, int from_delete) +static void pgp_free(struct tmem_page_descriptor *pgp) { struct tmem_pool *pool = NULL; ASSERT(pgp->us.obj != NULL); - ASSERT(pgp->us.obj->pool->client != NULL); - if ( from_delete ) - ASSERT(pgp_lookup_in_obj(pgp->us.obj,pgp->index) == NULL); ASSERT(pgp->us.obj->pool != NULL); + ASSERT(pgp->us.obj->pool->client != NULL); + pool = pgp->us.obj->pool; if ( !is_persistent(pool) ) { @@ -743,7 +742,8 @@ static void pgp_delete(struct tmem_page_descriptor *pgp) life = get_cycles() - pgp->timestamp; pgp->us.obj->pool->sum_life_cycles += life; pgp_delist(pgp); - pgp_free(pgp,1); + ASSERT(pgp_lookup_in_obj(pgp->us.obj,pgp->index) == NULL); + pgp_free(pgp); } /* called only indirectly by radix_tree_destroy */ @@ -756,7 +756,7 @@ static void pgp_destroy(void *v) ASSERT(pgp->us.obj != NULL); pgp->us.obj->pgp_count--; ASSERT(pgp->us.obj->pgp_count >= 0); - pgp_free(pgp,0); + pgp_free(pgp); } static int pgp_add_to_obj(struct tmem_object_root *obj, uint32_t index, struct tmem_page_descriptor *pgp) @@ -1350,7 +1350,7 @@ found: } /* pgp already delist, so call pgp_free directly */ - pgp_free(pgp, 1); + pgp_free(pgp); if ( obj->pgp_count == 0 ) { ASSERT_WRITELOCK(&pool->pool_rwlock); -- 1.7.10.4