From d7f3ba4c16b6e855b517f3d36f4bb620f4fd2677 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 24 Jun 2014 01:20:32 +0000 Subject: [PATCH] rculfhash: remove duplicated code Signed-off-by: Eric Wong Signed-off-by: Mathieu Desnoyers --- rculfhash.c | 19 ++++--------------- 1 file changed, 4 insertions(+), 15 deletions(-) diff --git a/rculfhash.c b/rculfhash.c index 8a8ede7..57f1a04 100644 --- a/rculfhash.c +++ b/rculfhash.c @@ -1176,6 +1176,10 @@ void partition_resize_helper(struct cds_lfht *ht, unsigned long i, int thread, ret; unsigned long nr_threads; + assert(nr_cpus_mask != -1); + if (nr_cpus_mask < 0 || len < 2 * MIN_PARTITION_PER_THREAD) + goto fallback; + /* * Note: nr_cpus_mask + 1 is always power of 2. * We spawn just the number of threads we need to satisfy the minimum @@ -1268,13 +1272,6 @@ static void init_table_populate(struct cds_lfht *ht, unsigned long i, unsigned long len) { - assert(nr_cpus_mask != -1); - if (nr_cpus_mask < 0 || len < 2 * MIN_PARTITION_PER_THREAD) { - ht->flavor->thread_online(); - init_table_populate_partition(ht, i, 0, len); - ht->flavor->thread_offline(); - return; - } partition_resize_helper(ht, i, len, init_table_populate_partition); } @@ -1367,14 +1364,6 @@ void remove_table_partition(struct cds_lfht *ht, unsigned long i, static void remove_table(struct cds_lfht *ht, unsigned long i, unsigned long len) { - - assert(nr_cpus_mask != -1); - if (nr_cpus_mask < 0 || len < 2 * MIN_PARTITION_PER_THREAD) { - ht->flavor->thread_online(); - remove_table_partition(ht, i, 0, len); - ht->flavor->thread_offline(); - return; - } partition_resize_helper(ht, i, len, remove_table_partition); } -- 2.34.1