diff mbox

batman-adv: Calculate hamming weight using optimized kernel functions

Message ID 1279616589-3086-1-git-send-email-sven.eckelmann@gmx.de (mailing list archive)
State Accepted, archived
Headers

Commit Message

Sven Eckelmann July 20, 2010, 9:03 a.m. UTC 
  The Kernighan algorithm is not able to calculate the number of set bits
in parallel and the compiler cannot replace it with optimized
instructions.

The kernel provides specialised functions for each cpu which can either
use a software implementation or hardware instruction depending on the
target cpu.

Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
---
 batman-adv/bitarray.c |   15 +++++----------
 batman-adv/bitarray.h |    3 ++-
 2 files changed, 7 insertions(+), 11 deletions(-)

Comments

Sven Eckelmann July 20, 2010, 2:54 p.m. UTC | #1 
Sven Eckelmann wrote:
> The Kernighan algorithm is not able to calculate the number of set bits
> in parallel and the compiler cannot replace it with optimized
> instructions.
> 
> The kernel provides specialised functions for each cpu which can either
> use a software implementation or hardware instruction depending on the
> target cpu.
> 
> Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>
> ---

Maybe someone could do me the favor and add before the Signed-off-by: line one 
that mentions that David S. Miller is the actual reporter. This would look 
something like this:

Reported-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>

thanks,
	Sven
Marek Lindner July 22, 2010, 3:50 p.m. UTC | #2 
On Tuesday, July 20, 2010 16:54:10 Sven Eckelmann wrote:
> Maybe someone could do me the favor and add before the Signed-off-by: line
> one  that mentions that David S. Miller is the actual reporter. This would
> look something like this:
> 
> Reported-by: David S. Miller <davem@davemloft.net>
> Signed-off-by: Sven Eckelmann <sven.eckelmann@gmx.de>

Ok, I applied the patch in revision 1740.

Thanks,
Marek
diff mbox

Patch

diff --git a/batman-adv/bitarray.c b/batman-adv/bitarray.c
index dd4193c..9dbaf1e 100644
--- a/batman-adv/bitarray.c
+++ b/batman-adv/bitarray.c
@@ -22,6 +22,8 @@ 
 #include "main.h"
 #include "bitarray.h"
 
+#include <linux/bitops.h>
+
 /* returns true if the corresponding bit in the given seq_bits indicates true
  * and curr_seqno is within range of last_seqno */
 uint8_t get_bit_status(TYPE_OF_WORD *seq_bits, uint32_t last_seqno,
@@ -187,21 +189,14 @@  char bit_get_packet(TYPE_OF_WORD *seq_bits, int32_t seq_num_diff,
 }
 
 /* count the hamming weight, how many good packets did we receive? just count
- * the 1's. The inner loop uses the Kernighan algorithm, see
- * http://graphics.stanford.edu/~seander/bithacks.html#CountBitsSetKernighan
+ * the 1's.
  */
 int bit_packet_count(TYPE_OF_WORD *seq_bits)
 {
 	int i, hamming = 0;
-	TYPE_OF_WORD word;
 
-	for (i = 0; i < NUM_WORDS; i++) {
-		word = seq_bits[i];
+	for (i = 0; i < NUM_WORDS; i++)
+		hamming += hweight_long(seq_bits[i]);
 
-		while (word) {
-			word &= word-1;
-			hamming++;
-		}
-	}
 	return hamming;
 }
diff --git a/batman-adv/bitarray.h b/batman-adv/bitarray.h
index 01897d6..c0c1730 100644
--- a/batman-adv/bitarray.h
+++ b/batman-adv/bitarray.h
@@ -22,7 +22,8 @@ 
 #ifndef _NET_BATMAN_ADV_BITARRAY_H_
 #define _NET_BATMAN_ADV_BITARRAY_H_
 
-/* you should choose something big, if you don't want to waste cpu */
+/* you should choose something big, if you don't want to waste cpu
+   and keep the type in sync with bit_packet_count */
 #define TYPE_OF_WORD unsigned long
 #define WORD_BIT_SIZE (sizeof(TYPE_OF_WORD) * 8)