From patchwork Sun Oct 24 01:14:16 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Marek Lindner X-Patchwork-Id: 485 Return-Path: Received: from nm8-vm0.bullet.mail.ukl.yahoo.com (nm8-vm0.bullet.mail.ukl.yahoo.com [217.146.183.238]) by open-mesh.org (Postfix) with SMTP id 21CF6154540 for ; Sun, 24 Oct 2010 03:16:19 +0200 (CEST) Received: from [217.146.183.217] by nm8.bullet.mail.ukl.yahoo.com with NNFMP; 24 Oct 2010 01:16:17 -0000 Received: from [217.146.183.33] by tm10.bullet.mail.ukl.yahoo.com with NNFMP; 24 Oct 2010 01:16:17 -0000 Received: from [127.0.0.1] by omp1022.mail.ukl.yahoo.com with NNFMP; 24 Oct 2010 01:16:17 -0000 X-Yahoo-Newman-Id: 908542.8693.bm@omp1022.mail.ukl.yahoo.com Received: (qmail 31216 invoked from network); 24 Oct 2010 01:16:17 -0000 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=s1024; d=yahoo.de; h=DKIM-Signature:Received:X-Yahoo-SMTP:X-YMail-OSG:X-Yahoo-Newman-Property:From:To:Cc:Subject:Date:Message-Id:X-Mailer:In-Reply-To:References; b=zHoUCQeHspSTMmm+2LDufORfAzgRyURxXttQyU7ne9cQPTcFLBfMrPphOGsGEubG5qqEdOCcN0c605FG5xrQNqBY0+5gvpdchIarNS/bv3yc4BaVKbAitDkVH5ubImPsmBSW3xsL1LI9ws9bCepAGpeOjXV0PqDdzhyjcBI9szg= ; DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.de; s=s1024; t=1287882977; bh=rtxKlVfqWGruI1z3R9TYcI4f218TzQ+Lq63w1Q93yhE=; h=Received:X-Yahoo-SMTP:X-YMail-OSG:X-Yahoo-Newman-Property:From:To:Cc:Subject:Date:Message-Id:X-Mailer:In-Reply-To:References; b=3kaPiKHlzcU8A2TIX89YpTzkPRiWJ+EVLKllTz/maouSXU0CPUV3Q9UGF/oDdUB31ho9muYOs4p9OIDQ4gDJTbydghGQbopi9oCJ7hVStwLqOV301w+Da7mDRPCXwMVgdsvNob5VXIDrXDQurw9VXJaMzUOfhQmr/vR4nf76hHk= Received: from localhost (lindner_marek@78.225.40.81 with plain) by smtp124.mail.ukl.yahoo.com with SMTP; 23 Oct 2010 18:16:17 -0700 PDT X-Yahoo-SMTP: tW.h3tiswBBMXO2coYcbPigGD5Lt6zY_.Zc- X-YMail-OSG: rjNkFLoVM1k3LOKrTt5eR8pQYFDTYphWkoA4VbhPg5xxpfF RimqruEtJAAw4IeX3aPOOUk3HEZ1LDmstKZTwecWvGhI2mNgdXD7tN3EDsU_ E4k2ZEqYPwzqbXV6rxctZv_0nnjA.RkyI3bfZl78X.SsEsuSMFkyVExPtDMg pJI15CudgCEN7ShBN6y7gRcOT3y7PO28.77_hMFU0HSRVmpfbQjIo0LU_dpF gwV6yYJmgks5XzAa8JMUZMrvA2mP.IdpwMGhT4l7UbGd0oyon X-Yahoo-Newman-Property: ymail-3 From: Marek Lindner To: b.a.t.m.a.n@lists.open-mesh.org Date: Sun, 24 Oct 2010 03:14:16 +0200 Message-Id: <1287882863-11314-1-git-send-email-lindner_marek@yahoo.de> X-Mailer: git-send-email 1.7.1 In-Reply-To: <201010240313.35215.lindner_marek@yahoo.de> References: <201010240313.35215.lindner_marek@yahoo.de> Cc: Marek Lindner Subject: [B.A.T.M.A.N.] [PATCH 1/8] batman-adv: remove redundant gw_node_list_free() function X-BeenThere: b.a.t.m.a.n@lists.open-mesh.org X-Mailman-Version: 2.1.11 Precedence: list Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 24 Oct 2010 01:16:19 -0000 Signed-off-by: Marek Lindner --- batman-adv/gateway_client.c | 30 +++++++++--------------------- batman-adv/gateway_client.h | 3 +-- batman-adv/main.c | 2 +- batman-adv/originator.c | 2 +- 4 files changed, 12 insertions(+), 25 deletions(-) diff --git a/batman-adv/gateway_client.c b/batman-adv/gateway_client.c index ac33c09..1cd7e0a 100644 --- a/batman-adv/gateway_client.c +++ b/batman-adv/gateway_client.c @@ -306,7 +306,7 @@ void gw_node_delete(struct bat_priv *bat_priv, struct orig_node *orig_node) return gw_node_update(bat_priv, orig_node, 0); } -void gw_node_purge_deleted(struct bat_priv *bat_priv) +void gw_node_purge(struct bat_priv *bat_priv) { struct gw_node *gw_node; struct hlist_node *node, *node_tmp; @@ -315,32 +315,20 @@ void gw_node_purge_deleted(struct bat_priv *bat_priv) spin_lock_bh(&bat_priv->gw_list_lock); hlist_for_each_entry_safe(gw_node, node, node_tmp, - &bat_priv->gw_list, list) { - if ((gw_node->deleted) && - (time_after(jiffies, gw_node->deleted + timeout))) { - - hlist_del_rcu(&gw_node->list); - call_rcu(&gw_node->rcu, gw_node_free_rcu); - } - } - - spin_unlock_bh(&bat_priv->gw_list_lock); -} - -void gw_node_list_free(struct bat_priv *bat_priv) -{ - struct gw_node *gw_node; - struct hlist_node *node, *node_tmp; + &bat_priv->gw_list, list) { + if (((!gw_node->deleted) || + (time_after(jiffies, gw_node->deleted + timeout))) && + atomic_read(&bat_priv->mesh_state) == MESH_ACTIVE) + continue; - spin_lock_bh(&bat_priv->gw_list_lock); + if (bat_priv->curr_gw == gw_node) + gw_deselect(bat_priv); - hlist_for_each_entry_safe(gw_node, node, node_tmp, - &bat_priv->gw_list, list) { hlist_del_rcu(&gw_node->list); call_rcu(&gw_node->rcu, gw_node_free_rcu); } - gw_deselect(bat_priv); + spin_unlock_bh(&bat_priv->gw_list_lock); } diff --git a/batman-adv/gateway_client.h b/batman-adv/gateway_client.h index ce0b4f0..38699ba 100644 --- a/batman-adv/gateway_client.h +++ b/batman-adv/gateway_client.h @@ -29,8 +29,7 @@ void gw_check_election(struct bat_priv *bat_priv, struct orig_node *orig_node); void gw_node_update(struct bat_priv *bat_priv, struct orig_node *orig_node, uint8_t new_gwflags); void gw_node_delete(struct bat_priv *bat_priv, struct orig_node *orig_node); -void gw_node_purge_deleted(struct bat_priv *bat_priv); -void gw_node_list_free(struct bat_priv *bat_priv); +void gw_node_purge(struct bat_priv *bat_priv); int gw_client_seq_print_text(struct seq_file *seq, void *offset); bool gw_is_target(struct bat_priv *bat_priv, struct sk_buff *skb); diff --git a/batman-adv/main.c b/batman-adv/main.c index 73d2752..6d6df7e 100644 --- a/batman-adv/main.c +++ b/batman-adv/main.c @@ -133,7 +133,7 @@ void mesh_free(struct net_device *soft_iface) vis_quit(bat_priv); - gw_node_list_free(bat_priv); + gw_node_purge(bat_priv); originator_free(bat_priv); hna_local_free(bat_priv); diff --git a/batman-adv/originator.c b/batman-adv/originator.c index 8c72a40..39f3160 100644 --- a/batman-adv/originator.c +++ b/batman-adv/originator.c @@ -294,7 +294,7 @@ static void _purge_orig(struct bat_priv *bat_priv) spin_unlock_bh(&bat_priv->orig_hash_lock); - gw_node_purge_deleted(bat_priv); + gw_node_purge(bat_priv); gw_election(bat_priv); softif_neigh_purge(bat_priv);