B.A.T.M.A.N

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

15 participants
4308 discussions

Internet connection problem with Batman
by faycel.benhajkhalifa＠eisox.com 30 Jan '20

30 Jan '20

Hello, I saw that you are contributing to BATMAN, may I ask you a few questions about my installation? I have several boards with OpenWRT firmware: Mips processor Kernel version 4.14.131 OpenWRT 18.06.01 BATMAN: Batctl openwrt-2018.1-1 [batman-adv: openwrt-2018.1-8] I have 8 connected mesh boards. Internet connection on the boards is not always available. When a board no longer has an internet connection, I connect to another board and try the following commands: batctl o: I find the board in the table with last-seen <1 and a quality between 191 and 233 (good quality) batctl ping does not succeed every time ping with the OpenWRT ping command does not work If I restart the board which no longer has internet, then it reconnects to the network and accesses the internet without problem (functional pings) I tried with a Static or Dynamic IP, the result is the same. The internet connection works then is interrupted and I can't find why. And sometimes without restarting the board, it finds the internet OpenWRT Config: /etc/config/network config interface 'wan' option type 'bridge' option ifname 'eth0 bat0' option dns '8.8.8.8' option stp '1' option gateway '192.168.1.1' option netmask '255.255.255.0' option ipaddr '192.168.1.101' option proto 'static' /etc/config/wireless config wifi-iface 'wmesh' option device 'radio0' option ifname 'adhoc0' option mode 'adhoc' option network 'mesh' option encryption 'psk2' option ssid 'ssid' option bssid 'bssid' option key 'password' /etc/config/batman-adv config 'mesh' 'bat0' option 'aggregated_ogms' option 'ap_isolation' option 'bonding' option 'fragmentation' option 'gw_bandwidth' option 'gw_mode' option 'gw_sel_class' option 'log_level' option 'orig_interval' option 'bridge_loop_avoidance' option 'distributed_arp_table' option 'multicast_mode' option 'network_coding' option 'hop_penalty' 0 option 'isolation_mark and I added a patch in batman-adv/patches --- a/net/batman-adv/main.h +++ b/net/batman-adv/main.h --- 2 2019-07-15 17:26:55.717093662 +0200 +++ 1 2019-07-15 17:26:46.565093715 +0200 @@ -43,7 +43,7 @@ /* purge originators after time in seconds if no valid packet comes in * -> TODO: check influence on BATADV_TQ_LOCAL_WINDOW_SIZE */ -#define BATADV_PURGE_TIMEOUT 200000 /* 200 seconds */ +#define BATADV_PURGE_TIMEOUT 10000 /* 10 seconds */ #define BATADV_TT_LOCAL_TIMEOUT 600000 /* in milliseconds */ #define BATADV_TT_CLIENT_ROAM_TIMEOUT 600000 /* in milliseconds */ #define BATADV_TT_CLIENT_TEMP_TIMEOUT 600000 /* in milliseconds */ The purpose of this patch is to make the network reconnect more quickly when a board being removed or added. Thanks for your help, I can provide more information about my network if you wish. Regards,

2 1

[PATCH] batman-adv: compat: Fix build on RHEL 8.1 and clones
by Felix Kaechele 28 Jan '20

28 Jan '20

RHEL 8.1 backported some fixes from newer kernels so we need to treat it's 4.18 kernel as if it were a newer kernel in some cases. Signed-off-by: Felix Kaechele <felix(a)kaechele.ca> --- compat-include/linux/igmp.h | 2 +- compat-include/linux/uaccess.h | 2 +- compat-include/net/addrconf.h | 2 +- compat.h | 6 ++++++ 4 files changed, 9 insertions(+), 3 deletions(-) diff --git a/compat-include/linux/igmp.h b/compat-include/linux/igmp.h index ecedc915..60606f05 100644 --- a/compat-include/linux/igmp.h +++ b/compat-include/linux/igmp.h @@ -17,7 +17,7 @@ int ip_mc_check_igmp(struct sk_buff *skb); -#elif LINUX_VERSION_CODE < KERNEL_VERSION(5, 1, 0) +#elif LINUX_VERSION_CODE < KERNEL_VERSION(5, 1, 0) && !defined(RHEL_81) static inline int batadv_ip_mc_check_igmp(struct sk_buff *skb) { diff --git a/compat-include/linux/uaccess.h b/compat-include/linux/uaccess.h index e81ed805..5a337441 100644 --- a/compat-include/linux/uaccess.h +++ b/compat-include/linux/uaccess.h @@ -13,7 +13,7 @@ #include <linux/version.h> #include_next <linux/uaccess.h> -#if LINUX_VERSION_CODE < KERNEL_VERSION(5, 0, 0) +#if LINUX_VERSION_CODE < KERNEL_VERSION(5, 0, 0) && !defined(RHEL_81) static inline int batadv_access_ok(int type, const void __user *p, unsigned long size) diff --git a/compat-include/net/addrconf.h b/compat-include/net/addrconf.h index b826ee95..5ab19043 100644 --- a/compat-include/net/addrconf.h +++ b/compat-include/net/addrconf.h @@ -17,7 +17,7 @@ int ipv6_mc_check_mld(struct sk_buff *skb); -#elif LINUX_VERSION_CODE < KERNEL_VERSION(5, 1, 0) +#elif LINUX_VERSION_CODE < KERNEL_VERSION(5, 1, 0) && !defined(RHEL_81) static inline int batadv_ipv6_mc_check_mld(struct sk_buff *skb) { diff --git a/compat.h b/compat.h index 980fca9f..7f4a5ee4 100644 --- a/compat.h +++ b/compat.h @@ -44,6 +44,12 @@ #endif /* < KERNEL_VERSION(4, 15, 0) */ +#ifdef RHEL_RELEASE_CODE +# if RHEL_RELEASE_CODE >= RHEL_RELEASE_VERSION(8, 1) +# define RHEL_81 +# endif +#endif /* RHEL_RELEASE_CODE */ + #endif /* __KERNEL__ */ #endif /* _NET_BATMAN_ADV_COMPAT_H_ */ -- 2.24.1

2 3

Re: KASAN: slab-out-of-bounds Read in bitmap_ipmac_destroy
by syzbot 25 Jan '20

25 Jan '20

syzbot has bisected this bug to: commit b9a1e627405d68d475a3c1f35e685ccfb5bbe668 Author: Cong Wang <xiyou.wangcong(a)gmail.com> Date: Thu Jul 4 00:21:13 2019 +0000 hsr: implement dellink to clean up resources bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=123fce01e00000 start commit: 4703d911 Merge tag 'xarray-5.5' of git://git.infradead.org.. git tree: upstream final crash: https://syzkaller.appspot.com/x/report.txt?x=113fce01e00000 console output: https://syzkaller.appspot.com/x/log.txt?x=163fce01e00000 kernel config: https://syzkaller.appspot.com/x/.config?x=83c00afca9cf5153 dashboard link: https://syzkaller.appspot.com/bug?extid=a85062dec5d65617cc1c syz repro: https://syzkaller.appspot.com/x/repro.syz?x=1301ed85e00000 C reproducer: https://syzkaller.appspot.com/x/repro.c?x=14b7b79ee00000 Reported-by: syzbot+a85062dec5d65617cc1c(a)syzkaller.appspotmail.com Fixes: b9a1e627405d ("hsr: implement dellink to clean up resources") For information about bisection process see: https://goo.gl/tpsmEJ#bisection

1 0

[PATCH AUTOSEL 4.4 1/9] batman-adv: Fix DAT candidate selection on little endian systems
by Sasha Levin 24 Jan '20

24 Jan '20

From: Sven Eckelmann <sven(a)narfation.org> [ Upstream commit 4cc4a1708903f404d2ca0dfde30e71e052c6cbc9 ] The distributed arp table is using a DHT to store and retrieve MAC address information for an IP address. This is done using unicast messages to selected peers. The potential peers are looked up using the IP address and the VID. While the IP address is always stored in big endian byte order, this is not the case of the VID. It can (depending on the host system) either be big endian or little endian. The host must therefore always convert it to big endian to ensure that all devices calculate the same peers for the same lookup data. Fixes: be1db4f6615b ("batman-adv: make the Distributed ARP Table vlan aware") Signed-off-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Simon Wunderlich <sw(a)simonwunderlich.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- net/batman-adv/distributed-arp-table.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/batman-adv/distributed-arp-table.c b/net/batman-adv/distributed-arp-table.c index c2dff7c6e9607..76808c5e81836 100644 --- a/net/batman-adv/distributed-arp-table.c +++ b/net/batman-adv/distributed-arp-table.c @@ -226,6 +226,7 @@ static u32 batadv_hash_dat(const void *data, u32 size) u32 hash = 0; const struct batadv_dat_entry *dat = data; const unsigned char *key; + __be16 vid; u32 i; key = (const unsigned char *)&dat->ip; @@ -235,7 +236,8 @@ static u32 batadv_hash_dat(const void *data, u32 size) hash ^= (hash >> 6); } - key = (const unsigned char *)&dat->vid; + vid = htons(dat->vid); + key = (__force const unsigned char *)&vid; for (i = 0; i < sizeof(dat->vid); i++) { hash += key[i]; hash += (hash << 10); -- 2.20.1

1 0

[PATCH AUTOSEL 4.9 01/18] batman-adv: Fix DAT candidate selection on little endian systems
by Sasha Levin 24 Jan '20

24 Jan '20

From: Sven Eckelmann <sven(a)narfation.org> [ Upstream commit 4cc4a1708903f404d2ca0dfde30e71e052c6cbc9 ] The distributed arp table is using a DHT to store and retrieve MAC address information for an IP address. This is done using unicast messages to selected peers. The potential peers are looked up using the IP address and the VID. While the IP address is always stored in big endian byte order, this is not the case of the VID. It can (depending on the host system) either be big endian or little endian. The host must therefore always convert it to big endian to ensure that all devices calculate the same peers for the same lookup data. Fixes: be1db4f6615b ("batman-adv: make the Distributed ARP Table vlan aware") Signed-off-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Simon Wunderlich <sw(a)simonwunderlich.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- net/batman-adv/distributed-arp-table.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/batman-adv/distributed-arp-table.c b/net/batman-adv/distributed-arp-table.c index df7c6a0801885..3b440b8d7c052 100644 --- a/net/batman-adv/distributed-arp-table.c +++ b/net/batman-adv/distributed-arp-table.c @@ -242,6 +242,7 @@ static u32 batadv_hash_dat(const void *data, u32 size) u32 hash = 0; const struct batadv_dat_entry *dat = data; const unsigned char *key; + __be16 vid; u32 i; key = (const unsigned char *)&dat->ip; @@ -251,7 +252,8 @@ static u32 batadv_hash_dat(const void *data, u32 size) hash ^= (hash >> 6); } - key = (const unsigned char *)&dat->vid; + vid = htons(dat->vid); + key = (__force const unsigned char *)&vid; for (i = 0; i < sizeof(dat->vid); i++) { hash += key[i]; hash += (hash << 10); -- 2.20.1

1 0

[PATCH AUTOSEL 4.14 01/32] batman-adv: Fix DAT candidate selection on little endian systems
by Sasha Levin 24 Jan '20

24 Jan '20

From: Sven Eckelmann <sven(a)narfation.org> [ Upstream commit 4cc4a1708903f404d2ca0dfde30e71e052c6cbc9 ] The distributed arp table is using a DHT to store and retrieve MAC address information for an IP address. This is done using unicast messages to selected peers. The potential peers are looked up using the IP address and the VID. While the IP address is always stored in big endian byte order, this is not the case of the VID. It can (depending on the host system) either be big endian or little endian. The host must therefore always convert it to big endian to ensure that all devices calculate the same peers for the same lookup data. Fixes: be1db4f6615b ("batman-adv: make the Distributed ARP Table vlan aware") Signed-off-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Simon Wunderlich <sw(a)simonwunderlich.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- net/batman-adv/distributed-arp-table.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/batman-adv/distributed-arp-table.c b/net/batman-adv/distributed-arp-table.c index 8d1d0fdb157e7..1519cbf70150b 100644 --- a/net/batman-adv/distributed-arp-table.c +++ b/net/batman-adv/distributed-arp-table.c @@ -243,6 +243,7 @@ static u32 batadv_hash_dat(const void *data, u32 size) u32 hash = 0; const struct batadv_dat_entry *dat = data; const unsigned char *key; + __be16 vid; u32 i; key = (const unsigned char *)&dat->ip; @@ -252,7 +253,8 @@ static u32 batadv_hash_dat(const void *data, u32 size) hash ^= (hash >> 6); } - key = (const unsigned char *)&dat->vid; + vid = htons(dat->vid); + key = (__force const unsigned char *)&vid; for (i = 0; i < sizeof(dat->vid); i++) { hash += key[i]; hash += (hash << 10); -- 2.20.1

1 0

[PATCH AUTOSEL 4.19 01/56] batman-adv: Fix DAT candidate selection on little endian systems
by Sasha Levin 24 Jan '20

24 Jan '20

From: Sven Eckelmann <sven(a)narfation.org> [ Upstream commit 4cc4a1708903f404d2ca0dfde30e71e052c6cbc9 ] The distributed arp table is using a DHT to store and retrieve MAC address information for an IP address. This is done using unicast messages to selected peers. The potential peers are looked up using the IP address and the VID. While the IP address is always stored in big endian byte order, this is not the case of the VID. It can (depending on the host system) either be big endian or little endian. The host must therefore always convert it to big endian to ensure that all devices calculate the same peers for the same lookup data. Fixes: be1db4f6615b ("batman-adv: make the Distributed ARP Table vlan aware") Signed-off-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Simon Wunderlich <sw(a)simonwunderlich.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- net/batman-adv/distributed-arp-table.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/batman-adv/distributed-arp-table.c b/net/batman-adv/distributed-arp-table.c index 2895e3b26e930..f2dc7499d2663 100644 --- a/net/batman-adv/distributed-arp-table.c +++ b/net/batman-adv/distributed-arp-table.c @@ -251,6 +251,7 @@ static u32 batadv_hash_dat(const void *data, u32 size) u32 hash = 0; const struct batadv_dat_entry *dat = data; const unsigned char *key; + __be16 vid; u32 i; key = (const unsigned char *)&dat->ip; @@ -260,7 +261,8 @@ static u32 batadv_hash_dat(const void *data, u32 size) hash ^= (hash >> 6); } - key = (const unsigned char *)&dat->vid; + vid = htons(dat->vid); + key = (__force const unsigned char *)&vid; for (i = 0; i < sizeof(dat->vid); i++) { hash += key[i]; hash += (hash << 10); -- 2.20.1

1 0

[PATCH AUTOSEL 5.4 001/107] batman-adv: Fix DAT candidate selection on little endian systems
by Sasha Levin 24 Jan '20

24 Jan '20

From: Sven Eckelmann <sven(a)narfation.org> [ Upstream commit 4cc4a1708903f404d2ca0dfde30e71e052c6cbc9 ] The distributed arp table is using a DHT to store and retrieve MAC address information for an IP address. This is done using unicast messages to selected peers. The potential peers are looked up using the IP address and the VID. While the IP address is always stored in big endian byte order, this is not the case of the VID. It can (depending on the host system) either be big endian or little endian. The host must therefore always convert it to big endian to ensure that all devices calculate the same peers for the same lookup data. Fixes: be1db4f6615b ("batman-adv: make the Distributed ARP Table vlan aware") Signed-off-by: Sven Eckelmann <sven(a)narfation.org> Signed-off-by: Simon Wunderlich <sw(a)simonwunderlich.de> Signed-off-by: Sasha Levin <sashal(a)kernel.org> --- net/batman-adv/distributed-arp-table.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/batman-adv/distributed-arp-table.c b/net/batman-adv/distributed-arp-table.c index b0af3a11d4069..ec7bf5a4a9fc7 100644 --- a/net/batman-adv/distributed-arp-table.c +++ b/net/batman-adv/distributed-arp-table.c @@ -285,6 +285,7 @@ static u32 batadv_hash_dat(const void *data, u32 size) u32 hash = 0; const struct batadv_dat_entry *dat = data; const unsigned char *key; + __be16 vid; u32 i; key = (const unsigned char *)&dat->ip; @@ -294,7 +295,8 @@ static u32 batadv_hash_dat(const void *data, u32 size) hash ^= (hash >> 6); } - key = (const unsigned char *)&dat->vid; + vid = htons(dat->vid); + key = (__force const unsigned char *)&vid; for (i = 0; i < sizeof(dat->vid); i++) { hash += key[i]; hash += (hash << 10); -- 2.20.1

1 0

Re: WARNING in __proc_create (2)
by syzbot 23 Jan '20

23 Jan '20

syzbot has bisected this bug to: commit f4b3526d83c40dd8bf5948b9d7a1b2c340f0dcc8 Author: David Howells <dhowells(a)redhat.com> Date: Thu Nov 2 15:27:48 2017 +0000 afs: Connect up the CB.ProbeUuid bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=1627a721e00000 start commit: d96d875e Merge tag 'fixes_for_v5.5-rc8' of git://git.kerne.. git tree: upstream final crash: https://syzkaller.appspot.com/x/report.txt?x=1527a721e00000 console output: https://syzkaller.appspot.com/x/log.txt?x=1127a721e00000 kernel config: https://syzkaller.appspot.com/x/.config?x=83c00afca9cf5153 dashboard link: https://syzkaller.appspot.com/bug?extid=b904ba7c947a37b4b291 syz repro: https://syzkaller.appspot.com/x/repro.syz?x=12c96185e00000 C reproducer: https://syzkaller.appspot.com/x/repro.c?x=14f859c9e00000 Reported-by: syzbot+b904ba7c947a37b4b291(a)syzkaller.appspotmail.com Fixes: f4b3526d83c4 ("afs: Connect up the CB.ProbeUuid") For information about bisection process see: https://goo.gl/tpsmEJ#bisection

1 0

Re: KASAN: use-after-free Read in bitmap_ip_destroy
by syzbot 21 Jan '20

21 Jan '20

syzbot has bisected this bug to: commit 354d0fab649d47045517cf7cae03d653a4dcb3b8 Author: Peng Li <lipeng321(a)huawei.com> Date: Thu Jul 4 14:04:26 2019 +0000 net: hns3: add default value for tc_size and tc_offset bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=15cc0685e00000 start commit: 8f8972a3 Merge tag 'mtd/fixes-for-5.5-rc7' of git://git.ke.. git tree: upstream final crash: https://syzkaller.appspot.com/x/report.txt?x=17cc0685e00000 console output: https://syzkaller.appspot.com/x/log.txt?x=13cc0685e00000 kernel config: https://syzkaller.appspot.com/x/.config?x=cfbb8fa33f49f9f3 dashboard link: https://syzkaller.appspot.com/bug?extid=8b5f151de2f35100bbc5 syz repro: https://syzkaller.appspot.com/x/repro.syz?x=12e22559e00000 C reproducer: https://syzkaller.appspot.com/x/repro.c?x=16056faee00000 Reported-by: syzbot+8b5f151de2f35100bbc5(a)syzkaller.appspotmail.com Fixes: 354d0fab649d ("net: hns3: add default value for tc_size and tc_offset") For information about bisection process see: https://goo.gl/tpsmEJ#bisection

1 0

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