From: Sven Eckelmann sven.eckelmann@gmx.de Date: Fri, 16 Jul 2010 16:39:16 +0200
+/* count the hamming weight, how many good packets did we receive? just count
- the 1's. The inner loop uses the Kernighan algorithm, see
- */
+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];while (word) {word &= word-1;hamming++;}- }
- return hamming;
+}
The kernel has a hamming weight library function which takes advantage of population count instructions on cpus that suport it, and also has a sw version than is faster than what you're doing here, please use it.
The interfaces are called "hweight{8,16,32,64}()" where the number in the name indicates the bit-size of the word the interface operates on.
I also notice that this code uses it's own internal buffering scheme with kmalloc()'d buffers, then seperately allocates actual SKB's and copies the data there.
Just use the SKB facilities how they were designed to be used, instead of needlessly inventing new things. Allocate your initial SKB and put the initial forwarding header in it, then when you want to send a copy off, skb_clone() it, and push the other bits you want at the head and/or the tail of the cloned SKB, then simply send it off.