Re: [PATCH v6 12/16] net: tip: fix a couple kernel-doc markups

2021-01-14 Thread Jon Maloy
* @dnode: address of destination node Acked-by: Jon Maloy

Re: [net] tipc: fix NULL pointer dereference in tipc_named_rcv

2020-10-09 Thread Jon Maloy
;lock#2); lock(&tn->nametbl_lock); lock(&n->lock#2); lock(&tn->nametbl_lock); *** DEADLOCK *** Regards, Hoang Ok. So although your solution is not optimal, we know it is safe. Again: Acked-by: Jon Maloy -Original Message- F

Re: [net] tipc: fix NULL pointer dereference in tipc_named_rcv

2020-10-08 Thread Jon Maloy
On 10/8/20 1:25 PM, Jakub Kicinski wrote: On Thu, 8 Oct 2020 14:31:56 +0700 Hoang Huu Le wrote: diff --git a/net/tipc/name_distr.c b/net/tipc/name_distr.c index 2f9c148f17e2..fe4edce459ad 100644 --- a/net/tipc/name_distr.c +++ b/net/tipc/name_distr.c @@ -327,8 +327,13 @@ static struct sk_buf

Re: [PATCH net-next 18/20] net: tipc: kerneldoc fixes

2020-07-13 Thread Jon Maloy
On 7/12/20 7:15 PM, Andrew Lunn wrote: Simple fixes which require no deep knowledge of the code. Cc: Jon Maloy Cc: Ying Xue Signed-off-by: Andrew Lunn --- net/tipc/bearer.c| 2 +- net/tipc/discover.c | 5 ++--- net/tipc/link.c | 6 +++--- net/tipc/msg.c | 2 +- net

Re: [iproute2-next] tipc: fixed a compile warning in tipc/link.c

2020-07-09 Thread Jon Maloy
L); - print_uint(PRINT_ANY, "ratio", " ratio:%u%\n", + print_uint(PRINT_ANY, "ratio", " ratio:%u\n", mnl_attr_get_u32(props[prop_ratio])); break; default: Acked-by: Jon Maloy

Re: [net-next] tipc: update a binding service via broadcast

2020-06-07 Thread Jon Maloy
we add another 'is_not_legacy' bit to all new messages so that those can be distinguished from legacy messages and the latter delivered directly. Signed-off-by: Hoang Huu Le Acked-by: Jon Maloy --- net/tipc/bcast.c | 6 +-- net/tipc/bcast.h | 4 +- net/tipc/l

Re: [PATCH net] tipc: block BH before using dst_cache

2020-05-22 Thread Jon Maloy
On 5/22/20 4:10 PM, Eric Dumazet wrote: On 5/22/20 12:47 PM, Jon Maloy wrote: On 5/22/20 11:57 AM, Eric Dumazet wrote: On 5/22/20 8:01 AM, Jon Maloy wrote: On 5/22/20 2:18 AM, Xin Long wrote: On Fri, May 22, 2020 at 1:55 PM Eric Dumazet wrote: Resend to the list in non HTML form On

Re: [PATCH net] tipc: block BH before using dst_cache

2020-05-22 Thread Jon Maloy
On 5/22/20 11:57 AM, Eric Dumazet wrote: On 5/22/20 8:01 AM, Jon Maloy wrote: On 5/22/20 2:18 AM, Xin Long wrote: On Fri, May 22, 2020 at 1:55 PM Eric Dumazet wrote: Resend to the list in non HTML form On Thu, May 21, 2020 at 10:53 PM Eric Dumazet wrote: On Thu, May 21, 2020 at 10

Re: [PATCH net] tipc: block BH before using dst_cache

2020-05-22 Thread Jon Maloy
0033:0x45ca29 Fixes: e9c1a793210f ("tipc: add dst_cache support for udp media") Cc: Xin Long Cc: Jon Maloy Signed-off-by: Eric Dumazet Reported-by: syzbot --- net/tipc/udp_media.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/net/tipc/udp_media.c b/net/t

RE: [patch net-next] net: tipc: prepare attrs in __tipc_nl_compat_dumpit()

2019-10-08 Thread Jon Maloy
Acked. Thanks Jiri. ///jon > -Original Message- > From: Jiri Pirko > Sent: 8-Oct-19 07:02 > To: netdev@vger.kernel.org > Cc: da...@davemloft.net; Jon Maloy ; > ying@windriver.com; johannes.b...@intel.com; mkube...@suse.cz; > ml...@mellanox.com > Subject: [p

[net-next v2 1/1] tipc: clean up skb list lock handling on send path

2019-08-15 Thread Jon Maloy
s where we incorrectly access the spinlock on the send/error path. This patch also reverts commit d12cffe9329f ("tipc: ensure head->lock is initialised") which has now become redundant. CC: Eric Dumazet Reported-by: Chris Packham Acked-by: Ying Xue Signed-off-by: Jon Maloy ---

RE: [net-next 1/1] tipc: clean up skb list lock handling on send path

2019-08-15 Thread Jon Maloy
> -Original Message- > From: netdev-ow...@vger.kernel.org On > Behalf Of Xin Long > Sent: 15-Aug-19 01:58 > To: Jon Maloy > Cc: da...@davemloft.net; netdev@vger.kernel.org; Tung Quang Nguyen > ; Hoang Huu Le > ; shu...@redhat.com; ying xue > ; eduma...@

[net-next 1/1] tipc: clean up skb list lock handling on send path

2019-08-14 Thread Jon Maloy
s where we incorrectly access the spinlock on the send/error path. This patch also reverts commit d12cffe9329f ("tipc: ensure head->lock is initialised") which has now become redundant. CC: Eric Dumazet Reported-by: Chris Packham Acked-by: Ying Xue Signed-off-by: Jon Maloy --- ne

[net 1/1] tipc: fix unitilized skb list crash

2019-07-30 Thread Jon Maloy
ther down in tipc_sk_filter_rcv() as it should be. Reported-by: Li Shuang Signed-off-by: Jon Maloy --- net/tipc/socket.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/net/tipc/socket.c b/net/tipc/socket.c index dd8537f..83ae41d 100644 --- a/net/tipc/socket.c +++ b/net/tipc/socket

[net-next 1/1] tipc: reduce risk of wakeup queue starvation

2019-07-30 Thread Jon Maloy
e risk of user starvation during link congestion") Reported-by: Tung Nguyen Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/link.c | 29 + 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/net/tipc/link.c b/net/tipc/link.c index 2c27477..dd3155

[net 1/1] tipc: initialize 'validated' field of received packets

2019-07-17 Thread Jon Maloy
t the packet is erroneously perceived as already validated by TIPC. We fix this by initializing the said field to 'false' before performing the initial validation. Signed-off-by: Jon Maloy --- net/tipc/node.c | 1 + 1 file changed, 1 insertion(+) diff --git a/net/tipc/node.c b/net/

[net-next 1/1] tipc: embed jiffies in macro TIPC_BC_RETR_LIM

2019-06-28 Thread Jon Maloy
The macro TIPC_BC_RETR_LIM is always used in combination with 'jiffies', so we can just as well perform the addition in the macro itself. This way, we get a few shorter code lines and one less line break. Signed-off-by: Jon Maloy --- net/tipc/link.c | 9 - 1 file changed, 4

[net-next 1/1] tipc: rename function msg_get_wrapped() to msg_inner_hdr()

2019-06-25 Thread Jon Maloy
We rename the inline function msg_get_wrapped() to the more comprehensible msg_inner_hdr(). Signed-off-by: Jon Maloy --- net/tipc/bcast.c | 4 ++-- net/tipc/link.c | 2 +- net/tipc/msg.h | 4 ++-- net/tipc/node.c | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/net

[net-next 1/1] tipc: eliminate unnecessary skb expansion during retransmission

2019-06-25 Thread Jon Maloy
We increase the allocated headroom for the buffer copies to be retransmitted. This eliminates the need for the lower stack levels (UDP/IP/L2) to expand the headroom in order to add their own headers. Signed-off-by: Jon Maloy --- net/tipc/link.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion

[net-next 1/1] tipc: simplify stale link failure criteria

2019-06-25 Thread Jon Maloy
e original, additional criteria of 99 failed retransmissions is now redundant, and may in some cases lead to failure detection times in the order of minutes instead of the expected 1.5 sec link tolerance value. We now remove this criteria altogether. Acked-by: Ying Xue Signed-off-by: Jon Maloy ---

RE: [net-next v2] tipc: add loopback device tracking

2019-06-24 Thread Jon Maloy
s and > > send the clones to the loopback interface. The clones are dropped at > > reception and have no functional role except making the traffic visible. > > > > The feature is turned on/off by enabling/disabling the loopback "bearer" > > "eth:lo"

RE: [PATCH net] tipc: add dst_cache support for udp media

2019-06-20 Thread Jon Maloy
Acked-by: Jon Maloy > -Original Message- > From: netdev-ow...@vger.kernel.org On > Behalf Of Xin Long > Sent: 20-Jun-19 07:04 > To: network dev > Cc: da...@davemloft.net; Jon Maloy ; Ying Xue > ; tipc-discuss...@lists.sourceforge.net; Paolo > Abeni > Sub

RE: [PATCH net] tipc: change to use register_pernet_device

2019-06-20 Thread Jon Maloy
Acked-by: Jon Maloy > -Original Message- > From: netdev-ow...@vger.kernel.org On > Behalf Of Xin Long > Sent: 20-Jun-19 06:39 > To: network dev > Cc: da...@davemloft.net; Jon Maloy ; Ying Xue > ; tipc-discuss...@lists.sourceforge.net > Subject: [PATCH

RE: WARNING: locking bug in icmp_send

2019-03-25 Thread Jon Maloy
Yet another duplicate of syzbot+a25307ad099309f1c...@syzkaller.appspotmail.com A fix has been posted. ///jon > -Original Message- > From: netdev-ow...@vger.kernel.org > On Behalf Of syzbot > Sent: 23-Mar-19 19:03 > To: da...@davemloft.net; Jon Maloy ; > kuz...@ms

[net 1/1] tipc: fix RDM/DGRAM connect() regression

2019-03-04 Thread Jon Maloy
From: Erik Hugne Fix regression bug introduced in commit 365ad353c256 ("tipc: reduce risk of user starvation during link congestion") Only signal -EDESTADDRREQ for RDM/DGRAM if we don't have a cached sockaddr. Signed-off-by: Erik Hugne Signed-off-by: Jon Maloy --- net/ti

RE: [Patch net] tipc: check group dests after tipc_wait_for_cond()

2018-12-17 Thread Jon Maloy
Acked-by: Jon Maloy Thank you, Cong. ///jon > -Original Message- > From: Cong Wang > Sent: 17-Dec-18 02:25 > To: netdev@vger.kernel.org > Cc: Cong Wang ; Ying Xue > ; Jon Maloy > Subject: [Patch net] tipc: check group dests after tipc_wait_for_cond() &

RE: KMSAN: uninit-value in __inet6_bind

2018-12-14 Thread Jon Maloy
> -Original Message- > From: netdev-ow...@vger.kernel.org > On Behalf Of Eric Dumazet > Sent: 14-Dec-18 10:15 > To: Jon Maloy ; Cong Wang > ; Dmitry Vyukov > Cc: syzbot+c56449ed3652e6720...@syzkaller.appspotmail.com; Ying Xue > ; tipc-discuss...@lists.sourcef

RE: KMSAN: uninit-value in __inet6_bind

2018-12-14 Thread Jon Maloy
> -Original Message- > From: Cong Wang > Sent: 12-Dec-18 01:17 > To: Dmitry Vyukov > Cc: syzbot+c56449ed3652e6720...@syzkaller.appspotmail.com; Jon Maloy > ; Ying Xue ; tipc- > discuss...@lists.sourceforge.net; David Miller ; > Alexey Kuznetsov ; LKML ker...

RE: KASAN: use-after-free Read in tipc_group_cong

2018-12-13 Thread Jon Maloy
> -Original Message- > From: Dmitry Vyukov > Sent: 13-Dec-18 04:47 > To: Jon Maloy > Cc: syzbot+9845fed98688e01f4...@syzkaller.appspotmail.com; David Miller > ; LKML ; netdev > ; syzkaller-bugs b...@googlegroups.com>; tipc-discuss...@lists.sourceforge.net; Y

RE: general protection fault in __ip_append_data

2018-12-12 Thread Jon Maloy
> -Original Message- > From: Dmitry Vyukov > Sent: 12-Dec-18 06:03 > To: syzbot+aab62b9c7b12e7c6a...@syzkaller.appspotmail.com; Jon Maloy > ; Ying Xue ; David > Miller ; tipc-discuss...@lists.sourceforge.net > Cc: Alexey Kuznetsov ; LKML ker...@vger.kernel.org>;

RE: KASAN: use-after-free Read in tipc_group_cong

2018-12-12 Thread Jon Maloy
> -Original Message- > From: syzbot > Sent: 12-Dec-18 06:11 > To: da...@davemloft.net; Jon Maloy ; linux- > ker...@vger.kernel.org; netdev@vger.kernel.org; syzkaller- > b...@googlegroups.com; tipc-discuss...@lists.sourceforge.net; > ying@windriver.com > S

RE: [Patch net] tipc: compare remote and local protocols in tipc_udp_enable()

2018-12-12 Thread Jon Maloy
> -Original Message- > From: Cong Wang > Sent: 10-Dec-18 18:24 > To: netdev@vger.kernel.org > Cc: Cong Wang ; Ying Xue > ; Jon Maloy > Subject: [Patch net] tipc: compare remote and local protocols in > tipc_udp_enable() > > When TIPC_NLA_UDP_REMOT

RE: [Patch net] tipc: fix a double kfree_skb()

2018-12-10 Thread Jon Maloy
Acked. Thank you for both your quick fixes, Cong. ///jon > -Original Message- > From: Cong Wang > Sent: 10-Dec-18 15:46 > To: netdev@vger.kernel.org > Cc: Cong Wang ; Ying Xue > ; Jon Maloy > Subject: [Patch net] tipc: fix a double kfree_skb() > > tipc_udp

[net 1/1] tipc: fix lockdep warning during node delete

2018-11-26 Thread Jon Maloy
timer cleanup function use spin_trylock() instead of just spin_lock(), and when it fails to grab the lock it just returns so that the timer handler can terminate its execution. This is safe to do, since tipc_node_stop() anyway is about to delete both the timer and the node instance. Fixes: 6a939f36

RE: [PATCH net] tipc: eliminate possible recursive locking detected by LOCKDEP

2018-10-11 Thread Jon Maloy
Acked-by: Jon Maloy ///jon > -Original Message- > From: Ying Xue > Sent: October 11, 2018 7:58 AM > To: Jon Maloy ; dvyu...@google.com > Cc: da...@davemloft.net; parthasarathy.bhuvara...@ericsson.com; > netdev@vger.kernel.org; linux-ker...@vger.kernel.or

RE: net/tipc: recursive locking in tipc_link_reset

2018-10-11 Thread Jon Maloy
Hi Dmitry, Yes, we are aware of this, the kernel test robot warned us about this a few days ago. I am looking into it. ///jon > -Original Message- > From: Dmitry Vyukov > Sent: October 11, 2018 3:55 AM > To: parthasarathy.bhuvara...@ericsson.com; Jon Maloy > ; David Mi

RE: [PATCH net-next] tipc: eliminate complaint of KMSAN uninit-value in tipc_conn_rcv_sub

2018-05-21 Thread Jon Maloy
> -Original Message- > From: netdev-ow...@vger.kernel.org > On Behalf Of David Miller > Sent: Saturday, May 19, 2018 23:00 > To: ying@windriver.com > Cc: netdev@vger.kernel.org; Jon Maloy ; > syzkaller-b...@googlegroups.com; tipc-discuss...@lists.sourcefo

[iproute2-next v3 1/1] tipc: fixed node and name table listings

2018-05-17 Thread Jon Maloy
from David Ahern v3: Fixed leaking socket as per comment from David Ahern Signed-off-by: Jon Maloy --- tipc/misc.c | 20 tipc/misc.h | 1 + tipc/nametable.c | 18 ++ tipc/node.c | 19 --- tipc/peer.c | 4 5 files ch

[iproute2-next v2 1/1] tipc: fixed node and name table listings

2018-05-15 Thread Jon Maloy
David Ahern Signed-off-by: Jon Maloy --- tipc/misc.c | 18 ++ tipc/misc.h | 1 + tipc/nametable.c | 18 ++ tipc/node.c | 19 --- tipc/peer.c | 4 5 files changed, 41 insertions(+), 19 deletions(-) diff --git a/tipc/

RE: [PATCH net] tipc: fix one byte leak in tipc_sk_set_orig_addr()

2018-05-09 Thread Jon Maloy
Acked-by: Jon Maloy Thank you Eric. > -Original Message- > From: Eric Dumazet [mailto:eduma...@google.com] > Sent: Wednesday, May 09, 2018 09:50 > To: David S . Miller > Cc: netdev ; Eric Dumazet > ; Eric Dumazet ; Jon > Maloy ; Ying Xue > Subject: [PATCH net]

[net-next 1/1] tipc: clean up removal of binding table items

2018-05-08 Thread Jon Maloy
kup the tipc_service -> service_range -> publication items in the correct hierarchical order. In this commit we try to make this right, and as a side effect obtain some code simplification. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/name_table.c | 103 ++-

[iproute2-next 1/1] tipc: fixed node and name table listings

2018-05-07 Thread Jon Maloy
ormer potentially is much longer than the latter, we make room for it by eliminating the (to the user) irrelevant publication key. We also reorder some of the columns so that the node id comes last, since this looks nicer and more logical. Signed-off-by: Jon Maloy --- tipc/misc.c | 18

RE: [PATCH net-next] flow_dissector: do not rely on implicit casts

2018-05-07 Thread Jon Maloy
Acked-by: Jon Maloy > -Original Message- > From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of Paolo Abeni > Sent: Monday, May 07, 2018 06:06 > To: netdev@vger.kernel.org > Cc: David S. Miller > Subject: [PATCH net-next] flow_d

RE: KMSAN: uninit-value in strcmp

2018-05-03 Thread Jon Maloy
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Thursday, May 03, 2018 15:22 > To: syzbot+df0257c92ffd4fcc5...@syzkaller.appspotmail.com > Cc: Jon Maloy ; linux-ker...@vger.kernel.org; > netdev@vger.kernel.org; syzkaller-b...@goog

RE: [PATCH] tipc: fix a potential missing-check bug

2018-05-01 Thread Jon Maloy
> -Original Message- > From: Wenwen Wang [mailto:wang6...@umn.edu] > Sent: Tuesday, May 01, 2018 00:26 > To: Wenwen Wang > Cc: Kangjie Lu ; Jon Maloy ; Ying > Xue ; David S. Miller ; > open list:TIPC NETWORK LAYER ; open list:TIPC > NETWORK LAYER ; open list

[net-next 1/1] tipc: introduce ioctl for fetching node identity

2018-04-25 Thread Jon Maloy
particularly useful when we extend some of the commands in the 'tipc' tool, but we also expect regular user applications to need this feature. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- include/uapi/linux/tipc.h | 12 net/tipc/node.c | 21 +

[net 1/1] tipc: fix bug in function tipc_nl_node_dump_monitor

2018-04-25 Thread Jon Maloy
he missing part. Fixes: 36a50a989ee8 ("tipc: fix infinite loop when dumping link monitor summary") Signed-off-by: Jon Maloy --- net/tipc/node.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/tipc/node.c b/net/tipc/node.c index 6f98b56..baaf93f 100644 --- a/n

[net 1/1] tipc: fix infinite loop when dumping link monitor summary

2018-04-17 Thread Jon Maloy
. This solution also caters for the case where there may be gaps in the bearer array. Signed-off-by: Tung Nguyen Signed-off-by: Jon Maloy --- net/tipc/monitor.c | 2 +- net/tipc/node.c| 11 --- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/net/tipc/monitor.c b/net/

[net 1/1] tipc: fix use-after-free in tipc_nametbl_stop

2018-04-17 Thread Jon Maloy
e delete of the service range item outside the said function. Instead, we now let the two functions calling it test if the list is empty and perform the removal when that is the case. Reported-by: syzbot+d64b64afc55660106...@syzkaller.appspotmail.com Signed-off-by: Jon Maloy --- net/tipc/na

RE: [PATCH net 0/2] tipc: Better check user provided attributes

2018-04-16 Thread Jon Maloy
Acked-by: Jon Maloy Thank you, Eric. > -Original Message- > From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of Eric Dumazet > Sent: Monday, April 16, 2018 11:30 > To: David S . Miller > Cc: netdev ; Eric Dumazet > ; Eric Dumaz

[net 1/1] tipc: fix missing initializer in tipc_sendmsg()

2018-04-11 Thread Jon Maloy
the named function if the given address type is illegal. Reported-by: syzbot+b0975ce9355b347c1...@syzkaller.appspotmail.com Signed-off-by: Jon Maloy --- net/tipc/socket.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/net/tipc/socket.c b/net/tipc/socket.c index 1fd1c8b..252a5

[net 1/1] tipc: fix unbalanced reference counter

2018-04-11 Thread Jon Maloy
plies back to the commit shown below. Given the low risk of this happening we believe this to be sufficient. Fixes: commit 218527fe27ad ("tipc: replace name table service range array with rb tree") Reported-by: syzbot+aa245f26d42b8305d...@syzkaller.appspotmail.com Signed-off-by: Jon

RE: [PATCH v3] net: tipc: Replace GFP_ATOMIC with GFP_KERNEL in tipc_mon_create

2018-04-11 Thread Jon Maloy
> -Original Message- > From: Ying Xue [mailto:ying@windriver.com] > Sent: Wednesday, April 11, 2018 06:27 > To: Jia-Ju Bai ; Jon Maloy > ; da...@davemloft.net > Cc: netdev@vger.kernel.org; tipc-discuss...@lists.sourceforge.net; linux- > ker...@vger.kernel.org >

RE: general protection fault in tipc_nametbl_unsubscribe

2018-04-03 Thread Jon Maloy
#syz dup: general protection fault in __list_del_entry_valid (3) > -Original Message- > From: syzbot > [mailto:syzbot+4859fe19555ea87c4...@syzkaller.appspotmail.com] > Sent: Monday, April 02, 2018 02:01 > To: da...@davemloft.net; Jon Maloy ; linux- > ker...@vger.

[net-next 1/1] tipc: Fix missing list initializations in struct tipc_subscription

2018-04-03 Thread Jon Maloy
ation functions"), which is still in net-next. We fix this for that commit by initializing the two lists properly. Fixes: 242e82cc95f6 ("tipc: collapse subscription creation functions") Reported-by: syzbot+0bb443b74ce09197e...@syzkaller.appspotmail.com Signed-off-by: Jon Maloy --- net/tip

RE: [iproute2-next 0/2] tipc: changes to addressing structure

2018-03-29 Thread Jon Maloy
> -Original Message- > From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of David Ahern > Sent: Thursday, March 29, 2018 13:59 > To: Jon Maloy ; da...@davemloft.net; > netdev@vger.kernel.org > Cc: Mohan Krishna Ghanta Krishn

[net-next v2 2/5] tipc: refactor name table translate function

2018-03-29 Thread Jon Maloy
The function tipc_nametbl_translate() function is ugly and hard to follow. This can be improved somewhat by introducing a stack variable for holding the publication list to be used and re-ordering the if- clauses for selection of algorithm. Signed-off-by: Jon Maloy --- net/tipc/name_table.c

[net-next v2 5/5] tipc: avoid possible string overflow

2018-03-29 Thread Jon Maloy
safe side we also replace the sprintf() call with snprintf(). Fixes: 25b0b9c4e835 ("tipc: handle collisions of 32-bit node address hash values") Reported-by: Arnd Bergmann Signed-off-by: Jon Maloy --- include/uapi/linux/tipc.h | 2 +- net/tipc/link.c | 3 ++- 2 files changed, 3

[net-next v2 1/5] tipc: replace name table service range array with rb tree

2018-03-29 Thread Jon Maloy
improving cache hit rate. Signed-off-by: Jon Maloy --- net/tipc/core.h |1 + net/tipc/link.c |2 +- net/tipc/name_table.c | 1032 ++--- net/tipc/name_table.h |2 +- net/tipc/node.c |4 +- net/tipc/subscr.h |4

[net-next v2 3/5] tipc: permit overlapping service ranges in name table

2018-03-29 Thread Jon Maloy
ication/withdrawal item from a peer node never will be rejected, and make it possible to eliminate the problematic backlog functionality we currently have for handling such cases. Signed-off-by: Jon Maloy --- net/tipc/name_distr.c | 90 +--

[net-next v2 4/5] tipc: tipc: rename address types in user api

2018-03-29 Thread Jon Maloy
ed-off-by: Jon Maloy --- include/uapi/linux/tipc.h | 57 +++ 1 file changed, 33 insertions(+), 24 deletions(-) diff --git a/include/uapi/linux/tipc.h b/include/uapi/linux/tipc.h index 4ac9f1f..156224a 100644 --- a/include/uapi/linux/tipc.h +++ b/inc

[net-next v2 0/5] tipc: slim down name table

2018-03-29 Thread Jon Maloy
replaced a few more macros to get a consistent terminology in the API. #5: Added new commit to fix a potential string overflow bug (it is still only in net-next) reported by Arnd Bergmann Jon Maloy (5): tipc: replace name table service range array with rb tree tipc: ref

[iproute2-next 1/2] tipc: introduce command for handling a new 128-bit node identity

2018-03-28 Thread Jon Maloy
same operations. For compatibility the old 'netid' commands are retained, -we just remove them from the help texts. Acked-by: GhantaKrishnamurthy MohanKrishna Signed-off-by: Jon Maloy --- include/uapi/linux/tipc_netlink.h | 2 + tipc/misc.c | 78 ++

[iproute2-next 0/2] tipc: changes to addressing structure

2018-03-28 Thread Jon Maloy
1: We introduce ability to set/get 128-bit node identities 2: We rename 'net id' to 'cluster id' in the command API, of course in a compatible way. 3: We print out all 32-bit node addresses as an integer in hex format, i.e., we remove the assumption about an internal s

[iproute2-next 2/2] tipc: change node address printout formats

2018-03-28 Thread Jon Maloy
Since a node address now per definition is only an unstructured 32-bit integer it makes no sense print it out as a structured string. In this commit, we replace all occurrences of "" printouts with just an "%x". Acked-by: GhantaKrishnamurthy MohanKrishna Signed-off-by:

[net-next 4/4] tipc: tipc: rename address types in user api

2018-03-28 Thread Jon Maloy
uct tipc_socket_addr struct tipc_name -> struct tipc_service_addr struct tipc_name_seq -> struct tipc_service_range For compatibility, we add macros that map the old names to the new ones. Signed-off-by: Jon Maloy --- include/uapi/linux/tipc.h | 32 ++-- 1 fil

[net-next 3/4] tipc: permit overlapping service ranges in name table

2018-03-28 Thread Jon Maloy
ication/withdrawal item from a peer node never will be rejected, and make it possible to eliminate the problematic backlog functionality we currently have for handling such cases. Signed-off-by: Jon Maloy --- net/tipc/name_distr.c | 90 +--

[net-next 2/4] tipc: refactor name table translate function

2018-03-28 Thread Jon Maloy
The function tipc_nametbl_translate() function is ugly and hard to follow. This can be improved somewhat by introducing a stack variable for holding the publication list to be used and re-ordering the if- clauses for selection of algorithm. Signed-off-by: Jon Maloy --- net/tipc/name_table.c

[net-next 1/4] tipc: replace name table service range array with rb tree

2018-03-28 Thread Jon Maloy
improving cache hit rate. Signed-off-by: Jon Maloy --- net/tipc/core.h |1 + net/tipc/link.c |2 +- net/tipc/name_table.c | 1032 ++--- net/tipc/name_table.h |2 +- net/tipc/node.c |4 +- net/tipc/subscr.h |4

[net-next 0/4] tipc: slim down name table

2018-03-28 Thread Jon Maloy
We clean up and improve the name binding table: - Replace the memory consuming 'sub_sequence/service range' array with an RB tree. - Introduce support for overlapping service sequences/ranges Jon Maloy (4): tipc: replace name table service range array with rb tree tipc: ref

RE: [PATCH] tipc: avoid possible string overflow

2018-03-28 Thread Jon Maloy
> -Original Message- > From: Arnd Bergmann [mailto:a...@arndb.de] > Sent: Wednesday, March 28, 2018 10:02 > To: Jon Maloy ; Ying Xue > ; David S. Miller > Cc: Arnd Bergmann ; Parthasarathy Bhuvaragan > ; netdev@vger.kernel.org; tipc- > discuss...@lists.so

RE: [PATCH net-next] tipc: tipc_node_create() can be static

2018-03-26 Thread Jon Maloy
Acked-by: Jon Maloy Thanks ///jon > -Original Message- > From: Wei Yongjun [mailto:weiyongj...@huawei.com] > Sent: Monday, March 26, 2018 10:33 > To: Jon Maloy ; Ying Xue > > Cc: Wei Yongjun ; netdev@vger.kernel.org; > tipc-discuss...@lists.sourcefo

RE: [PATCH net-next] tipc: fix error handling in tipc_udp_enable()

2018-03-26 Thread Jon Maloy
Acked-by: Jon Maloy Thank you, Wei. > -Original Message- > From: Wei Yongjun [mailto:weiyongj...@huawei.com] > Sent: Monday, March 26, 2018 10:33 > To: Jon Maloy ; Ying Xue > > Cc: Wei Yongjun ; netdev@vger.kernel.org; > tipc-discuss...@lists.sourcefo

RE: [RFC PATCH net-next] tipc: tipc_disc_addr_trial_msg() can be static

2018-03-24 Thread Jon Maloy
Acked-by: Jon Maloy jon.ma...@ericsson.com Thanks, Fengguang > -Original Message- > From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of kbuild test robot > Sent: Friday, March 23, 2018 15:48 > To: Jon Maloy > Cc: kbuild-

RE: [iproute2 1/1] ss: Add support for TIPC socket diag in ss tool

2018-03-23 Thread Jon Maloy
- > From: Mohan Krishna Ghanta Krishnamurthy > Sent: Friday, March 23, 2018 10:01 > To: tipc-discuss...@lists.sourceforge.net; Jon Maloy > ; ma...@donjonn.com; > ying@windriver.com; Mohan Krishna Ghanta Krishnamurthy > ; > netdev@vger.kernel.org; step...@networkplumber.o

[net-next 1/8] tipc: refactor function tipc_enable_bearer()

2018-03-22 Thread Jon Maloy
As a preparation for the next commits we try to reduce the footprint of the function tipc_enable_bearer(), while hopefully making is simpler to follow. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/bearer.c | 136 -- 1 file changed

[net-next 7/8] tipc: handle collisions of 32-bit node address hash values

2018-03-22 Thread Jon Maloy
s the address explicitly, there will be no trial period and messages, so this protocol addition is completely backwards compatible. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/addr.c | 3 +- net/tipc/bearer.c | 3 +- net/tipc/core.c | 2 + net/tipc/core.h | 2 +

[net-next 8/8] tipc: obtain node identity from interface by default

2018-03-22 Thread Jon Maloy
address in the case of an L2 bearer, IPv4/IPv6 address in the case of a UDP bearer. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/bearer.c| 24 +++- net/tipc/net.h | 1 + net/tipc/udp_media.c | 13 + 3 files changed, 29 insertions(+), 9

[net-next 5/8] tipc: remove direct accesses to own_addr field in struct tipc_net

2018-03-22 Thread Jon Maloy
As a preparation to changing the addressing structure of TIPC we replace all direct accesses to the tipc_net::own_addr field with the function dedicated for this, tipc_own_addr(). There are no changes to program logics in this commit. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc

[net-next 2/8] tipc: some cleanups in the file discover.c

2018-03-22 Thread Jon Maloy
Xue Signed-off-by: Jon Maloy --- net/tipc/bearer.c | 8 +- net/tipc/bearer.h | 2 +- net/tipc/discover.c | 303 +--- net/tipc/discover.h | 8 +- 4 files changed, 155 insertions(+), 166 deletions(-) diff --git a/net/tipc/bearer.c b/net

[net-next 4/8] tipc: allow closest-first lookup algorithm when legacy address is configured

2018-03-22 Thread Jon Maloy
en a legacy user indicates a zero lookup domain 'closest-first' is selected, and in all other cases we use 'round-robin'. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/addr.c | 12 +++- net/tipc/addr.h | 2 +- net/tipc/core.h | 3 ++- net

[net-next 3/8] tipc: remove restrictions on node address values

2018-03-22 Thread Jon Maloy
epted, provided both peers have the same network id. This secures backwards compatibility for users who have been using zone or cluster identities as cluster separators, instead of the intended 'network id'. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/addr.c |

[net-next 6/8] tipc: add 128-bit node identifier

2018-03-22 Thread Jon Maloy
. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- include/uapi/linux/tipc_netlink.h | 2 + net/tipc/addr.c | 81 --- net/tipc/addr.h | 28 +++--- net/tipc/core.c | 4 +- net/tipc/core.h

[net-next 0/8] tipc: introduce 128-bit auto-configurable node id

2018-03-22 Thread Jon Maloy
or IP address. Jon Maloy (8): tipc: refactor function tipc_enable_bearer() tipc: some cleanups in the file discover.c tipc: remove restrictions on node address values tipc: allow closest-first lookup algorithm when legacy address is configured tipc: remove direct accesses to own_addr fie

RE: [net-next 1/5] tipc: obsolete TIPC_ZONE_SCOPE

2018-03-15 Thread Jon Maloy
No, it won't. I just moved those functions and #defines to the bottom of the same file, and marked them as 'deprecated'. BR ///jon > -Original Message- > From: Jiri Pirko [mailto:j...@resnulli.us] > Sent: Thursday, March 15, 2018 12:11 > To: Jon Maloy > C

[net-next 2/5] tipc: remove zone publication list in name table

2018-03-15 Thread Jon Maloy
As a consequence of the previous commit we nan now eliminate zone scope related lists in the name table. We start with name_table::publ_list[3], which can now be replaced with two lists, one for node scope publications and one for cluster scope publications. Acked-by: Ying Xue Signed-off-by: Jon

[net-next 5/5] tipc: some name changes

2018-03-15 Thread Jon Maloy
onal changes in this commit. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/name_distr.c | 12 ++--- net/tipc/name_distr.h | 2 +- net/tipc/name_table.c | 143 -- net/tipc/name_table.h | 38 -- net/tipc/socket.c |

[net-next 4/5] tipc: merge two lists in struct publication

2018-03-15 Thread Jon Maloy
e' which reflects what it really is. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/name_distr.c | 20 ++-- net/tipc/name_table.h | 5 ++--- 2 files changed, 12 insertions(+), 13 deletions(-) diff --git a/net/tipc/name_distr.c b/net/tipc/name_distr.c index 11ce205.

[net-next 3/5] tipc: remove zone_list member in struct publication

2018-03-15 Thread Jon Maloy
e of those lists, since they don't serve any purpose. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- net/tipc/name_table.c | 101 ++ net/tipc/name_table.h | 5 +-- 2 files changed, 30 insertions(+), 76 deletions(-) diff --git a/net/tipc/nam

[net-next 0/5] tipc: obsolete zone concept

2018-03-15 Thread Jon Maloy
Functionality related to the 'zone' concept was never implemented in TIPC. In this series we eliminate the remaining traces of it in the code, and can hence take a first step in reducing the footprint and complexity of the binding table. Jon Maloy (5): tipc: obsolete TIPC_ZONE_SC

[net-next 1/5] tipc: obsolete TIPC_ZONE_SCOPE

2018-03-15 Thread Jon Maloy
for use during lookup, with the same meaning as ZONE_SCOPE/CLUSTER_SCOPE. We now permit it even as binding scope, but for compatibility reasons we choose to not change the value of TIPC_CLUSTER_SCOPE. Acked-by: Ying Xue Signed-off-by: Jon Maloy --- include/uapi/linux/tipc.h | 102

[net-next 0/5] tipc: obsolete zone concept

2018-03-15 Thread Jon Maloy
Functionality related to the zone concept was never implemented in TIPC. In this series we eliminate the remaining traces of it in the code, and can hence take a first step in reducing the footprint and complexity of the binding table. Jon Maloy (5): tipc: obsolete TIPC_ZONE_SCOPE tipc

[net-next 1/1] tipc: sockopt(TIPC_SO_RCVBUF) for setting receive buffer

2018-02-27 Thread Jon Maloy
From: Hoang Le We introduce a set/getsockopt for setting socket receive buffer per individual socket. This has turned out to sometimes be necessary for anycast and multicast receivers when used without flow control. Signed-off-by: Hoang Le Signed-off-by: Jon Maloy --- include/uapi/linux

[net 1/1] tipc: correct initial value for group congestion flag

2018-02-26 Thread Jon Maloy
y request so. This commit corrects this bug. Reported-by: Hoang Le Signed-off-by: Jon Maloy --- net/tipc/group.c | 1 + net/tipc/socket.c | 1 + 2 files changed, 2 insertions(+) diff --git a/net/tipc/group.c b/net/tipc/group.c index 03086cc..d7a7bef 100644 --- a/net/tipc/group.c +++ b/net

RE: [PATCH net-next] tipc: don't call sock_release() in atomic context

2018-02-19 Thread Jon Maloy
> -Original Message- > From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of Paolo Abeni > Sent: Monday, February 19, 2018 19:02 > To: netdev@vger.kernel.org > Cc: Jon Maloy ; Ying Xue > ; David S. Miller > Subject: [PATCH ne

RE: [net-next v2 1/1] tipc: avoid unnecessary copying of bundled messages

2018-02-19 Thread Jon Maloy
> -Original Message- > From: David Laight [mailto:david.lai...@aculab.com] > Sent: Monday, February 19, 2018 14:30 > To: Jon Maloy > Cc: netdev@vger.kernel.org; Mohan Krishna Ghanta Krishnamurthy > ; Tung Quang Nguyen > ; Hoang Huu Le > ; Canh Duc Luu > ;

RE: BUG: sleeping function called from invalid context at net/core/sock.c:LINE (3)

2018-02-19 Thread Jon Maloy
com] > Sent: Saturday, February 17, 2018 23:23 > To: Dmitry Vyukov ; syzbot > ; Jon Maloy > ; Ying Xue > Cc: Andrei Vagin ; David Miller > ; Eric W. Biederman ; > Florian Westphal ; LKML ; > netdev ; Nicolas Dichtel > ; roman.k...@sysgo.com; syzkaller-

[net-next 1/1] tipc: fix bug on error path in tipc_topsrv_kern_subscr()

2018-02-19 Thread Jon Maloy
In commit cc1ea9ffadf7 ("tipc: eliminate struct tipc_subscriber") we re-introduced an old bug on the error path in the function tipc_topsrv_kern_subscr(). We now re-introduce the correction too. Reported-by: syzbot+f62e0f2a0ef578703...@syzkaller.appspotmail.com Signed-off-by: Jon Maloy

RE: [net-next v2 1/1] tipc: avoid unnecessary copying of bundled messages

2018-02-17 Thread Jon Maloy
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Friday, February 16, 2018 21:33 > To: Jon Maloy > Cc: netdev@vger.kernel.org; Mohan Krishna Ghanta Krishnamurthy > ; Tung Quang Nguyen > ; Hoang Huu Le > ; Canh Duc Luu > ;

[net-next v2 1/1] tipc: avoid unnecessary copying of bundled messages

2018-02-15 Thread Jon Maloy
alue of the buffer clones to be the truesize of the clone itself plus a calculated fraction of the base buffer's overhead. This change eliminates the overaccounting and at least mitigates the occurrence of unnecessary buffer copying. Reported-by: Hoang Le Acked-by: Ying Xue Signed-off-by: Jon Mal

  1   2   3   4   5   >