B.A.T.M.A.N

b.a.t.m.a.n@lists.open-mesh.org

15 participants
4309 discussions

[B.A.T.M.A.N.] alfred and ipv6
by cmsv 30 May '13

30 May '13

As i am about to try out alfred i already face myself with a couple issues: http://www.open-mesh.org/projects/open-mesh/wiki/Alfred http://git.open-mesh.org/alfred.git/blob_plain/refs/heads/master:/README I the past i had removed ipv6 support since i do not need it and it helps trimming up the final image. (most routers and people still use 4 mb flash) I would like to use alfred and would also like to know if compiling the image with <*> kmod-ipv6................................................... IPv6 support Leaving out : [ ] ping6 [ ] Enable IPv6 support [ ] Enable IPv6 support in packages will allow Alfred to work as intended. As anyone tested? Another problem with using ipv6 is the final image size. kmod-ipv6_3.3.8-1_ar71xx.ipk is at least 138K An that is quite a bit ... What are exactly the reasons why alfred does not use ipv4 ? -- Site: http://wirelesspt.net Mesh: http://tinyurl.com/wirelesspt Admin: http://wirelesspt.net/wiki/Cmsv Twitter: http://twitter.com/wirelesspt Suporte técnico via sms: 91 19 11 798 Donativos/Paypal: http://tinyurl.com/doar-verba Chave publica PGP/SSH: http://wirelesspt.net/arquivos/pk Email assinado digitalmente pelo emissor assegurando autenticidade

2 1

[B.A.T.M.A.N.] [PATCHv2] batctl: create common header for ICMP packets
by Antonio Quartulli 29 May '13

29 May '13

From: Antonio Quartulli <antonio(a)open-mesh.com> the icmp and the icmp_rr packets share the same initial fields since they use the same code to be processed and forwarded. Extract the common fields and put them into a separate struct so that future ICMP packets can be easily added without bloating the packet definition. However, keep the seqno field outside of the newly created common header because future ICMP types may require a bigger sequence number space. This change breaks compatibility due to fields reordering in the ICMP headers. Signed-off-by: Antonio Quartulli <antonio(a)open-mesh.com> --- v2: - added changes to packet.h packet.h | 38 ++++++++++++++++++++++++++++---------- ping.c | 24 ++++++++++++++---------- tcpdump.c | 27 +++++++++++++++++---------- traceroute.c | 31 +++++++++++++++++++------------ 4 files changed, 78 insertions(+), 42 deletions(-) diff --git a/packet.h b/packet.h index 57d2002..8b8b03f 100644 --- a/packet.h +++ b/packet.h @@ -186,29 +186,47 @@ struct batadv_ogm_packet { #define BATADV_OGM_HLEN sizeof(struct batadv_ogm_packet) -struct batadv_icmp_packet { +/** + * batadc_icmp_header - common ICMP header + * @header: common batman header + * @msg_type: ICMP packet type + * @dst: address of the destination node + * @orig: address of the source node + * @uid: local ICMP socket identifier + */ +struct batadv_icmp_header { struct batadv_header header; uint8_t msg_type; /* see ICMP message types above */ uint8_t dst[ETH_ALEN]; uint8_t orig[ETH_ALEN]; - __be16 seqno; uint8_t uid; +}; + +/** + * batadv_icmp_packet - ICMP packet + * @icmph: common ICMP header + * @reserved: not used - useful for alignment + * @seqno: ICMP sequence number + */ +struct batadv_icmp_packet { + struct batadv_icmp_header icmph; uint8_t reserved; + __be16 seqno; }; #define BATADV_RR_LEN 16 -/* icmp_packet_rr must start with all fields from imcp_packet - * as this is assumed by code that handles ICMP packets +/** + * batadv_icmp_packet_rr - ICMP RouteRecord packet + * @icmph: common ICMP header + * @rr_cur: number of entries the rr array + * @seqno: ICMP sequence number + * @rr: route record array */ struct batadv_icmp_packet_rr { - struct batadv_header header; - uint8_t msg_type; /* see ICMP message types above */ - uint8_t dst[ETH_ALEN]; - uint8_t orig[ETH_ALEN]; - __be16 seqno; - uint8_t uid; + struct batadv_icmp_header icmph; uint8_t rr_cur; + __be16 seqno; uint8_t rr[BATADV_RR_LEN][ETH_ALEN]; }; diff --git a/ping.c b/ping.c index 5d6b7e9..6679604 100644 --- a/ping.c +++ b/ping.c @@ -171,11 +171,11 @@ int ping(char *mesh_iface, int argc, char **argv) packet_len = sizeof(struct batadv_icmp_packet); - memcpy(&icmp_packet_out.dst, dst_mac, ETH_ALEN); - icmp_packet_out.header.packet_type = BATADV_ICMP; - icmp_packet_out.header.version = BATADV_COMPAT_VERSION; - icmp_packet_out.msg_type = BATADV_ECHO_REQUEST; - icmp_packet_out.header.ttl = 50; + memcpy(&icmp_packet_out.icmph.dst, dst_mac, ETH_ALEN); + icmp_packet_out.icmph.header.packet_type = BATADV_ICMP; + icmp_packet_out.icmph.header.version = BATADV_COMPAT_VERSION; + icmp_packet_out.icmph.msg_type = BATADV_ECHO_REQUEST; + icmp_packet_out.icmph.header.ttl = 50; icmp_packet_out.seqno = 0; if (rr) { @@ -245,12 +245,14 @@ read_packet: if (htons(seq_counter) != icmp_packet_in.seqno) goto read_packet; - switch (icmp_packet_in.msg_type) { + switch (icmp_packet_in.icmph.msg_type) { case BATADV_ECHO_REPLY: time_delta = end_timer(); printf("%zd bytes from %s icmp_seq=%hu ttl=%d time=%.2f ms", - read_len, dst_string, ntohs(icmp_packet_in.seqno), - icmp_packet_in.header.ttl, time_delta); + read_len, dst_string, + ntohs(icmp_packet_in.seqno), + icmp_packet_in.icmph.header.ttl, + time_delta); if (read_len == sizeof(struct batadv_icmp_packet_rr)) { if (last_rr_cur == icmp_packet_in.rr_cur @@ -299,11 +301,13 @@ read_packet: break; case BATADV_PARAMETER_PROBLEM: fprintf(stderr, "Error - the batman adv kernel module version (%d) differs from ours (%d)\n", - icmp_packet_in.header.version, BATADV_COMPAT_VERSION); + icmp_packet_in.icmph.header.version, + BATADV_COMPAT_VERSION); printf("Please make sure to use compatible versions!\n"); goto out; default: - printf("Unknown message type %d len %zd received\n", icmp_packet_in.msg_type, read_len); + printf("Unknown message type %d len %zd received\n", + icmp_packet_in.icmph.msg_type, read_len); break; } diff --git a/tcpdump.c b/tcpdump.c index 3e932cd..7e0987b 100644 --- a/tcpdump.c +++ b/tcpdump.c @@ -361,32 +361,39 @@ static void dump_batman_icmp(unsigned char *packet_buff, ssize_t buff_len, int r if (!time_printed) print_time(); - printf("BAT %s > ", get_name_by_macaddr((struct ether_addr *)icmp_packet->orig, read_opt)); + printf("BAT %s > ", + get_name_by_macaddr((struct ether_addr *)icmp_packet->icmph.orig, + read_opt)); - name = get_name_by_macaddr((struct ether_addr *)icmp_packet->dst, read_opt); + name = get_name_by_macaddr((struct ether_addr *)icmp_packet->icmph.dst, + read_opt); - switch (icmp_packet->msg_type) { + switch (icmp_packet->icmph.msg_type) { case BATADV_ECHO_REPLY: printf("%s: ICMP echo reply, id %hhu, seq %hu, ttl %2d, v %d, length %zu\n", - name, icmp_packet->uid, ntohs(icmp_packet->seqno), - icmp_packet->header.ttl, icmp_packet->header.version, + name, icmp_packet->icmph.uid, ntohs(icmp_packet->seqno), + icmp_packet->icmph.header.ttl, + icmp_packet->icmph.header.version, (size_t)buff_len - sizeof(struct ether_header)); break; case BATADV_ECHO_REQUEST: printf("%s: ICMP echo request, id %hhu, seq %hu, ttl %2d, v %d, length %zu\n", - name, icmp_packet->uid, ntohs(icmp_packet->seqno), - icmp_packet->header.ttl, icmp_packet->header.version, + name, icmp_packet->icmph.uid, ntohs(icmp_packet->seqno), + icmp_packet->icmph.header.ttl, + icmp_packet->icmph.header.version, (size_t)buff_len - sizeof(struct ether_header)); break; case BATADV_TTL_EXCEEDED: printf("%s: ICMP time exceeded in-transit, id %hhu, seq %hu, ttl %2d, v %d, length %zu\n", - name, icmp_packet->uid, ntohs(icmp_packet->seqno), - icmp_packet->header.ttl, icmp_packet->header.version, + name, icmp_packet->icmph.uid, ntohs(icmp_packet->seqno), + icmp_packet->icmph.header.ttl, + icmp_packet->icmph.header.version, (size_t)buff_len - sizeof(struct ether_header)); break; default: printf("%s: ICMP type %hhu, length %zu\n", - name, icmp_packet->msg_type, (size_t)buff_len - sizeof(struct ether_header)); + name, icmp_packet->icmph.msg_type, + (size_t)buff_len - sizeof(struct ether_header)); break; } } diff --git a/traceroute.c b/traceroute.c index 38d3796..362d293 100644 --- a/traceroute.c +++ b/traceroute.c @@ -129,17 +129,19 @@ int traceroute(char *mesh_iface, int argc, char **argv) goto out; } - memcpy(&icmp_packet_out.dst, dst_mac, ETH_ALEN); - icmp_packet_out.header.version = BATADV_COMPAT_VERSION; - icmp_packet_out.header.packet_type = BATADV_ICMP; - icmp_packet_out.msg_type = BATADV_ECHO_REQUEST; + memcpy(&icmp_packet_out.icmph.dst, dst_mac, ETH_ALEN); + icmp_packet_out.icmph.header.version = BATADV_COMPAT_VERSION; + icmp_packet_out.icmph.header.packet_type = BATADV_ICMP; + icmp_packet_out.icmph.msg_type = BATADV_ECHO_REQUEST; icmp_packet_out.seqno = 0; icmp_packet_out.reserved = 0; printf("traceroute to %s (%s), %d hops max, %zu byte packets\n", dst_string, mac_string, TTL_MAX, sizeof(icmp_packet_out)); - for (icmp_packet_out.header.ttl = 1; !dst_reached && icmp_packet_out.header.ttl < TTL_MAX; icmp_packet_out.header.ttl++) { + for (icmp_packet_out.icmph.header.ttl = 1; + !dst_reached && icmp_packet_out.icmph.header.ttl < TTL_MAX; + icmp_packet_out.icmph.header.ttl++) { return_mac = NULL; bat_host = NULL; @@ -183,7 +185,7 @@ read_packet: if (htons(seq_counter) != icmp_packet_in.seqno) goto read_packet; - switch (icmp_packet_in.msg_type) { + switch (icmp_packet_in.icmph.msg_type) { case BATADV_ECHO_REPLY: dst_reached = 1; /* fall through */ @@ -191,10 +193,10 @@ read_packet: time_delta[i] = end_timer(); if (!return_mac) { - return_mac = ether_ntoa_long((struct ether_addr *)&icmp_packet_in.orig); + return_mac = ether_ntoa_long((struct ether_addr *)&icmp_packet_in.icmph.orig); if (read_opt & USE_BAT_HOSTS) - bat_host = bat_hosts_find_by_mac((char *)&icmp_packet_in.orig); + bat_host = bat_hosts_find_by_mac((char *)&icmp_packet_in.icmph.orig); } break; @@ -203,19 +205,24 @@ read_packet: goto out; case BATADV_PARAMETER_PROBLEM: fprintf(stderr, "Error - the batman adv kernel module version (%d) differs from ours (%d)\n", - icmp_packet_in.header.version, BATADV_COMPAT_VERSION); + icmp_packet_in.icmph.header.version, + BATADV_COMPAT_VERSION); fprintf(stderr, "Please make sure to use compatible versions!\n"); goto out; default: - printf("Unknown message type %d len %zd received\n", icmp_packet_in.msg_type, read_len); + printf("Unknown message type %d len %zd received\n", + icmp_packet_in.icmph.msg_type, read_len); break; } } if (!bat_host) - printf("%2hhu: %s", icmp_packet_out.header.ttl, (return_mac ? return_mac : "*")); + printf("%2hhu: %s", icmp_packet_out.icmph.header.ttl, + (return_mac ? return_mac : "*")); else - printf("%2hhu: %s (%s)", icmp_packet_out.header.ttl, bat_host->name, return_mac); + printf("%2hhu: %s (%s)", + icmp_packet_out.icmph.header.ttl, + bat_host->name, return_mac); for (i = 0; i < NUM_PACKETS; i++) { if (time_delta[i]) -- 1.8.1.5

2 1

[B.A.T.M.A.N.] [PATCH] batctl: create common header for ICMP packets
by Antonio Quartulli 29 May '13

29 May '13

From: Antonio Quartulli <antonio(a)open-mesh.com> the icmp and the icmp_rr packets share the same initial fields since they use the same code to be processed and forwarded. Extract the common fields and put them into a separate struct so that future ICMP packets can be easily added without bloating the packet definition. However, keep the seqno field outside of the newly created common header because future ICMP types may require a bigger sequence number space. This change breaks compatibility due to fields reordering in the ICMP headers. Signed-off-by: Antonio Quartulli <antonio(a)open-mesh.com> --- This patch assumes that ("batman-adv: fix typo in kernel doc") and ("batman-adv: correctly align the tt_tvlv_data struct") have been applied to batctl too. ping.c | 24 ++++++++++++++---------- tcpdump.c | 27 +++++++++++++++++---------- traceroute.c | 31 +++++++++++++++++++------------ 3 files changed, 50 insertions(+), 32 deletions(-) diff --git a/ping.c b/ping.c index 5d6b7e9..6679604 100644 --- a/ping.c +++ b/ping.c @@ -171,11 +171,11 @@ int ping(char *mesh_iface, int argc, char **argv) packet_len = sizeof(struct batadv_icmp_packet); - memcpy(&icmp_packet_out.dst, dst_mac, ETH_ALEN); - icmp_packet_out.header.packet_type = BATADV_ICMP; - icmp_packet_out.header.version = BATADV_COMPAT_VERSION; - icmp_packet_out.msg_type = BATADV_ECHO_REQUEST; - icmp_packet_out.header.ttl = 50; + memcpy(&icmp_packet_out.icmph.dst, dst_mac, ETH_ALEN); + icmp_packet_out.icmph.header.packet_type = BATADV_ICMP; + icmp_packet_out.icmph.header.version = BATADV_COMPAT_VERSION; + icmp_packet_out.icmph.msg_type = BATADV_ECHO_REQUEST; + icmp_packet_out.icmph.header.ttl = 50; icmp_packet_out.seqno = 0; if (rr) { @@ -245,12 +245,14 @@ read_packet: if (htons(seq_counter) != icmp_packet_in.seqno) goto read_packet; - switch (icmp_packet_in.msg_type) { + switch (icmp_packet_in.icmph.msg_type) { case BATADV_ECHO_REPLY: time_delta = end_timer(); printf("%zd bytes from %s icmp_seq=%hu ttl=%d time=%.2f ms", - read_len, dst_string, ntohs(icmp_packet_in.seqno), - icmp_packet_in.header.ttl, time_delta); + read_len, dst_string, + ntohs(icmp_packet_in.seqno), + icmp_packet_in.icmph.header.ttl, + time_delta); if (read_len == sizeof(struct batadv_icmp_packet_rr)) { if (last_rr_cur == icmp_packet_in.rr_cur @@ -299,11 +301,13 @@ read_packet: break; case BATADV_PARAMETER_PROBLEM: fprintf(stderr, "Error - the batman adv kernel module version (%d) differs from ours (%d)\n", - icmp_packet_in.header.version, BATADV_COMPAT_VERSION); + icmp_packet_in.icmph.header.version, + BATADV_COMPAT_VERSION); printf("Please make sure to use compatible versions!\n"); goto out; default: - printf("Unknown message type %d len %zd received\n", icmp_packet_in.msg_type, read_len); + printf("Unknown message type %d len %zd received\n", + icmp_packet_in.icmph.msg_type, read_len); break; } diff --git a/tcpdump.c b/tcpdump.c index 3e932cd..7e0987b 100644 --- a/tcpdump.c +++ b/tcpdump.c @@ -361,32 +361,39 @@ static void dump_batman_icmp(unsigned char *packet_buff, ssize_t buff_len, int r if (!time_printed) print_time(); - printf("BAT %s > ", get_name_by_macaddr((struct ether_addr *)icmp_packet->orig, read_opt)); + printf("BAT %s > ", + get_name_by_macaddr((struct ether_addr *)icmp_packet->icmph.orig, + read_opt)); - name = get_name_by_macaddr((struct ether_addr *)icmp_packet->dst, read_opt); + name = get_name_by_macaddr((struct ether_addr *)icmp_packet->icmph.dst, + read_opt); - switch (icmp_packet->msg_type) { + switch (icmp_packet->icmph.msg_type) { case BATADV_ECHO_REPLY: printf("%s: ICMP echo reply, id %hhu, seq %hu, ttl %2d, v %d, length %zu\n", - name, icmp_packet->uid, ntohs(icmp_packet->seqno), - icmp_packet->header.ttl, icmp_packet->header.version, + name, icmp_packet->icmph.uid, ntohs(icmp_packet->seqno), + icmp_packet->icmph.header.ttl, + icmp_packet->icmph.header.version, (size_t)buff_len - sizeof(struct ether_header)); break; case BATADV_ECHO_REQUEST: printf("%s: ICMP echo request, id %hhu, seq %hu, ttl %2d, v %d, length %zu\n", - name, icmp_packet->uid, ntohs(icmp_packet->seqno), - icmp_packet->header.ttl, icmp_packet->header.version, + name, icmp_packet->icmph.uid, ntohs(icmp_packet->seqno), + icmp_packet->icmph.header.ttl, + icmp_packet->icmph.header.version, (size_t)buff_len - sizeof(struct ether_header)); break; case BATADV_TTL_EXCEEDED: printf("%s: ICMP time exceeded in-transit, id %hhu, seq %hu, ttl %2d, v %d, length %zu\n", - name, icmp_packet->uid, ntohs(icmp_packet->seqno), - icmp_packet->header.ttl, icmp_packet->header.version, + name, icmp_packet->icmph.uid, ntohs(icmp_packet->seqno), + icmp_packet->icmph.header.ttl, + icmp_packet->icmph.header.version, (size_t)buff_len - sizeof(struct ether_header)); break; default: printf("%s: ICMP type %hhu, length %zu\n", - name, icmp_packet->msg_type, (size_t)buff_len - sizeof(struct ether_header)); + name, icmp_packet->icmph.msg_type, + (size_t)buff_len - sizeof(struct ether_header)); break; } } diff --git a/traceroute.c b/traceroute.c index 38d3796..362d293 100644 --- a/traceroute.c +++ b/traceroute.c @@ -129,17 +129,19 @@ int traceroute(char *mesh_iface, int argc, char **argv) goto out; } - memcpy(&icmp_packet_out.dst, dst_mac, ETH_ALEN); - icmp_packet_out.header.version = BATADV_COMPAT_VERSION; - icmp_packet_out.header.packet_type = BATADV_ICMP; - icmp_packet_out.msg_type = BATADV_ECHO_REQUEST; + memcpy(&icmp_packet_out.icmph.dst, dst_mac, ETH_ALEN); + icmp_packet_out.icmph.header.version = BATADV_COMPAT_VERSION; + icmp_packet_out.icmph.header.packet_type = BATADV_ICMP; + icmp_packet_out.icmph.msg_type = BATADV_ECHO_REQUEST; icmp_packet_out.seqno = 0; icmp_packet_out.reserved = 0; printf("traceroute to %s (%s), %d hops max, %zu byte packets\n", dst_string, mac_string, TTL_MAX, sizeof(icmp_packet_out)); - for (icmp_packet_out.header.ttl = 1; !dst_reached && icmp_packet_out.header.ttl < TTL_MAX; icmp_packet_out.header.ttl++) { + for (icmp_packet_out.icmph.header.ttl = 1; + !dst_reached && icmp_packet_out.icmph.header.ttl < TTL_MAX; + icmp_packet_out.icmph.header.ttl++) { return_mac = NULL; bat_host = NULL; @@ -183,7 +185,7 @@ read_packet: if (htons(seq_counter) != icmp_packet_in.seqno) goto read_packet; - switch (icmp_packet_in.msg_type) { + switch (icmp_packet_in.icmph.msg_type) { case BATADV_ECHO_REPLY: dst_reached = 1; /* fall through */ @@ -191,10 +193,10 @@ read_packet: time_delta[i] = end_timer(); if (!return_mac) { - return_mac = ether_ntoa_long((struct ether_addr *)&icmp_packet_in.orig); + return_mac = ether_ntoa_long((struct ether_addr *)&icmp_packet_in.icmph.orig); if (read_opt & USE_BAT_HOSTS) - bat_host = bat_hosts_find_by_mac((char *)&icmp_packet_in.orig); + bat_host = bat_hosts_find_by_mac((char *)&icmp_packet_in.icmph.orig); } break; @@ -203,19 +205,24 @@ read_packet: goto out; case BATADV_PARAMETER_PROBLEM: fprintf(stderr, "Error - the batman adv kernel module version (%d) differs from ours (%d)\n", - icmp_packet_in.header.version, BATADV_COMPAT_VERSION); + icmp_packet_in.icmph.header.version, + BATADV_COMPAT_VERSION); fprintf(stderr, "Please make sure to use compatible versions!\n"); goto out; default: - printf("Unknown message type %d len %zd received\n", icmp_packet_in.msg_type, read_len); + printf("Unknown message type %d len %zd received\n", + icmp_packet_in.icmph.msg_type, read_len); break; } } if (!bat_host) - printf("%2hhu: %s", icmp_packet_out.header.ttl, (return_mac ? return_mac : "*")); + printf("%2hhu: %s", icmp_packet_out.icmph.header.ttl, + (return_mac ? return_mac : "*")); else - printf("%2hhu: %s (%s)", icmp_packet_out.header.ttl, bat_host->name, return_mac); + printf("%2hhu: %s (%s)", + icmp_packet_out.icmph.header.ttl, + bat_host->name, return_mac); for (i = 0; i < NUM_PACKETS; i++) { if (time_delta[i]) -- 1.8.1.5

2 2

[B.A.T.M.A.N.] [PATCH] batman-adv: fix network order complaints in the TVLV code
by Antonio Quartulli 29 May '13

29 May '13

Intorduced by 1ab746a3f2441d3466592b86134fbc72a9fe0558 ("batman-adv: properly align tvlv infrastructure") Signed-off-by: Antonio Quartulli <ordex(a)autistici.org> --- main.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/main.c b/main.c index 9df71bc..1192e61 100644 --- a/main.c +++ b/main.c @@ -630,7 +630,7 @@ static uint16_t batadv_tvlv_container_list_size(struct batadv_priv *bat_priv) hlist_for_each_entry(tvlv, &bat_priv->tvlv.container_list, list) { tvlv_len += sizeof(struct batadv_tvlv_hdr); - tvlv_len += tvlv->tvlv_hdr.len; + tvlv_len += ntohs(tvlv->tvlv_hdr.len); } return tvlv_len; @@ -701,9 +701,9 @@ void batadv_tvlv_container_register(struct batadv_priv *bat_priv, tvlv_new->tvlv_hdr.version = version; tvlv_new->tvlv_hdr.type = type; - tvlv_new->tvlv_hdr.len = tvlv_value_len; + tvlv_new->tvlv_hdr.len = htons(tvlv_value_len); - memcpy(tvlv_new + 1, tvlv_value, tvlv_new->tvlv_hdr.len); + memcpy(tvlv_new + 1, tvlv_value, ntohs(tvlv_new->tvlv_hdr.len)); INIT_HLIST_NODE(&tvlv_new->list); atomic_set(&tvlv_new->refcount, 1); @@ -790,10 +790,10 @@ uint16_t batadv_tvlv_container_ogm_append(struct batadv_priv *bat_priv, tvlv_hdr = tvlv_value; tvlv_hdr->type = tvlv->tvlv_hdr.type; tvlv_hdr->version = tvlv->tvlv_hdr.version; - tvlv_hdr->len = htons(tvlv->tvlv_hdr.len); + tvlv_hdr->len = tvlv->tvlv_hdr.len; tvlv_value = tvlv_hdr + 1; - memcpy(tvlv_value, tvlv + 1, tvlv->tvlv_hdr.len); - tvlv_value = (uint8_t *)tvlv_value + tvlv->tvlv_hdr.len; + memcpy(tvlv_value, tvlv + 1, ntohs(tvlv->tvlv_hdr.len)); + tvlv_value = (uint8_t *)tvlv_value + ntohs(tvlv->tvlv_hdr.len); } end: -- 1.8.1.5

2 1

[B.A.T.M.A.N.] pull request: batman-adv 2013-05-29
by Antonio Quartulli 29 May '13

29 May '13

Hello David, this is our pull request intended for net-next/linux-3.11. The only real behavioural change in this batch is the broadcast overhead reduction on non-wireless interfaces brought by Matthias Schiffer. The rest is made up by code cleanups and refactoring. Two highlights on the refactoring are the introduction of netdev_alloc_skb_ip_align() to avoid to deal with NET_IP_ALIGN manually and the introduction of the eth_hdr() wherever it was possible in order to improve readability. Please pull or let me know of any problem. Thanks a lot, Antonio The following changes since commit 06ecf24bdf2b7afc6c8fd13de6dba2a96dd331b6: net: Fix build warnings after mac_header and transport_header became __u16. (2013-05-28 13:15:50 -0700) are available in the git repository at: git://git.open-mesh.org/linux-merge.git tags/batman-adv-for-davem for you to fetch changes up to 6715fd3f0538e805b6a769d66823ec16b8b647ac: batman-adv: Start new development cycle (2013-05-29 02:44:56 +0200) ---------------------------------------------------------------- Included changes: - reduce broadcast overhead on non-lossy wired links - fix typos in kernel doc - use eth_hdr() when possible - use netdev_allock_skb_ip_align() and don't deal with NET_IP_ALIGN - change VID semantic in the BLA component - other minor cleanups and code refactoring ---------------------------------------------------------------- Antonio Quartulli (14): batman-adv: drop useless argument seqno in neighbor creation batman-adv: slightly improve neighbor creation debug message batman-adv: don't check the source address twice batman-adv: don't check compat version twice batman-adv: do not silently ignore wrong condition batman-adv: don't initialise batman_iv private members in hard-interface.c batman-adv: use eth_hdr() when it makes sense batman-adv: move ring_buffer helper functions in bat_iv_ogm batman-adv: move batadv_slide_own_bcast_window to bat_iv_ogm.c batman-adv: refactor batadv_tt_local_event() batman-adv: don't deal with NET_IP_ALIGN manually batman-adv: pass a 16bit long flag argument to tt_global_add() batman-adv: change VID semantic in the BLA code batman-adv: print the VID properly Linus Lüssing (1): batman-adv: Remove unnecessary INIT_HLIST_NODE() calls Marek Lindner (2): batman-adv: fix typos in kernel doc & comments batman-adv: do not print orig nodes without nc neighbors on nc table print Martin Hundebøll (1): batman-adv: Move call to batadv_nc_skb_forward() from routing.c to send.c Matthias Schiffer (2): batman-adv: split batadv_is_wifi_iface() into two functions batman-adv: send each broadcast only once on non-wireless interfaces Simon Wunderlich (1): batman-adv: Start new development cycle net/batman-adv/Makefile | 1 - net/batman-adv/bat_iv_ogm.c | 122 ++++++++++++++++++++++++--------- net/batman-adv/bridge_loop_avoidance.c | 94 +++++++++++++------------ net/batman-adv/bridge_loop_avoidance.h | 12 ++-- net/batman-adv/distributed-arp-table.c | 82 +++++++++++----------- net/batman-adv/hard-interface.c | 96 +++++++++++++++----------- net/batman-adv/icmp_socket.c | 4 +- net/batman-adv/main.h | 18 ++++- net/batman-adv/network-coding.c | 22 +++--- net/batman-adv/network-coding.h | 6 +- net/batman-adv/originator.c | 6 +- net/batman-adv/originator.h | 2 +- net/batman-adv/ring_buffer.c | 51 -------------- net/batman-adv/ring_buffer.h | 27 -------- net/batman-adv/routing.c | 64 +++++------------ net/batman-adv/routing.h | 1 - net/batman-adv/send.c | 36 ++++++---- net/batman-adv/send.h | 6 +- net/batman-adv/soft-interface.c | 6 +- net/batman-adv/translation-table.c | 74 +++++++++++++------- net/batman-adv/translation-table.h | 2 +- net/batman-adv/types.h | 6 +- net/batman-adv/unicast.c | 2 +- net/batman-adv/vis.c | 19 ++--- 24 files changed, 395 insertions(+), 364 deletions(-) delete mode 100644 net/batman-adv/ring_buffer.c delete mode 100644 net/batman-adv/ring_buffer.h

2 15

[B.A.T.M.A.N.] Basic Multicast Optimizations
by Linus Lüssing 28 May '13

28 May '13

This is the second revision of the basic multicast optimization patches. It includes one functional fix and many style fixes, thanks to Antonios feedback. Three things were not included from these suggestions: * The default mcast TVLV flags were left as it is not as easy to switch. (see answer/explanation to [PATCH 2/3]) * for_each_pmc_rcu() was not moved to a commen net header file, only a TODO was added. Will do that after this code might have landed in net. Additionally the limitation of 255 multicast listeners per originator was removed (it was more a relic of the pre-TT announcement mechanism and such limitations should be taken care of by the current TT infrastructure instead). Cheers, Linus

4 11

[B.A.T.M.A.N.] [PATCH] batman-adv: consider network coding overhead when calculating required mtu
by Marek Lindner 28 May '13

28 May '13

The module prints a warning when the MTU on the hard interface is too small to transfer payload traffic without fragmentation. The required MTU is calculated based on the encapsulation header size. If network coding is compild into the module its header size is taken into account as well. Signed-off-by: Marek Lindner <lindner_marek(a)yahoo.de> --- hard-interface.c | 19 ++++++++++--------- main.c | 25 +++++++++++++++++++++++++ main.h | 1 + soft-interface.c | 2 +- types.h | 7 ------- 5 files changed, 37 insertions(+), 17 deletions(-) diff --git a/hard-interface.c b/hard-interface.c index c343aa8..ca50dd4 100644 --- a/hard-interface.c +++ b/hard-interface.c @@ -269,9 +269,10 @@ int batadv_hardif_min_mtu(struct net_device *soft_iface) const struct batadv_priv *bat_priv = netdev_priv(soft_iface); const struct batadv_hard_iface *hard_iface; /* allow big frames if all devices are capable to do so - * (have MTU > 1500 + BAT_HEADER_LEN) + * (have MTU > 1500 + batadv_max_header_len()) */ int min_mtu = ETH_DATA_LEN; + int max_header_len = batadv_max_header_len(); if (atomic_read(&bat_priv->fragmentation)) goto out; @@ -285,8 +286,7 @@ int batadv_hardif_min_mtu(struct net_device *soft_iface) if (hard_iface->soft_iface != soft_iface) continue; - min_mtu = min_t(int, - hard_iface->net_dev->mtu - BATADV_HEADER_LEN, + min_mtu = min_t(int, hard_iface->net_dev->mtu - max_header_len, min_mtu); } rcu_read_unlock(); @@ -380,6 +380,7 @@ int batadv_hardif_enable_interface(struct batadv_hard_iface *hard_iface, struct batadv_priv *bat_priv; struct net_device *soft_iface, *master; __be16 ethertype = __constant_htons(ETH_P_BATMAN); + int max_header_len = batadv_max_header_len(); int ret; if (hard_iface->if_status != BATADV_IF_NOT_IN_USE) @@ -448,18 +449,18 @@ int batadv_hardif_enable_interface(struct batadv_hard_iface *hard_iface, hard_iface->net_dev->name); if (atomic_read(&bat_priv->fragmentation) && - hard_iface->net_dev->mtu < ETH_DATA_LEN + BATADV_HEADER_LEN) + hard_iface->net_dev->mtu < ETH_DATA_LEN + max_header_len) batadv_info(hard_iface->soft_iface, - "The MTU of interface %s is too small (%i) to handle the transport of batman-adv packets. Packets going over this interface will be fragmented on layer2 which could impact the performance. Setting the MTU to %zi would solve the problem.\n", + "The MTU of interface %s is too small (%i) to handle the transport of batman-adv packets. Packets going over this interface will be fragmented on layer2 which could impact the performance. Setting the MTU to %i would solve the problem.\n", hard_iface->net_dev->name, hard_iface->net_dev->mtu, - ETH_DATA_LEN + BATADV_HEADER_LEN); + ETH_DATA_LEN + max_header_len); if (!atomic_read(&bat_priv->fragmentation) && - hard_iface->net_dev->mtu < ETH_DATA_LEN + BATADV_HEADER_LEN) + hard_iface->net_dev->mtu < ETH_DATA_LEN + max_header_len) batadv_info(hard_iface->soft_iface, - "The MTU of interface %s is too small (%i) to handle the transport of batman-adv packets. If you experience problems getting traffic through try increasing the MTU to %zi.\n", + "The MTU of interface %s is too small (%i) to handle the transport of batman-adv packets. If you experience problems getting traffic through try increasing the MTU to %i.\n", hard_iface->net_dev->name, hard_iface->net_dev->mtu, - ETH_DATA_LEN + BATADV_HEADER_LEN); + ETH_DATA_LEN + max_header_len); if (batadv_hardif_is_iface_up(hard_iface)) batadv_hardif_activate_interface(hard_iface); diff --git a/main.c b/main.c index 7188beb..25520b1 100644 --- a/main.c +++ b/main.c @@ -252,6 +252,31 @@ out: return primary_if; } +/** + * batadv_max_header_len - calculate maximum encapsulation overhead for a + * payload packet + * + * Return the maximum encapsulation overhead in bytes. + */ +int batadv_max_header_len(void) +{ + int header_len = 0; + + header_len = max_t(int, header_len, + sizeof(struct batadv_unicast_packet)); + header_len = max_t(int, header_len, + sizeof(struct batadv_unicast_4addr_packet)); + header_len = max_t(int, header_len, + sizeof(struct batadv_bcast_packet)); + +#ifdef CONFIG_BATMAN_ADV_NC + header_len = max_t(int, header_len, + sizeof(struct batadv_coded_packet)); +#endif + + return header_len; +} + static int batadv_recv_unhandled_packet(struct sk_buff *skb, struct batadv_hard_iface *recv_if) { diff --git a/main.h b/main.h index 3db6c2e..1f0b135 100644 --- a/main.h +++ b/main.h @@ -193,6 +193,7 @@ void batadv_mesh_free(struct net_device *soft_iface); int batadv_is_my_mac(struct batadv_priv *bat_priv, const uint8_t *addr); struct batadv_hard_iface * batadv_seq_print_text_primary_if_get(struct seq_file *seq); +int batadv_max_header_len(void); int batadv_batman_skb_recv(struct sk_buff *skb, struct net_device *dev, struct packet_type *ptype, struct net_device *orig_dev); diff --git a/soft-interface.c b/soft-interface.c index e157463..991a91f 100644 --- a/soft-interface.c +++ b/soft-interface.c @@ -615,7 +615,7 @@ static void batadv_softif_init_early(struct net_device *dev) */ dev->mtu = ETH_DATA_LEN; /* reserve more space in the skbuff for our header */ - dev->hard_header_len = BATADV_HEADER_LEN; + dev->hard_header_len = batadv_max_header_len(); /* generate random address */ eth_hw_addr_random(dev); diff --git a/types.h b/types.h index c388e47..18ea0d7 100644 --- a/types.h +++ b/types.h @@ -24,13 +24,6 @@ #include "bitarray.h" #include <linux/kernel.h> -/** - * Maximum overhead for the encapsulation for a payload packet - */ -#define BATADV_HEADER_LEN \ - (ETH_HLEN + max(sizeof(struct batadv_unicast_packet), \ - sizeof(struct batadv_bcast_packet))) - #ifdef CONFIG_BATMAN_ADV_DAT /* batadv_dat_addr_t is the type used for all DHT addresses. If it is changed, -- 1.7.10.4

1 1

[B.A.T.M.A.N.] [PATCH 1/2] batman-adv: fix num_changes type to uint16_t
by Antonio Quartulli 28 May '13

28 May '13

Since the introduction of the tvlv_tt_data struct variables storing the number of TT changes are always declared as uint16_t, however some of them were not changed. Switch them al to uint16_t. Introduced by 8405301b979466610a726b7d595b6bfee5306cf7 ("batman-adv: tvlv - convert tt data sent within OGMs") Signed-off-by: Antonio Quartulli <ordex(a)autistici.org> --- translation-table.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/translation-table.c b/translation-table.c index 44e7789..f6b96cb 100644 --- a/translation-table.c +++ b/translation-table.c @@ -1486,7 +1486,7 @@ static void batadv_tt_req_list_free(struct batadv_priv *bat_priv) static void batadv_tt_save_orig_buffer(struct batadv_priv *bat_priv, struct batadv_orig_node *orig_node, const unsigned char *tt_buff, - uint8_t tt_num_changes) + uint16_t tt_num_changes) { uint16_t tt_buff_len = batadv_tt_len(tt_num_changes); @@ -2407,8 +2407,8 @@ out: static void batadv_tt_update_orig(struct batadv_priv *bat_priv, struct batadv_orig_node *orig_node, const unsigned char *tt_buff, - uint8_t tt_num_changes, - uint8_t ttvn, uint32_t tt_crc) + uint16_t tt_num_changes, uint8_t ttvn, + uint32_t tt_crc) { uint8_t orig_ttvn = (uint8_t)atomic_read(&orig_node->last_ttvn); bool full_table = true; -- 1.8.1.5

2 3

[B.A.T.M.A.N.] [PATCH 1/2] batman-adv: properly align tvlv infrastructure
by Marek Lindner 28 May '13

28 May '13

4bytes long members must start at an address multiple of 4 in order to avoid unaligned memory access. Therefore, the tvlv short type is eliminated in favor of tvlv long and that distinction removed from the code. Introduced by 0b6aa0d43767889eeda43a132cf5e73df4e63bf2 "batman-adv: tvlv - basic infrastructure" Signed-off-by: Marek Lindner <lindner_marek(a)yahoo.de> --- main.c | 91 +++++++++++++++++--------------------------------------------- packet.h | 44 +++++++----------------------- types.h | 1 - 3 files changed, 33 insertions(+), 103 deletions(-) diff --git a/main.c b/main.c index 7188beb..0194172 100644 --- a/main.c +++ b/main.c @@ -604,12 +604,8 @@ static uint16_t batadv_tvlv_container_list_size(struct batadv_priv *bat_priv) uint16_t tvlv_len = 0; hlist_for_each_entry(tvlv, &bat_priv->tvlv.container_list, list) { - if (tvlv->tvlv_hdr.long_tvlv) - tvlv_len += sizeof(struct batadv_tvlv_long); - else - tvlv_len += sizeof(struct batadv_tvlv_short); - - tvlv_len += tvlv->value_len; + tvlv_len += sizeof(struct batadv_tvlv_hdr); + tvlv_len += tvlv->tvlv_hdr.len; } return tvlv_len; @@ -678,13 +674,11 @@ void batadv_tvlv_container_register(struct batadv_priv *bat_priv, if (!tvlv_new) return; - tvlv_new->value_len = tvlv_value_len; tvlv_new->tvlv_hdr.version = version; tvlv_new->tvlv_hdr.type = type; - if (tvlv_new->value_len > 255) - tvlv_new->tvlv_hdr.long_tvlv = 1; + tvlv_new->tvlv_hdr.len = tvlv_value_len; - memcpy(tvlv_new + 1, tvlv_value, tvlv_new->value_len); + memcpy(tvlv_new + 1, tvlv_value, tvlv_new->tvlv_hdr.len); INIT_HLIST_NODE(&tvlv_new->list); atomic_set(&tvlv_new->refcount, 1); @@ -748,8 +742,7 @@ uint16_t batadv_tvlv_container_ogm_append(struct batadv_priv *bat_priv, int packet_min_len) { struct batadv_tvlv_container *tvlv; - struct batadv_tvlv_short *tvlv_short; - struct batadv_tvlv_long *tvlv_long; + struct batadv_tvlv_hdr *tvlv_hdr; uint16_t tvlv_value_len; void *tvlv_value; bool ret; @@ -769,20 +762,13 @@ uint16_t batadv_tvlv_container_ogm_append(struct batadv_priv *bat_priv, tvlv_value = (*packet_buff) + packet_min_len; hlist_for_each_entry(tvlv, &bat_priv->tvlv.container_list, list) { - if (tvlv->tvlv_hdr.long_tvlv) { - tvlv_long = tvlv_value; - tvlv_long->tvlv_hdr = tvlv->tvlv_hdr; - tvlv_long->len = htons(tvlv->value_len); - tvlv_value = tvlv_long + 1; - } else { - tvlv_short = tvlv_value; - tvlv_short->tvlv_hdr = tvlv->tvlv_hdr; - tvlv_short->len = tvlv->value_len; - tvlv_value = tvlv_short + 1; - } - - memcpy(tvlv_value, tvlv + 1, tvlv->value_len); - tvlv_value = (uint8_t *)tvlv_value + tvlv->value_len; + tvlv_hdr = tvlv_value; + tvlv_hdr->type = tvlv->tvlv_hdr.type; + tvlv_hdr->version = tvlv->tvlv_hdr.version; + tvlv_hdr->len = htons(tvlv->tvlv_hdr.len); + tvlv_value = tvlv_hdr + 1; + memcpy(tvlv_value, tvlv + 1, tvlv->tvlv_hdr.len); + tvlv_value = (uint8_t *)tvlv_value + tvlv->tvlv_hdr.len; } end: @@ -865,30 +851,16 @@ int batadv_tvlv_containers_process(struct batadv_priv *bat_priv, void *tvlv_value, uint16_t tvlv_value_len) { struct batadv_tvlv_handler *tvlv_handler; - struct batadv_tvlv_short *tvlv_short; - struct batadv_tvlv_long *tvlv_long; struct batadv_tvlv_hdr *tvlv_hdr; uint16_t tvlv_value_cont_len; uint8_t cifnotfound = BATADV_TVLV_HANDLER_OGM_CIFNOTFND; int ret = NET_RX_SUCCESS; - while (tvlv_value_len >= sizeof(*tvlv_short)) { + while (tvlv_value_len >= sizeof(*tvlv_hdr)) { tvlv_hdr = tvlv_value; - - if (tvlv_hdr->long_tvlv) { - if (sizeof(*tvlv_long) > tvlv_value_len) - break; - - tvlv_long = tvlv_value; - tvlv_value_cont_len = ntohs(tvlv_long->len); - tvlv_value = tvlv_long + 1; - tvlv_value_len -= sizeof(*tvlv_long); - } else { - tvlv_short = tvlv_value; - tvlv_value_cont_len = tvlv_short->len; - tvlv_value = tvlv_short + 1; - tvlv_value_len -= sizeof(*tvlv_short); - } + tvlv_value_cont_len = ntohs(tvlv_hdr->len); + tvlv_value = tvlv_hdr + 1; + tvlv_value_len -= sizeof(*tvlv_hdr); if (tvlv_value_cont_len > tvlv_value_len) break; @@ -1042,8 +1014,7 @@ void batadv_tvlv_unicast_send(struct batadv_priv *bat_priv, uint8_t *src, void *tvlv_value, uint16_t tvlv_value_len) { struct batadv_unicast_tvlv_packet *unicast_tvlv_packet; - struct batadv_tvlv_short *tvlv_short; - struct batadv_tvlv_long *tvlv_long; + struct batadv_tvlv_hdr *tvlv_hdr; struct batadv_orig_node *orig_node; struct sk_buff *skb = NULL; unsigned char *tvlv_buff; @@ -1058,11 +1029,7 @@ void batadv_tvlv_unicast_send(struct batadv_priv *bat_priv, uint8_t *src, if (tvlv_value_len > 255) tvlv_long_hdr = 1; - tvlv_len = tvlv_value_len; - if (tvlv_long_hdr) - tvlv_len += sizeof(*tvlv_long); - else - tvlv_len += sizeof(*tvlv_short); + tvlv_len = sizeof(*tvlv_hdr) + tvlv_value_len; skb = netdev_alloc_skb_ip_align(NULL, ETH_HLEN + hdr_len + tvlv_len); if (!skb) @@ -1076,26 +1043,16 @@ void batadv_tvlv_unicast_send(struct batadv_priv *bat_priv, uint8_t *src, unicast_tvlv_packet->header.ttl = BATADV_TTL; unicast_tvlv_packet->reserved = 0; unicast_tvlv_packet->tvlv_len = htons(tvlv_len); + unicast_tvlv_packet->align = 0; memcpy(unicast_tvlv_packet->src, src, ETH_ALEN); memcpy(unicast_tvlv_packet->dst, dst, ETH_ALEN); tvlv_buff = (unsigned char *)(unicast_tvlv_packet + 1); - - if (tvlv_long_hdr) { - tvlv_long = (struct batadv_tvlv_long *)tvlv_buff; - tvlv_long->tvlv_hdr.version = version; - tvlv_long->tvlv_hdr.type = type; - tvlv_long->tvlv_hdr.long_tvlv = 1; - tvlv_long->len = htons(tvlv_value_len); - tvlv_buff += sizeof(*tvlv_long); - } else { - tvlv_short = (struct batadv_tvlv_short *)tvlv_buff; - tvlv_short->tvlv_hdr.version = version; - tvlv_short->tvlv_hdr.type = type; - tvlv_short->len = tvlv_value_len; - tvlv_buff += sizeof(*tvlv_short); - } - + tvlv_hdr = (struct batadv_tvlv_hdr *)tvlv_buff; + tvlv_hdr->version = version; + tvlv_hdr->type = type; + tvlv_hdr->len = htons(tvlv_value_len); + tvlv_buff += sizeof(*tvlv_hdr); memcpy(tvlv_buff, tvlv_value, tvlv_value_len); if (batadv_send_skb_to_orig(skb, orig_node, NULL) != NET_XMIT_DROP) diff --git a/packet.h b/packet.h index 2a73ebd..840d73d 100644 --- a/packet.h +++ b/packet.h @@ -327,53 +327,27 @@ struct batadv_coded_packet { * @src: address of the source * @dst: address of the destination * @tvlv_len: length of tvlv data following the unicast tvlv header + * @align: 2 bytes to align the header to a 4 byte boundry */ struct batadv_unicast_tvlv_packet { struct batadv_header header; - uint8_t reserved; - uint8_t dst[ETH_ALEN]; - uint8_t src[ETH_ALEN]; - __be16 tvlv_len; + uint8_t reserved; + uint8_t dst[ETH_ALEN]; + uint8_t src[ETH_ALEN]; + __be16 tvlv_len; + uint16_t align; }; /** * struct batadv_tvlv_hdr - base tvlv header struct - * @long_tvlv: flag indicating whether this is a short tvlv container (max 256 - * bytes) or a long tvlv one (up to ETH_DATA_LEN) * @type: tvlv container type (see batadv_tvlv_type) * @version: tvlv container version + * @len: tvlv container length */ struct batadv_tvlv_hdr { -#if defined(__BIG_ENDIAN_BITFIELD) - uint8_t long_tvlv:1; - uint8_t type:7; -#elif defined(__LITTLE_ENDIAN_BITFIELD) - uint8_t type:7; - uint8_t long_tvlv:1; -#else -#error "unknown bitfield endianess" -#endif + uint8_t type; uint8_t version; -}; - -/** - * struct batadv_tvlv_short - short tvlv header struct - * @tvlv_hdr: base tvlv header - * @len: tvlv container length (limited to 255 bytes) - */ -struct batadv_tvlv_short { - struct batadv_tvlv_hdr tvlv_hdr; - uint8_t len; -}; - -/** - * struct batadv_tvlv_long - long tvlv header struct - * @tvlv_hdr: base tvlv header - * @len: tvlv container length - */ -struct batadv_tvlv_long { - struct batadv_tvlv_hdr tvlv_hdr; - __be16 len; + __be16 len; }; /** diff --git a/types.h b/types.h index c388e47..f99b508 100644 --- a/types.h +++ b/types.h @@ -976,7 +976,6 @@ struct batadv_dat_candidate { struct batadv_tvlv_container { struct hlist_node list; struct batadv_tvlv_hdr tvlv_hdr; - uint16_t value_len; atomic_t refcount; }; -- 1.7.10.4

1 3

[B.A.T.M.A.N.] [PATCH] batman-adv: correctly align the tt_tvlv_data struct
by Antonio Quartulli 28 May '13

28 May '13

4bytes long members must start at an address multiple of 4 in order to avoid unaligned memory access. Add 2 padding bytes to satisfy this requirement. Introduced by c5e7a06a29612056006e4dc01910c0b44e858583 "batman-adv: use CRC32C instead of CRC16 in TT code" Signed-off-by: Antonio Quartulli <ordex(a)autistici.org> --- packet.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/packet.h b/packet.h index 21254a4..8675b52 100644 --- a/packet.h +++ b/packet.h @@ -391,11 +391,13 @@ struct batadv_tvlv_gateway_data { * struct batadv_tvlv_tt_data - tt data propagated through the tt tvlv container * @flags: translation table flags (see batadv_tt_data_flags) * @ttvn: translation table version number + * @reserved: not used, useful for alignment purposes * @crc: crc32 checksum of the local translation table */ struct batadv_tvlv_tt_data { uint8_t flags; uint8_t ttvn; + uint16_t reserved; __be32 crc; }; -- 1.8.1.5

2 1

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

B.A.T.M.A.N