B.A.T.M.A.N

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

15 participants
4308 discussions

Re: [B.A.T.M.A.N.] batctl oops on batctl gateways call
by Fabrice Dossin 08 May '14

08 May '14

Hi, I could not reproduce this. I keep you update if I see this again. Best regards, Fabrice On 3 April 2014 10:21, Fabrice Dossin <fabrice.dossin(a)gmail.com> wrote: > (Sorry, reformatting again, leaving text mode) > > > Hi, > > Sorry, my mail client did this behind my back. I did resend the stack > here under for readability. > > For now, it has append only one time on one node. This particular node > does not have any other node in his range. I execute a "batctl > gateways" each 5 minutes for monitoring. I will let you know if it > appends again today or if I found information for reproducibility. > > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.500226] Unable to handle kernel > NULL pointer dereference at virtual address 00000022 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.508175] pgd = d4918000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.514355] [00000022] *pgd=138f9831, > *pte=00000000, *ppte=00000000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.521733] Internal error: Oops: 17 > [#5] PREEMPT ARM > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.527131] Modules linked in: tun > ipt_MASQUERADE iptable_nat nf_conntrack_ipv4 nf_defrag_ipv4 nf_nat_ipv4 > nf_nat nf_conntrack iptable_filter ip_tables x_tables ipv6 bcm2708_rng > bcm2708_wdog batman_adv(O) rtc_ds1307 qmi_wwan option usb_wwan arc4 > usbserial cdc_wdm rt2800usb rt2800lib rt2x00usb rt2x00lib mac80211 cfg80211 > crc_ccitt sr_mod cdrom leds_gpio led_class rt5572sta(O) spi_bcm2708 > i2c_bcm2708 btusb bluetooth rfkill > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.544610] CPU: 0 PID: 7261 Comm: > batctl Tainted: G D O 3.10.30+ #1 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.550263] task: d49f3180 ti: d38d0000 > task.ti: d38d0000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.556017] PC is at > batadv_orig_router_get+0x50/0x9c [batman_adv] > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.561587] LR is at > batadv_orig_router_get+0x10/0x9c [batman_adv] > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.566974] pc : [<bf35e218>] lr : > [<bf35e1d8>] psr: 20000013 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.566974] sp : d38d1ea8 ip : > 0000000a fp : dda60600 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.577350] r10: cccccccd r9 : > ddab7000 r8 : dda60600 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.582439] r7 : 00000000 r6 : > 00000000 r5 : d4826e00 r4 : 00000000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.587535] r3 : d4ac0000 r2 : > 00000000 r1 : 00000000 r0 : 00000002 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.592468] Flags: nzCv IRQs on FIQs > on Mode SVC_32 ISA ARM Segment user > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.597357] Control: 00c5387d Table: > 14918008 DAC: 00000015 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.602319] Process batctl (pid: 7261, > stack limit = 0xd38d01b0) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.607230] Stack: (0xd38d1ea8 to > 0xd38d2000) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.611903] 1ea0: > d49f3180 d49c2580 00000000 bf35a62c 000000ff bf368ed8 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.616816] 1ec0: bf368ee0 bf368eec > dc89b000 dbe3cd08 ddab7000 dc83bba0 dbd10900 00000000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.621720] 1ee0: 00100073 00000000 > d25a0aa0 00000000 00000000 00000001 d38d1f20 00000001 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.626477] 1f00: dda60600 c00f653c > d1f20790 d38d1f80 00001000 b6f18000 dda60628 00000000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.631263] 1f20: 00000000 00000000 > 00001000 dc83bba0 00000001 d25a0aa0 b6f18000 d38d0000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.636092] 1f40: d38d1f80 00001000 > d38d0000 00001000 bea8eb54 c00d7530 00000000 00000003 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.640834] 1f60: 00000000 00000000 > 00000000 d25a0aa0 b6f18000 00000000 00001000 c00d77c4 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.645450] 1f80: 00000000 00000000 > ffffffff 00916008 fbad2488 00000000 00000003 c000dd68 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.650123] 1fa0: 00000000 c000dbc0 > 00916008 fbad2488 00000003 b6f18000 00001000 00000000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.654758] 1fc0: 00916008 fbad2488 > 00000000 00000003 00000000 00021450 b6f1f000 bea8eb54 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.659333] 1fe0: 00000000 bea8e8c8 > b6d945f8 b6de52bc 60000010 00000003 00000000 00000000 > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.664121] [<bf35e218>] > (batadv_orig_router_get+0x50/0x9c [batman_adv]) from [<bf35a62c>] > (batadv_gw_client_seq_print_text+0xac/0x220 [batman_adv]) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.673531] [<bf35a62c>] > (batadv_gw_client_seq_print_text+0xac/0x220 [batman_adv]) from [<c00f653c>] > (seq_read+0x1d0/0x4cc) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.678287] [<c00f653c>] > (seq_read+0x1d0/0x4cc) from [<c00d7530>] (vfs_read+0x98/0x17c) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.683071] [<c00d7530>] > (vfs_read+0x98/0x17c) from [<c00d77c4>] (SyS_read+0x3c/0x70) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.687982] [<c00d77c4>] > (SyS_read+0x3c/0x70) from [<c000dbc0>] (ret_fast_syscall+0x0/0x30) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.692822] Code: e8bd8038 e593000c > e3500000 0afffff8 (e5902020) > Apr 2 11:45:08 pv2-1dnci kernel: [ 7432.718781] ---[ end trace > 1e5ba977f5516301 ]--- > > Regards, > Fabrice > >> >>> Hi Fabrice, >>> >>> On 02/04/14 13:50, Fabrice Dossin wrote: >>>> Dear all, >>>> >>>> Hello. >>>> I am not sure it has been notified so I post this issue here. >>>> It is my first post, so tell me if I do it bad... >>>> >>> >>> no problem, it's fine! Maybe sending the stack trace without adding new >>> lines is better, but this is fine anyway :) >>> >>> Are you able to reproduce this problem somehow? Or it happened only once >>> under no particular conditions? >>> >>> Cheers, >>> >>> >>> -- >>> Antonio Quartulli >>> >

3 2

[B.A.T.M.A.N.] [PATCHv2 maint] batman-adv: increase orig refcount when storing ref in gw_node
by Antonio Quartulli 04 May '14

04 May '14

From: Antonio Quartulli <antonio(a)open-mesh.com> A pointer to the orig_node representing a bat-gateway is stored in the gw_node->orig_node member, but the refcount for such orig_node is never increased. This leads to memory faults when gw_node->orig_node is accessed and the originator has already been freed. Fix this by increasing the refcount on gw_node creation and decreasing it on gw_node free. Signed-off-by: Antonio Quartulli <antonio(a)open-mesh.com> --- Change since v1: - properly release orig_node in case of gw_node allocation failure gateway_client.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/gateway_client.c b/gateway_client.c index d7fafc1..d5a40ab 100644 --- a/gateway_client.c +++ b/gateway_client.c @@ -42,8 +42,10 @@ static void batadv_gw_node_free_ref(struct batadv_gw_node *gw_node) { - if (atomic_dec_and_test(&gw_node->refcount)) + if (atomic_dec_and_test(&gw_node->refcount)) { + batadv_orig_node_free_ref(gw_node->orig_node); kfree_rcu(gw_node, rcu); + } } static struct batadv_gw_node * @@ -406,9 +408,14 @@ static void batadv_gw_node_add(struct batadv_priv *bat_priv, if (gateway->bandwidth_down == 0) return; + if (!atomic_inc_not_zero(&orig_node->refcount)) + return; + gw_node = kzalloc(sizeof(*gw_node), GFP_ATOMIC); - if (!gw_node) + if (!gw_node) { + batadv_orig_node_free_ref(orig_node); return; + } INIT_HLIST_NODE(&gw_node->list); gw_node->orig_node = orig_node; -- 1.8.3.2

2 1

[B.A.T.M.A.N.] [PATCHv4 maint] batman-adv: fix TT VLAN inconsistency on VLAN re-add
by Antonio Quartulli 04 May '14

04 May '14

From: Antonio Quartulli <antonio(a)open-mesh.com> When a VLAN interface (on top of batX) is removed and re-added within a short timeframe TT does not have enough time to properly cleanup. This creates an internal TT state mismatch as the newly created softif_vlan will be initialized from scratch with a TT client count of zero (even if TT entries for this VLAN still exist). The resulting TT messages are bogus due to the counter / tt client listing mismatch, thus creating inconsistencies on every node in the network To fix this issue destroy_vlan() has to not free the VLAN object immediately but it has to be kept alive until all the TT entries for this VLAN have been removed. destroy_vlan() still removes the sysfs folder so that the user has the feeling that everything went fine. If the same VLAN is re-added before the old object is free'd, then the latter is resurrected and re-used. Implement such behaviour by increasing the reference counter of a softif_vlan object every time a new local TT entry for such VLAN is created and remove the object from the list only when all the TT entries have been destroyed. Signed-off-by: Antonio Quartulli <antonio(a)open-mesh.com> --- Changes since v3: - always re-add NO_PURGE local entry on add_vid() Changes since v2: - remove cleanup_work member that is not needed anymore in this approach - reword commit subject - reword commit message (Thanks Marek!) Changes since v1: - destroy and create vlan sysfs folder within softif_vlan_destroy/create() to avoid lock troubles with soft-interface destruction and delayed jobs. soft-interface.c | 53 ++++++++++++++++++++++++++++++++++++++++------------- translation-table.c | 26 ++++++++++++++++++++++++++ types.h | 2 ++ 3 files changed, 68 insertions(+), 13 deletions(-) diff --git a/soft-interface.c b/soft-interface.c index d962363..321c6bd 100644 --- a/soft-interface.c +++ b/soft-interface.c @@ -437,10 +437,15 @@ out: * possibly free it * @softif_vlan: the vlan object to release */ -void batadv_softif_vlan_free_ref(struct batadv_softif_vlan *softif_vlan) +void batadv_softif_vlan_free_ref(struct batadv_softif_vlan *vlan) { - if (atomic_dec_and_test(&softif_vlan->refcount)) - kfree_rcu(softif_vlan, rcu); + if (atomic_dec_and_test(&vlan->refcount)) { + spin_lock_bh(&vlan->bat_priv->softif_vlan_list_lock); + hlist_del_rcu(&vlan->list); + spin_unlock_bh(&vlan->bat_priv->softif_vlan_list_lock); + + kfree_rcu(vlan, rcu); + } } /** @@ -494,6 +499,7 @@ int batadv_softif_create_vlan(struct batadv_priv *bat_priv, unsigned short vid) if (!vlan) return -ENOMEM; + vlan->bat_priv = bat_priv; vlan->vid = vid; atomic_set(&vlan->refcount, 1); @@ -505,6 +511,10 @@ int batadv_softif_create_vlan(struct batadv_priv *bat_priv, unsigned short vid) return err; } + spin_lock_bh(&bat_priv->softif_vlan_list_lock); + hlist_add_head_rcu(&vlan->list, &bat_priv->softif_vlan_list); + spin_unlock_bh(&bat_priv->softif_vlan_list_lock); + /* add a new TT local entry. This one will be marked with the NOPURGE * flag */ @@ -512,10 +522,6 @@ int batadv_softif_create_vlan(struct batadv_priv *bat_priv, unsigned short vid) bat_priv->soft_iface->dev_addr, vid, BATADV_NULL_IFINDEX, BATADV_NO_MARK); - spin_lock_bh(&bat_priv->softif_vlan_list_lock); - hlist_add_head_rcu(&vlan->list, &bat_priv->softif_vlan_list); - spin_unlock_bh(&bat_priv->softif_vlan_list_lock); - return 0; } @@ -527,18 +533,13 @@ int batadv_softif_create_vlan(struct batadv_priv *bat_priv, unsigned short vid) static void batadv_softif_destroy_vlan(struct batadv_priv *bat_priv, struct batadv_softif_vlan *vlan) { - spin_lock_bh(&bat_priv->softif_vlan_list_lock); - hlist_del_rcu(&vlan->list); - spin_unlock_bh(&bat_priv->softif_vlan_list_lock); - - batadv_sysfs_del_vlan(bat_priv, vlan); - /* explicitly remove the associated TT local entry because it is marked * with the NOPURGE flag */ batadv_tt_local_remove(bat_priv, bat_priv->soft_iface->dev_addr, vlan->vid, "vlan interface destroyed", false); + batadv_sysfs_del_vlan(bat_priv, vlan); batadv_softif_vlan_free_ref(vlan); } @@ -556,6 +557,8 @@ static int batadv_interface_add_vid(struct net_device *dev, __be16 proto, unsigned short vid) { struct batadv_priv *bat_priv = netdev_priv(dev); + struct batadv_softif_vlan *vlan; + int ret; /* only 802.1Q vlans are supported. * batman-adv does not know how to handle other types @@ -565,6 +568,30 @@ static int batadv_interface_add_vid(struct net_device *dev, __be16 proto, vid |= BATADV_VLAN_HAS_TAG; + /* if a new vlan is getting created, but it is found in the list, it + * means that it was not deleted yet. Make batadv_softif_vlan_get() + * increase the refcounter by one and return + */ + vlan = batadv_softif_vlan_get(bat_priv, vid); + if (vlan) { + /* add a new TT local entry. This one will be marked with the + * NOPURGE flag. This must be added again, even if the vlan + * object already exists, because the entry was deleted by + * kill_vid() + */ + batadv_tt_local_add(bat_priv->soft_iface, + bat_priv->soft_iface->dev_addr, vid, + BATADV_NULL_IFINDEX, BATADV_NO_MARK); + ret = 0; + if (!vlan->kobj) { + ret = batadv_sysfs_add_vlan(bat_priv->soft_iface, vlan); + if (ret) + batadv_softif_vlan_free_ref(vlan); + } + + return ret; + } + return batadv_softif_create_vlan(bat_priv, vid); } diff --git a/translation-table.c b/translation-table.c index 959dde7..b689981 100644 --- a/translation-table.c +++ b/translation-table.c @@ -485,6 +485,7 @@ bool batadv_tt_local_add(struct net_device *soft_iface, const uint8_t *addr, struct batadv_priv *bat_priv = netdev_priv(soft_iface); struct batadv_tt_local_entry *tt_local; struct batadv_tt_global_entry *tt_global; + struct batadv_softif_vlan *vlan; struct net_device *in_dev = NULL; struct hlist_head *head; struct batadv_tt_orig_list_entry *orig_entry; @@ -544,6 +545,9 @@ bool batadv_tt_local_add(struct net_device *soft_iface, const uint8_t *addr, if (!tt_local) goto out; + /* increase the refcounter of the related vlan */ + vlan = batadv_softif_vlan_get(bat_priv, vid); + batadv_dbg(BATADV_DBG_TT, bat_priv, "Creating new local tt entry: %pM (vid: %d, ttvn: %d)\n", addr, BATADV_PRINT_VID(vid), @@ -573,6 +577,7 @@ bool batadv_tt_local_add(struct net_device *soft_iface, const uint8_t *addr, if (unlikely(hash_added != 0)) { /* remove the reference for the hash */ batadv_tt_local_entry_free_ref(tt_local); + batadv_softif_vlan_free_ref(vlan); goto out; } @@ -978,6 +983,7 @@ uint16_t batadv_tt_local_remove(struct batadv_priv *bat_priv, { struct batadv_tt_local_entry *tt_local_entry; uint16_t flags, curr_flags = BATADV_NO_FLAGS; + struct batadv_softif_vlan *vlan; tt_local_entry = batadv_tt_local_hash_find(bat_priv, addr, vid); if (!tt_local_entry) @@ -1008,6 +1014,11 @@ uint16_t batadv_tt_local_remove(struct batadv_priv *bat_priv, hlist_del_rcu(&tt_local_entry->common.hash_entry); batadv_tt_local_entry_free_ref(tt_local_entry); + /* decrease the reference held for this vlan */ + vlan = batadv_softif_vlan_get(bat_priv, vid); + batadv_softif_vlan_free_ref(vlan); + batadv_softif_vlan_free_ref(vlan); + out: if (tt_local_entry) batadv_tt_local_entry_free_ref(tt_local_entry); @@ -1080,6 +1091,7 @@ static void batadv_tt_local_table_free(struct batadv_priv *bat_priv) spinlock_t *list_lock; /* protects write access to the hash lists */ struct batadv_tt_common_entry *tt_common_entry; struct batadv_tt_local_entry *tt_local; + struct batadv_softif_vlan *vlan; struct hlist_node *node_tmp; struct hlist_head *head; uint32_t i; @@ -1100,6 +1112,13 @@ static void batadv_tt_local_table_free(struct batadv_priv *bat_priv) tt_local = container_of(tt_common_entry, struct batadv_tt_local_entry, common); + + /* decrease the reference held for this vlan */ + vlan = batadv_softif_vlan_get(bat_priv, + tt_common_entry->vid); + batadv_softif_vlan_free_ref(vlan); + batadv_softif_vlan_free_ref(vlan); + batadv_tt_local_entry_free_ref(tt_local); } spin_unlock_bh(list_lock); @@ -3078,6 +3097,7 @@ static void batadv_tt_local_purge_pending_clients(struct batadv_priv *bat_priv) struct batadv_hashtable *hash = bat_priv->tt.local_hash; struct batadv_tt_common_entry *tt_common; struct batadv_tt_local_entry *tt_local; + struct batadv_softif_vlan *vlan; struct hlist_node *node_tmp; struct hlist_head *head; spinlock_t *list_lock; /* protects write access to the hash lists */ @@ -3106,6 +3126,12 @@ static void batadv_tt_local_purge_pending_clients(struct batadv_priv *bat_priv) tt_local = container_of(tt_common, struct batadv_tt_local_entry, common); + + /* decrease the reference held for this vlan */ + vlan = batadv_softif_vlan_get(bat_priv, tt_common->vid); + batadv_softif_vlan_free_ref(vlan); + batadv_softif_vlan_free_ref(vlan); + batadv_tt_local_entry_free_ref(tt_local); } spin_unlock_bh(list_lock); diff --git a/types.h b/types.h index 78370ab..c4aba2f 100644 --- a/types.h +++ b/types.h @@ -638,6 +638,7 @@ struct batadv_priv_nc { /** * struct batadv_softif_vlan - per VLAN attributes set + * @bat_priv: pointer to the mesh object * @vid: VLAN identifier * @kobj: kobject for sysfs vlan subdirectory * @ap_isolation: AP isolation state @@ -647,6 +648,7 @@ struct batadv_priv_nc { * @rcu: struct used for freeing in a RCU-safe manner */ struct batadv_softif_vlan { + struct batadv_priv *bat_priv; unsigned short vid; struct kobject *kobj; atomic_t ap_isolation; /* boolean */ -- 1.8.3.2

1 1

Re: [B.A.T.M.A.N.] [OpenWrt-Users] [Battlemesh] Batman-adv high TQ and Low throughput
by cmsv 02 May '14

02 May '14

Hello Ben Last night i decreased all the network access points to 17 dBm thinking if my problem is due too much noise from co-existent channels from same network nodes and unless i my conclusion is wrong i do not think the Ap's cause problems to each other because after the decrease in txpower; not only batman-adv TQ got lower as well as dBm and throughput. In other words there was no improvement and this is not the first time i try the sort of test. Today i reverted to normal higher values and things improved but still far from whet they were when the nodes were upgraded. I will describe the situation i have with 2 of the 3 access points. They all run ath9k on one radio with vap interfaces. AP (A) uses a 15 dbi omni Atheros AR9132 rev 2 AP (B) uses a 8 dbi omni Atheros AR7240 rev 2 Distance from each other is about 100 metres. Right now AP A is at 20 dbm and AP B at 19 dBm >From AP (B) to AP (A) i get: Station AP (B) (on adhoc0) inactive time: 20 ms rx bytes: 1087140 rx packets: 9708 tx bytes: 906 tx packets: 7 tx retries: 14 tx failed: 2 signal: -49 [-49, -58] dBm signal avg: -49 [-49, -58] dBm tx bitrate: 2.0 MBit/s rx bitrate: 1.0 MBit/s authorized: yes authenticated: yes preamble: long WMM/WME: yes MFP: no TDLS peer: no Batman-adv TQ at 91 and with a hop in the middle. Without the hop batman-adv does not even see AP (A). The hop has a penalty of 150. iperf reports: 0.0-10.3 sec 7.38 MBytes 6.02 Mbits/sec However batman-adv chooses another gateway which is further away but doe not have a hop and gets less throughput and continues to inactive time: 30 ms rx bytes: 4320939 rx packets: 40264 tx bytes: 2561 tx packets: 18 tx retries: 55 tx failed: 9 signal: -59 [-59, -68] dBm signal avg: -58 [-59, -67] dBm tx bitrate: 2.0 MBit/s rx bitrate: 1.0 MBit/s >From AP (A) to AP (B) Station AP (A) (on adhoc0) inactive time: 1110 ms rx bytes: 106752 rx packets: 629 tx bytes: 1753 tx packets: 13 tx retries: 70 tx failed: 5 signal: -91 [-94, -111, -93] dBm signal avg: -89 [-93, -107, -91] dBm tx bitrate: 6.5 MBit/s MCS 0 rx bitrate: 6.0 MBit/s authorized: yes authenticated: yes preamble: long WMM/WME: yes MFP: no TDLS peer: no Batman-adv TQ at 101 with the same hop in the middle iperf reports 7.38 MBytes 6.11 Mbits/sec And this is somewhat an okay scenario since this link was at 17 dBm from AP (B) providing 15 mbit and with a TQ of 250 average. Usually i get tx and rx bitrate around 1mbit from AP (A) side. There is a huge discrepancy here with link quality and how batman-adv is doing the choice as well as signal and signal average. Before Both sides were around -50 dBm in average but now AP (A) is always around -80 and worse. Notice that these results are only possible due to the existence of a middle node that was setup as a backup. Without the middle node AP (A) does not see AP (B). I am inclined to think about a driver issue that got into play after i played with some configurations. I see this as a possibility for the tp-link routers i have in the mesh as the dlinks seem to perform better. and continues to inactive time: 1730 ms rx bytes: 186480 rx packets: 1303 tx bytes: 2381 tx packets: 17 tx retries: 92 tx failed: 8 Some more info bellow from AP (A) Wireless kernel debug Reset: Baseband Hang: 0 Baseband Watchdog: 0 Fatal HW Error: 0 TX HW error: 0 TX Path Hang: 0 PLL RX Hang: 0 MCI Reset: 0 Ani: ANI: ENABLED ANI RESET: 175 SPUR UP: 37 SPUR DOWN: 37 OFDM WS-DET ON: 0 OFDM WS-DET OFF: 0 MRC-CCK ON: 0 MRC-CCK OFF: 0 FIR-STEP UP: 27 FIR-STEP DOWN: 49 INV LISTENTIME: 0 OFDM ERRORS: 192247 CCK ERRORS: 200380 Interrupt: RX: 2352822 RXEOL: 0 RXORN: 0 TX: 498420 TXURN: 0 MIB: 0 RXPHY: 0 RXKCM: 0 SWBA: 2634014 BMISS: 0 BNR: 0 CST: 788 GTT: 34321 TIM: 0 CABEND: 0 DTIMSYNC: 0 DTIM: 0 TSFOOR: 0 MCI: 0 GENTIMER: 0 TOTAL: 5313102 SYNC_CAUSE stats: Sync-All: 5165669 RTC-IRQ: 2863390 MAC-IRQ: 4227348 EEPROM-Illegal-Access: 2837451 APB-Timeout: 57826 PCI-Mode-Conflict: 57422 HOST1-Fatal: 43642 HOST1-Perr: 66006 TRCV-FIFO-Perr: 92477 RADM-CPL-EP: 58826 RADM-CPL-DLLP-Abort: 60567 RADM-CPL-TLP-Abort: 51149 RADM-CPL-ECRC-Err: 54651 RADM-CPL-Timeout: 89152 Local-Bus-Timeout: 69195 PM-Access: 80875 MAC-Awake: 104915 MAC-Asleep: 137511 MAC-Sleep-Access: 761892 Dma: Raw DMA Debug values: 0: 88888888 1: 00000000 2: 12249249 3: 00000000 4: 00000000 5: 00000000 6: 001d264c 7: 000281c0 Num QCU: chain_st fsp_ok fsp_st DCU: chain_st 0 0 1 1 0 1 0 1 1 0 2 0 1 1 0 3 0 1 1 0 4 0 1 1 0 5 0 1 1 0 6 0 1 1 0 7 0 1 1 0 8 0 0 1 0 9 0 0 1 0 qcu_stitch state: 0 qcu_fetch state: 0 qcu_complete state: 0 dcu_complete state: 0 dcu_arb state: 0 dcu_fp state: 0 chan_idle_dur: 147 chan_idle_dur_valid: 1 txfifo_valid_0: 0 txfifo_valid_1: 0 txfifo_dcu_num_0: 9 txfifo_dcu_num_1: 14 pcu observe: 0x2890 AR_CR: 0x4 Xmit: BE BK VI VO MPDUs Queued: 5747 0 0 7405 MPDUs Completed: 137383 0 0 4183 MPDUs XRetried: 7730 0 0 3222 Aggregates: 17209 0 0 0 AMPDUs Queued HW: 0 0 0 0 AMPDUs Queued SW: 308165 0 0 0 AMPDUs Completed: 168767 0 0 0 AMPDUs Retried: 12546 0 0 0 AMPDUs XRetried: 32 0 0 0 TXERR Filtered: 3 0 0 0 FIFO Underrun: 0 0 0 0 TXOP Exceeded: 0 0 0 0 TXTIMER Expiry: 0 0 0 0 DESC CFG Error: 0 0 0 0 DATA Underrun: 0 0 0 0 DELIM Underrun: 0 0 0 0 TX-Pkts-All: 313912 0 0 7405 TX-Bytes-All: 161389767 0 0 233291 HW-put-tx-buf: 12 0 0 146 HW-tx-start: 262235 0 0 7405 HW-tx-proc-desc: 262416 0 0 7405 TX-Failed: 0 0 0 0 Queues: (VO): qnum: 0 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 (VI): qnum: 1 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 (BE): qnum: 2 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 (BK): qnum: 3 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 (CAB): qnum: 8 qdepth: 0 ampdu-depth: 0 pending: 0 stopped: 0 On 04/30/2014 10:00 PM, Ben West wrote: > What does "iw wlan0 station dump" have to say? In particular, what is > the ratio of TX packets to TX retries (or failures)? > > I've generally seen 10% TX retries, compared to total TX packets, to be > quite ideal. A substantially higher percentage of TX retries, > especially if over 100%, usually indicates a problem with noise or > interference. > > > On Wed, Apr 30, 2014 at 6:57 PM, cmsv <cmsv(a)wirelesspt.net > <mailto:cmsv@wirelesspt.net>> wrote: > > inline reply > > On 04/24/2014 05:27 AM, Nicolás Echániz wrote: > > El 24/04/14 05:34, cmsv escribió: > >> Recently i have experienced something quite usual to my > experience and > >> that was not happening previously in the same scenario where it is > >> happening now. > >> > >> Here is the example: > >> 2 nodes at with adhoc at 2.4ghz at a distance of 415 meters > >> one uses an omni 15 dbi antenna at 18dbi and another 24 dbi grid > at 16 dbi. > >> > >> This link was previously providing up to 21 mbit being 17 mbit a the > >> lowest. Batman-adv TQ value was always around 230 being many > times above > >> 250. > >> > >> However this has recently changed the same link with 230~250 TQ > and the > >> same txpower is now outputting 1.93 Mbits/sec > >> > >> I have used iperf for bandwidth tests. Batman-adv version is 2013.4. > >> Although i have considered several types of scenarios that can be > >> causing this issue i have also tested against some which leaves me > >> puzzled in regards to the presented TQ quality and dBm values as the > >> nodes dBm are sometimes bad (ie: -80 ~ -90) > >> > >> One of the main questions is: shouldn't batman-adv TQ reflect or be > >> influenced by dBm values ? How come with -90 dBm i get batman-adv to > >> show me 230 TQ ? > >> > >> Also any feedback based on your experience regarding this type of > >> scenario is welcome. This situation happened overnight without any > >> access point new configurations or physical setup changes. > > > what's the mcast_rate value? > > > > having 230~250 TQ on a -80/-90dBm link seems like you have a very low > > mcast_rate. > > I have always used the driver default setup to decide the bitrate and > that worked up to a certain moment. Specifying it did not do much better > but today i found this message > > ath: phy0: unsupported hw bitrate detected 0x1f using 1 Mbit > > Currently: > wireless.(a)wifi-iface[0].mcast_rate=6000 > > and i tested with up to 36000 > > But i have also gotten the unsupported hw bitrate message without > setting mcast_rate > > iwinfo reports: > Tx-Power: 19 dBm Link Quality: 36/70 > Signal: -74 dBm Noise: -95 dBm > Bit Rate: 29.7 MBit/s > Encryption: WPA PSK (CCMP) > Type: nl80211 HW Mode(s): 802.11bgn > Hardware: unknown [Generic MAC80211] > TX power offset: unknown > Frequency offset: unknown > Supports VAPs: yes > > iw reports: > > Frequencies: > * 2412 MHz [1] (20.0 dBm) > * 2417 MHz [2] (20.0 dBm) > * 2422 MHz [3] (20.0 dBm) > * 2427 MHz [4] (20.0 dBm) > * 2432 MHz [5] (20.0 dBm) > * 2437 MHz [6] (20.0 dBm) > * 2442 MHz [7] (20.0 dBm) > * 2447 MHz [8] (20.0 dBm) > * 2452 MHz [9] (20.0 dBm) > * 2457 MHz [10] (20.0 dBm) > * 2462 MHz [11] (20.0 dBm) > * 2467 MHz [12] (20.0 dBm) > * 2472 MHz [13] (20.0 dBm) > * 2484 MHz [14] (disabled) > > And o notice that i am at the moment unable to set 20dBm unless i > re-flash the device. > > wireless.radio0.htmode=HT20 > > Bitrates (non-HT): > * 1.0 Mbps > * 2.0 Mbps (short preamble supported) > * 5.5 Mbps (short preamble supported) > * 11.0 Mbps (short preamble supported) > * 6.0 Mbps > * 9.0 Mbps > * 12.0 Mbps > * 18.0 Mbps > * 24.0 Mbps > * 36.0 Mbps > * 48.0 Mbps > * 54.0 Mbps > > I also tested while removing HT functionality but without better results > > Horst shows; > 91% usage for 1M rate > 3.4% usage for 6M rate > 4% usage for 36M rate > > another node shows me : > ath: phy0: unsupported hw bitrate detected 0x33 using 1 Mbit > > Chips in at the moment in use: Atheros AR9130 rev 2 and Atheros > AR9132 rev 2 > > So far it looks like a i have a bug according to atheros mailing lists > _______________________________________________ > > Battlemesh mailing list > > Battlemesh(a)ml.ninux.org <mailto:Battlemesh@ml.ninux.org> > > http://ml.ninux.org/mailman/listinfo/battlemesh > > > > _______________________________________________ > openwrt-users mailing list > openwrt-users(a)lists.openwrt.org <mailto:openwrt-users@lists.openwrt.org> > https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-users > > > > > -- > Ben West > http://gowasabi.net > ben(a)gowasabi.net <mailto:ben@gowasabi.net> > 314-246-9434

1 0

[B.A.T.M.A.N.] [PATCH 1/2] alfred: Handle EPERM on every sendto
by Sven Eckelmann 02 May '14

02 May '14

The announcements are currently only sent by alfred instances in master mode. Slave instances don't announce themself and thus don't detect the broken socket. To also catch such problems, it is useful to handle the detection of EPERM during the send in send_alfred_packet which is always called when a link-local UDP packet is send over the network. Signed-off-by: Sven Eckelmann <sven(a)open-mesh.com> --- send.c | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/send.c b/send.c index 89e9c07..dc0e428 100644 --- a/send.c +++ b/send.c @@ -32,7 +32,6 @@ int announce_master(struct globals *globals) { - ssize_t ret; struct alfred_announce_master_v0 announcement; if (globals->netsock < 0) @@ -42,14 +41,8 @@ int announce_master(struct globals *globals) announcement.header.version = ALFRED_VERSION; announcement.header.length = htons(0); - ret = send_alfred_packet(globals, &in6addr_localmcast, &announcement, - sizeof(announcement)); - if (ret == -EPERM) { - fprintf(stderr, "Error during announcement\n"); - netsock_close(globals->netsock); - globals->netsock = -1; - } - + send_alfred_packet(globals, &in6addr_localmcast, &announcement, + sizeof(announcement)); return 0; } @@ -176,6 +169,11 @@ ssize_t send_alfred_packet(struct globals *globals, const struct in6_addr *dest, ret = sendto(globals->netsock, buf, length, 0, (struct sockaddr *)&dest_addr, sizeof(struct sockaddr_in6)); + if (ret == -EPERM) { + fprintf(stderr, "Error during sent\n"); + netsock_close(globals->netsock); + globals->netsock = -1; + } return ret; } -- 1.9.2

2 2

[B.A.T.M.A.N.] [PATCHv3 1/2] batctl: add switch for setting multicast_mode
by Linus Lüssing 02 May '14

02 May '14

This patch adds a switch for the new multicast_mode setting in batman-adv. Signed-off-by: Linus Lüssing <linus.luessing(a)web.de> --- v3: fix merge conflict (ordex was quicker with his isolation mark support patch :P) README | 7 +++++++ man/batctl.8 | 4 ++++ sys.c | 6 ++++++ sys.h | 1 + 4 files changed, 18 insertions(+) diff --git a/README b/README index a4507b7..fe8eed9 100644 --- a/README +++ b/README @@ -392,6 +392,13 @@ Usage: batctl network_coding|nc [0|1] Note that network coding requires a working promiscuous mode on all interfaces. +batctl multicast_mode +===================== + +display or modify the multicast mode setting + +Usage: batctl multicast_mode|mm [0|1] + batctl aggregation ================== diff --git a/man/batctl.8 b/man/batctl.8 index b808293..f92bf3a 100644 --- a/man/batctl.8 +++ b/man/batctl.8 @@ -88,6 +88,10 @@ disable fragmentation. If no parameter is given the current network coding mode setting is displayed. Otherwise the parameter is used to enable or disable network coding. .br +.IP "\fBmulticast_mode\fP|\fBmm\fP [\fB0\fP|\fB1\fP]" +If no parameter is given the current multicast mode setting is displayed. Otherwise the parameter is used to enable or +disable multicast optimizations (i.e. disabling means always sending own multicast frames via classic flooding). +.br .IP "\fBloglevel\fP|\fBll\fP [\fBlevel\fP[ \fBlevel\fP[ \fBlevel\fP]] \fB...\fP]" If no parameter is given the current log level settings are displayed otherwise the parameter(s) is/are used to set the log level. Level 'none' disables all verbose logging. Level 'batman' enables messages related to routing / flooding / broadcasting. diff --git a/sys.c b/sys.c index fd6e107..d20696b 100644 --- a/sys.c +++ b/sys.c @@ -103,6 +103,12 @@ const struct settings_data batctl_settings[BATCTL_SETTINGS_NUM] = { .sysfs_name = "isolation_mark", .params = NULL, }, + { + .opt_long = "multicast_mode", + .opt_short = "mm", + .sysfs_name = "multicast_mode", + .params = sysfs_param_enable, + }, }; static void interface_usage(void) diff --git a/sys.h b/sys.h index 5633822..dd0bee5 100644 --- a/sys.h +++ b/sys.h @@ -47,6 +47,7 @@ enum batctl_settings_list { BATCTL_SETTINGS_FRAGMENTATION, BATCTL_SETTINGS_NETWORK_CODING, BATCTL_SETTINGS_ISOLATION_MARK, + BATCTL_SETTINGS_MULTICAST_MODE, BATCTL_SETTINGS_NUM, }; -- 1.7.10.4

2 7

[B.A.T.M.A.N.] [PATCH maint] batman-adv: fix reference counting imbalance while sending fragment
by Antonio Quartulli 02 May '14

02 May '14

From: Antonio Quartulli <antonio(a)open-mesh.com> In the new fragmentation code the batadv_frag_send_packet() function obtains a reference to the primary_if, but it does not release it upon return. This reference imbalance prevents the primary_if (and then the related netdevice) to be properly released on shut down. Fix this by releasing the primary_if in batadv_frag_send_packet(). Introduced by db56e4ecf5c2b179a0101138eacc2ec52b6ef45d ("batman-adv: Fragment and send skbs larger than mtu") Cc: Martin Hundebøll <martin(a)hundeboll.net> Signed-off-by: Antonio Quartulli <antonio(a)open-mesh.com> --- fragmentation.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/fragmentation.c b/fragmentation.c index 88df9b1..cc1cfd6 100644 --- a/fragmentation.c +++ b/fragmentation.c @@ -418,12 +418,13 @@ bool batadv_frag_send_packet(struct sk_buff *skb, struct batadv_neigh_node *neigh_node) { struct batadv_priv *bat_priv; - struct batadv_hard_iface *primary_if; + struct batadv_hard_iface *primary_if = NULL; struct batadv_frag_packet frag_header; struct sk_buff *skb_fragment; unsigned mtu = neigh_node->if_incoming->net_dev->mtu; unsigned header_size = sizeof(frag_header); unsigned max_fragment_size, max_packet_size; + bool ret = false; /* To avoid merge and refragmentation at next-hops we never send * fragments larger than BATADV_FRAG_MAX_FRAG_SIZE @@ -483,7 +484,11 @@ bool batadv_frag_send_packet(struct sk_buff *skb, skb->len + ETH_HLEN); batadv_send_skb_packet(skb, neigh_node->if_incoming, neigh_node->addr); - return true; + ret = true; + out_err: - return false; + if (primary_if) + batadv_hardif_free_ref(primary_if); + + return ret; } -- 1.8.3.2

3 2

[B.A.T.M.A.N.] [PATCHv2 next] batman-adv: fix NULL pointer dereferences
by Marek Lindner 02 May '14

02 May '14

Reported-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Marek Lindner <mareklindner(a)neomailbox.ch> --- Changes from v1: - add next branch indication multicast.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/multicast.c b/multicast.c index 8c7ca81..96b66fd 100644 --- a/multicast.c +++ b/multicast.c @@ -415,7 +415,7 @@ batadv_mcast_forw_ipv4_node_get(struct batadv_priv *bat_priv) hlist_for_each_entry_rcu(tmp_orig_node, &bat_priv->mcast.want_all_ipv4_list, mcast_want_all_ipv4_node) { - if (!atomic_inc_not_zero(&orig_node->refcount)) + if (!atomic_inc_not_zero(&tmp_orig_node->refcount)) continue; orig_node = tmp_orig_node; @@ -442,7 +442,7 @@ batadv_mcast_forw_ipv6_node_get(struct batadv_priv *bat_priv) hlist_for_each_entry_rcu(tmp_orig_node, &bat_priv->mcast.want_all_ipv6_list, mcast_want_all_ipv6_node) { - if (!atomic_inc_not_zero(&orig_node->refcount)) + if (!atomic_inc_not_zero(&tmp_orig_node->refcount)) continue; orig_node = tmp_orig_node; @@ -493,7 +493,7 @@ batadv_mcast_forw_unsnoop_node_get(struct batadv_priv *bat_priv) hlist_for_each_entry_rcu(tmp_orig_node, &bat_priv->mcast.want_all_unsnoopables_list, mcast_want_all_unsnoopables_node) { - if (!atomic_inc_not_zero(&orig_node->refcount)) + if (!atomic_inc_not_zero(&tmp_orig_node->refcount)) continue; orig_node = tmp_orig_node; -- 2.0.0.rc0

3 3

[B.A.T.M.A.N.] [PATCH maint] batman-adv: fix indirect hard_iface NULL dereference
by Marek Lindner 02 May '14

02 May '14

If hard_iface is NULL and goto out is made batadv_hardif_free_ref() doesn't check for NULL before dereferencing it to get to refcount. Reported-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Marek Lindner <mareklindner(a)neomailbox.ch> --- originator.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/originator.c b/originator.c index aa2468b..35b42f8 100644 --- a/originator.c +++ b/originator.c @@ -1074,7 +1074,8 @@ int batadv_orig_hardif_seq_print_text(struct seq_file *seq, void *offset) bat_priv->bat_algo_ops->bat_orig_print(bat_priv, seq, hard_iface); out: - batadv_hardif_free_ref(hard_iface); + if (hard_iface) + batadv_hardif_free_ref(hard_iface); return 0; } -- 2.0.0.rc0

2 2

[B.A.T.M.A.N.] [PATCH] batman-adv: fix NULL pointer dereferences
by Marek Lindner 02 May '14

02 May '14

Reported-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Marek Lindner <mareklindner(a)neomailbox.ch> --- multicast.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/multicast.c b/multicast.c index 8c7ca81..96b66fd 100644 --- a/multicast.c +++ b/multicast.c @@ -415,7 +415,7 @@ batadv_mcast_forw_ipv4_node_get(struct batadv_priv *bat_priv) hlist_for_each_entry_rcu(tmp_orig_node, &bat_priv->mcast.want_all_ipv4_list, mcast_want_all_ipv4_node) { - if (!atomic_inc_not_zero(&orig_node->refcount)) + if (!atomic_inc_not_zero(&tmp_orig_node->refcount)) continue; orig_node = tmp_orig_node; @@ -442,7 +442,7 @@ batadv_mcast_forw_ipv6_node_get(struct batadv_priv *bat_priv) hlist_for_each_entry_rcu(tmp_orig_node, &bat_priv->mcast.want_all_ipv6_list, mcast_want_all_ipv6_node) { - if (!atomic_inc_not_zero(&orig_node->refcount)) + if (!atomic_inc_not_zero(&tmp_orig_node->refcount)) continue; orig_node = tmp_orig_node; @@ -493,7 +493,7 @@ batadv_mcast_forw_unsnoop_node_get(struct batadv_priv *bat_priv) hlist_for_each_entry_rcu(tmp_orig_node, &bat_priv->mcast.want_all_unsnoopables_list, mcast_want_all_unsnoopables_node) { - if (!atomic_inc_not_zero(&orig_node->refcount)) + if (!atomic_inc_not_zero(&tmp_orig_node->refcount)) continue; orig_node = tmp_orig_node; -- 2.0.0.rc0

3 2

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