[B.A.T.M.A.N.] [PATCHv4 4/6] batman-adv: remove packed from batadv_ogm_packet

2013-04-25 Thread Simon Wunderlich
As we decreased the struct size from 26 to 24 byte, we can remove __packed as the compiler will not add any more padding. Signed-off-by: Simon Wunderlich s...@hrz.tu-chemnitz.de --- packet.h |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packet.h b/packet.h index

[B.A.T.M.A.N.] [PATCHv4 2/6] batman-adv: add build check macros for packet member offset

2013-04-25 Thread Simon Wunderlich
Since we removed the __packed from most of the packets, we should make sure that the offset generated by the compiler are correct for sent/received data. Signed-off-by: Simon Wunderlich s...@hrz.tu-chemnitz.de --- main.c |8 1 file changed, 8 insertions(+) diff --git a/main.c

[B.A.T.M.A.N.] [PATCHv4 1/6] batman-adv: remove vis functionality

2013-04-25 Thread Simon Wunderlich
This is replaced by a userspace program, we don't need this functionality to bloat the kernel. Signed-off-by: Simon Wunderlich s...@hrz.tu-chemnitz.de --- Makefile.kbuild |1 - bat_iv_ogm.c |7 - debugfs.c|9 - hard-interface.c |9 - main.c | 11 -

[B.A.T.M.A.N.] [PATCHv4 3/6] batman-adv: reorder packet types

2013-04-25 Thread Simon Wunderlich
Reordering the packet type numbers allows us to handle unicast packets in a general way - even if we don't know the specific packet type, we can still forward it. There was already code handling this for a couple of unicast packets, and this is the more generalized version to do that.

[B.A.T.M.A.N.] [PATCHv4 6/6] batman-adv: only add recordroute information to icmp request/reply

2013-04-25 Thread Simon Wunderlich
Adding host information for record route is only required for ICMP requests and replys, and should not be added to just any (future?) packet type. Signed-off-by: Simon Wunderlich s...@hrz.tu-chemnitz.de --- routing.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git

[B.A.T.M.A.N.] [PATCHv4 5/6] batman-adv: reorder batadv_iv_flags

2013-04-25 Thread Simon Wunderlich
The vis flag is not needed anymore, and since we do a compat bump we can start with the first bit again Signed-off-by: Simon Wunderlich s...@hrz.tu-chemnitz.de --- packet.h | 19 ++- 1 file changed, 14 insertions(+), 5 deletions(-) diff --git a/packet.h b/packet.h index

[B.A.T.M.A.N.] [PATCHv5 1/6] batman-adv: remove vis functionality

2013-04-25 Thread 1366879046-24507-2-git-send-email-siwu
From: Simon Wunderlich s...@hrz.tu-chemnitz.de This is replaced by a userspace program, we don't need this functionality to bloat the kernel. Signed-off-by: Simon Wunderlich s...@hrz.tu-chemnitz.de --- Changes from v4: * remove orig_node-flags (only needed for vis flags) * remove vis from

Re: [B.A.T.M.A.N.] [PATCHv5 1/6] batman-adv: remove vis functionality

2013-04-25 Thread Marek Lindner
On Thursday, April 25, 2013 17:57:42 1366879046-24507-2-git-send-email- s...@hrz.tu-chemnitz.de wrote: diff --git a/packet.h b/packet.h index aee6b96..0ce4867 100644 --- a/packet.h +++ b/packet.h @@ -69,12 +69,6 @@ enum batadv_icmp_packettype { BATADV_PARAMETER_PROBLEM = 12,