<feed xmlns='http://www.w3.org/2005/Atom'>
<title>kernel/linux.git/net/tipc/discover.h, branch v6.12.80</title>
<subtitle>Linux kernel stable tree (mirror)</subtitle>
<id>https://git.radix-linux.su/kernel/linux.git/atom?h=v6.12.80</id>
<link rel='self' href='https://git.radix-linux.su/kernel/linux.git/atom?h=v6.12.80'/>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/'/>
<updated>2018-03-23T17:12:17+00:00</updated>
<entry>
<title>tipc: some cleanups in the file discover.c</title>
<updated>2018-03-23T17:12:17+00:00</updated>
<author>
<name>Jon Maloy</name>
<email>jon.maloy@ericsson.com</email>
</author>
<published>2018-03-22T19:42:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=b39e465e56ec38ca64b4c0affeb6411eb0ed7267'/>
<id>urn:sha1:b39e465e56ec38ca64b4c0affeb6411eb0ed7267</id>
<content type='text'>
To facilitate the coming changes in the neighbor discovery functionality
we make some renaming and refactoring of that code. The functional changes
in this commit are trivial, e.g., that we move the message sending call in
tipc_disc_timeout() outside the spinlock protected region.

Acked-by: Ying Xue &lt;ying.xue@windriver.com&gt;
Signed-off-by: Jon Maloy &lt;jon.maloy@ericsson.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>tipc: eliminate buffer leak in bearer layer</title>
<updated>2016-04-07T21:00:13+00:00</updated>
<author>
<name>Jon Paul Maloy</name>
<email>jon.maloy@ericsson.com</email>
</author>
<published>2016-04-07T14:09:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=4e801fa14f68223d36480bced975ebf0c5f9a284'/>
<id>urn:sha1:4e801fa14f68223d36480bced975ebf0c5f9a284</id>
<content type='text'>
When enabling a bearer we create a 'neigbor discoverer' instance by
calling the function tipc_disc_create() before the bearer is actually
registered in the list of enabled bearers. Because of this, the very
first discovery broadcast message, created by the mentioned function,
is lost, since it cannot find any valid bearer to use. Furthermore,
the used send function, tipc_bearer_xmit_skb() does not free the given
buffer when it cannot find a  bearer, resulting in the leak of exactly
one send buffer each time a bearer is enabled.

This commit fixes this problem by introducing two changes:

1) Instead of attemting to send the discovery message directly, we let
   tipc_disc_create() return the discovery buffer to the calling
   function, tipc_enable_bearer(), so that the latter can send it
   when the enabling sequence is finished.

2) In tipc_bearer_xmit_skb(), as well as in the two other transmit
   functions at the bearer layer, we now free the indicated buffer or
   buffer chain when a valid bearer cannot be found.

Acked-by: Ying Xue &lt;ying.xue@windriver.com&gt;
Signed-off-by: Jon Maloy &lt;jon.maloy@ericsson.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>tipc: involve namespace infrastructure</title>
<updated>2015-01-12T21:24:32+00:00</updated>
<author>
<name>Ying Xue</name>
<email>ying.xue@windriver.com</email>
</author>
<published>2015-01-09T07:27:04+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=c93d3baa24095887005647984cff5de8c63d3611'/>
<id>urn:sha1:c93d3baa24095887005647984cff5de8c63d3611</id>
<content type='text'>
Involve namespace infrastructure, make the "tipc_net_id" global
variable aware of per namespace, and rename it to "net_id". In
order that the conversion can be successfully done, an instance
of networking namespace must be passed to relevant functions,
allowing them to access the "net_id" variable of per namespace.

Signed-off-by: Ying Xue &lt;ying.xue@windriver.com&gt;
Tested-by: Tero Aho &lt;Tero.Aho@coriant.com&gt;
Reviewed-by: Jon Maloy &lt;jon.maloy@ericsson.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>tipc: fix race in disc create/delete</title>
<updated>2014-04-23T01:17:53+00:00</updated>
<author>
<name>Ying Xue</name>
<email>ying.xue@windriver.com</email>
</author>
<published>2014-04-21T02:55:52+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=a8b9b96e959f3c035af20b1bd2ba67b0b7269b19'/>
<id>urn:sha1:a8b9b96e959f3c035af20b1bd2ba67b0b7269b19</id>
<content type='text'>
Commit a21a584d6720ce349b05795b9bcfab3de8e58419 (tipc: fix neighbor
detection problem after hw address change) introduces a race condition
involving tipc_disc_delete() and tipc_disc_add/remove_dest that can
cause TIPC to dereference the pointer to the bearer discovery request
structure after it has been freed since a stray pointer is left in the
bearer structure.

In order to fix the issue, the process of resetting the discovery
request handler is optimized: the discovery request handler and request
buffer are just reset instead of being freed, allocated and initialized.
As the request point is always valid and the request's lock is taken
while the request handler is reset, the race doesn't happen any more.

Reported-by: Erik Hugne &lt;erik.hugne@ericsson.com&gt;
Signed-off-by: Ying Xue &lt;ying.xue@windriver.com&gt;
Reviewed-by: Erik Hugne &lt;erik.hugne@ericsson.com&gt;
Tested-by: Erik Hugne &lt;erik.hugne@ericsson.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>tipc: make discovery domain a bearer attribute</title>
<updated>2014-03-28T18:46:29+00:00</updated>
<author>
<name>Erik Hugne</name>
<email>erik.hugne@ericsson.com</email>
</author>
<published>2014-03-28T09:32:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=16470111ed2aad67d2e2407aae2a8a53a4a38060'/>
<id>urn:sha1:16470111ed2aad67d2e2407aae2a8a53a4a38060</id>
<content type='text'>
The node discovery domain is assigned when a bearer is enabled.
In the previous commit we reflect this attribute directly in the
bearer structure since it's needed to reinitialize the node
discovery mechanism after a hardware address change.

There's no need to replicate this attribute anywhere else, so we
remove it from the tipc_link_req structure.

Signed-off-by: Erik Hugne &lt;erik.hugne@ericsson.com&gt;
Reviewed-by: Ying Xue &lt;ying.xue@windriver.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>tipc: align tipc function names with common naming practice in the network</title>
<updated>2014-02-18T22:31:59+00:00</updated>
<author>
<name>Ying Xue</name>
<email>ying.xue@windriver.com</email>
</author>
<published>2014-02-18T08:06:46+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=247f0f3c3176c55b46cb9a20011d3d6757634815'/>
<id>urn:sha1:247f0f3c3176c55b46cb9a20011d3d6757634815</id>
<content type='text'>
Rename the following functions, which are shorter and more in line
with common naming practice in the network subsystem.

tipc_bclink_send_msg-&gt;tipc_bclink_xmit
tipc_bclink_recv_pkt-&gt;tipc_bclink_rcv
tipc_disc_recv_msg-&gt;tipc_disc_rcv
tipc_link_send_proto_msg-&gt;tipc_link_proto_xmit
link_recv_proto_msg-&gt;tipc_link_proto_rcv
link_send_sections_long-&gt;tipc_link_iovec_long_xmit
tipc_link_send_sections_fast-&gt;tipc_link_iovec_xmit_fast
tipc_link_send_sync-&gt;tipc_link_sync_xmit
tipc_link_recv_sync-&gt;tipc_link_sync_rcv
tipc_link_send_buf-&gt;__tipc_link_xmit
tipc_link_send-&gt;tipc_link_xmit
tipc_link_send_names-&gt;tipc_link_names_xmit
tipc_named_recv-&gt;tipc_named_rcv
tipc_link_recv_bundle-&gt;tipc_link_bundle_rcv
tipc_link_dup_send_queue-&gt;tipc_link_dup_queue_xmit
link_send_long_buf-&gt;tipc_link_frag_xmit

tipc_multicast-&gt;tipc_port_mcast_xmit
tipc_port_recv_mcast-&gt;tipc_port_mcast_rcv
tipc_port_reject_sections-&gt;tipc_port_iovec_reject
tipc_port_recv_proto_msg-&gt;tipc_port_proto_rcv
tipc_connect-&gt;tipc_port_connect
__tipc_connect-&gt;__tipc_port_connect
__tipc_disconnect-&gt;__tipc_port_disconnect
tipc_disconnect-&gt;tipc_port_disconnect
tipc_shutdown-&gt;tipc_port_shutdown
tipc_port_recv_msg-&gt;tipc_port_rcv
tipc_port_recv_sections-&gt;tipc_port_iovec_rcv

release-&gt;tipc_release
accept-&gt;tipc_accept
bind-&gt;tipc_bind
get_name-&gt;tipc_getname
poll-&gt;tipc_poll
send_msg-&gt;tipc_sendmsg
send_packet-&gt;tipc_send_packet
send_stream-&gt;tipc_send_stream
recv_msg-&gt;tipc_recvmsg
recv_stream-&gt;tipc_recv_stream
connect-&gt;tipc_connect
listen-&gt;tipc_listen
shutdown-&gt;tipc_shutdown
setsockopt-&gt;tipc_setsockopt
getsockopt-&gt;tipc_getsockopt

Above changes have no impact on current users of the functions.

Signed-off-by: Ying Xue &lt;ying.xue@windriver.com&gt;
Reviewed-by: Jon Maloy &lt;jon.maloy@ericsson.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
</content>
</entry>
<entry>
<title>tipc: rename struct link* to struct tipc_link*</title>
<updated>2011-12-30T02:53:30+00:00</updated>
<author>
<name>Paul Gortmaker</name>
<email>paul.gortmaker@windriver.com</email>
</author>
<published>2011-12-30T01:58:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=a18c4bc3ea3c23f658655b1eee4f62cb71d51efd'/>
<id>urn:sha1:a18c4bc3ea3c23f658655b1eee4f62cb71d51efd</id>
<content type='text'>
This converts the following:

	struct link		-&gt;	struct tipc_link
	struct link_req		-&gt;	struct tipc_link_req
	struct link_name	-&gt;	struct tipc_link_name

Signed-off-by: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
</content>
</entry>
<entry>
<title>tipc: Add monitoring of number of nodes discovered by bearer</title>
<updated>2011-05-10T20:04:01+00:00</updated>
<author>
<name>Allan Stephens</name>
<email>Allan.Stephens@windriver.com</email>
</author>
<published>2011-04-22T00:05:25+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=1209966cd5d2ec7f89ad2ed58a6a342aa8ea8712'/>
<id>urn:sha1:1209966cd5d2ec7f89ad2ed58a6a342aa8ea8712</id>
<content type='text'>
Augments TIPC's discovery object to track the number of neighboring nodes
having an active link to the associated bearer.

This means tipc_disc_update_link_req() becomes either one of:

       tipc_disc_add_dest()
or:
       tipc_disc_remove_dest()

depending on the code flow direction of things.

Signed-off-by: Allan Stephens &lt;Allan.Stephens@windriver.com&gt;
Signed-off-by: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
</content>
</entry>
<entry>
<title>tipc: Enhance handling of discovery object creation failures</title>
<updated>2011-05-10T20:03:59+00:00</updated>
<author>
<name>Allan Stephens</name>
<email>Allan.Stephens@windriver.com</email>
</author>
<published>2011-04-21T18:58:26+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=3a777ff8b14456e15991c9fcc225943453dc3a75'/>
<id>urn:sha1:3a777ff8b14456e15991c9fcc225943453dc3a75</id>
<content type='text'>
Modifies bearer creation and deletion code to improve handling of
scenarios when a neighbor discovery object cannot be created. The
creation routine now aborts the creation of a bearer if its discovery
object cannot be created, and deletes the newly created bearer, rather
than failing quietly and leaving an unusable bearer hanging around.

Since the exit via the goto label really isn't a definitive failure
in all cases, relabel it appropriately.

Signed-off-by: Allan Stephens &lt;Allan.Stephens@windriver.com&gt;
Signed-off-by: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
</content>
</entry>
<entry>
<title>tipc: Remove unused message header field for requested number of links</title>
<updated>2011-02-23T23:05:22+00:00</updated>
<author>
<name>Allan Stephens</name>
<email>Allan.Stephens@windriver.com</email>
</author>
<published>2011-01-25T19:39:59+00:00</published>
<link rel='alternate' type='text/html' href='https://git.radix-linux.su/kernel/linux.git/commit/?id=2e07dda1659095115e5e36a2fed0fddc1e3ea1c8'/>
<id>urn:sha1:2e07dda1659095115e5e36a2fed0fddc1e3ea1c8</id>
<content type='text'>
Eliminates support for the "number of requested links" field in a neighbor
discovery message. This field was never used and has been removed from
the TIPC 2.0 protocol specification.

Signed-off-by: Allan Stephens &lt;Allan.Stephens@windriver.com&gt;
Signed-off-by: Paul Gortmaker &lt;paul.gortmaker@windriver.com&gt;
</content>
</entry>
</feed>
