Commit 9dce285b authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 'tipc-next' 



Richard Alpe says:

====================
tipc: new compat layer for the legacy NL API

This is a compatibility / transcoding layer for the old netlink API.
It relies on the new netlink API to collect data or perform actions
(dumpit / doit).

The main benefit of this compat layer is that it removes a lot of
complex code from the tipc core as only the new API needs to be able
harness data or perform actions. I.e. the compat layer isn't concerned
with locking or how the internal data-structures look. As long as the
new API stays relatively intact the compat layer should be fine.

The main challenge in this compat layer is the randomness of the legacy
API. Some commands send binary data and some send ASCII data, some are
very picky in optimizing there buffer sizes and some just don't care.
Most legacy commands put there data in a single TLV (data container) but some
segment the data into multiple TLV's. This list of randomness goes on and on..
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents c8ac18f2 941787b8

include/uapi/linux/tipc_config.h

+20 −0
Original line number Diff line number Diff line
@@ -272,6 +272,26 @@ static inline int TLV_CHECK(const void *tlv, __u16 space, __u16 exp_type) 
		(ntohs(((struct tlv_desc *)tlv)->tlv_type) == exp_type);

}



static inline int TLV_GET_LEN(struct tlv_desc *tlv)

{

	return ntohs(tlv->tlv_len);

}



static inline void TLV_SET_LEN(struct tlv_desc *tlv, __u16 len)

{

	tlv->tlv_len = htons(len);

}



static inline int TLV_CHECK_TYPE(struct tlv_desc *tlv,  __u16 type)

{

	return (ntohs(tlv->tlv_type) == type);

}



static inline void TLV_SET_TYPE(struct tlv_desc *tlv, __u16 type)

{

	tlv->tlv_type = htons(type);

}



static inline int TLV_SET(void *tlv, __u16 type, void *data, __u16 len)

{

	struct tlv_desc *tlv_ptr;

net/tipc/Makefile

+3 −3
Original line number Diff line number Diff line
@@ -4,11 +4,11 @@ 


obj-$(CONFIG_TIPC) := tipc.o



tipc-y	+= addr.o bcast.o bearer.o config.o \

tipc-y	+= addr.o bcast.o bearer.o \

	   core.o link.o discover.o msg.o  \

	   name_distr.o  subscr.o name_table.o net.o  \

	   netlink.o node.o socket.o log.o eth_media.o \

	   server.o

	   netlink.o netlink_compat.o node.o socket.o eth_media.o \

	   server.o socket.o



tipc-$(CONFIG_TIPC_MEDIA_IB)	+= ib_media.o

tipc-$(CONFIG_SYSCTL)		+= sysctl.o

net/tipc/bcast.c

+1 −44
Original line number Diff line number Diff line
@@ -810,7 +810,7 @@ int tipc_nl_add_bc_link(struct net *net, struct tipc_nl_msg *msg) 


	tipc_bclink_lock(net);



	hdr = genlmsg_put(msg->skb, msg->portid, msg->seq, &tipc_genl_v2_family,

	hdr = genlmsg_put(msg->skb, msg->portid, msg->seq, &tipc_genl_family,

			  NLM_F_MULTI, TIPC_NL_LINK_GET);

	if (!hdr)

		return -EMSGSIZE;
@@ -860,49 +860,6 @@ msg_full: 
	return -EMSGSIZE;

}



int tipc_bclink_stats(struct net *net, char *buf, const u32 buf_size)

{

	int ret;

	struct tipc_stats *s;

	struct tipc_net *tn = net_generic(net, tipc_net_id);

	struct tipc_link *bcl = tn->bcl;



	if (!bcl)

		return 0;



	tipc_bclink_lock(net);



	s = &bcl->stats;



	ret = tipc_snprintf(buf, buf_size, "Link <%s>\n"

			    "  Window:%u packets\n",

			    bcl->name, bcl->queue_limit[0]);

	ret += tipc_snprintf(buf + ret, buf_size - ret,

			     "  RX packets:%u fragments:%u/%u bundles:%u/%u\n",

			     s->recv_info, s->recv_fragments,

			     s->recv_fragmented, s->recv_bundles,

			     s->recv_bundled);

	ret += tipc_snprintf(buf + ret, buf_size - ret,

			     "  TX packets:%u fragments:%u/%u bundles:%u/%u\n",

			     s->sent_info, s->sent_fragments,

			     s->sent_fragmented, s->sent_bundles,

			     s->sent_bundled);

	ret += tipc_snprintf(buf + ret, buf_size - ret,

			     "  RX naks:%u defs:%u dups:%u\n",

			     s->recv_nacks, s->deferred_recv, s->duplicates);

	ret += tipc_snprintf(buf + ret, buf_size - ret,

			     "  TX naks:%u acks:%u dups:%u\n",

			     s->sent_nacks, s->sent_acks, s->retransmitted);

	ret += tipc_snprintf(buf + ret, buf_size - ret,

			     "  Congestion link:%u  Send queue max:%u avg:%u\n",

			     s->link_congs, s->max_queue_sz,

			     s->queue_sz_counts ?

			     (s->accu_queue_sz / s->queue_sz_counts) : 0);



	tipc_bclink_unlock(net);

	return ret;

}



int tipc_bclink_reset_stats(struct net *net)

{

	struct tipc_net *tn = net_generic(net, tipc_net_id);

net/tipc/bcast.h

+0 −1
Original line number Diff line number Diff line
@@ -127,7 +127,6 @@ u32 tipc_bclink_get_last_sent(struct net *net); 
u32  tipc_bclink_acks_missing(struct tipc_node *n_ptr);

void tipc_bclink_update_link_state(struct tipc_node *node,

				   u32 last_sent);

int  tipc_bclink_stats(struct net *net, char *stats_buf, const u32 buf_size);

int  tipc_bclink_reset_stats(struct net *net);

int  tipc_bclink_set_queue_limits(struct net *net, u32 limit);

void tipc_bcbearer_sort(struct net *net, struct tipc_node_map *nm_ptr,

net/tipc/bearer.c

+10 −76
Original line number Diff line number Diff line
@@ -36,7 +36,6 @@ 


#include <net/sock.h>

#include "core.h"

#include "config.h"

#include "bearer.h"

#include "link.h"

#include "discover.h"
@@ -112,37 +111,17 @@ void tipc_media_addr_printf(char *buf, int len, struct tipc_media_addr *a) 
	m_ptr = media_find_id(a->media_id);



	if (m_ptr && !m_ptr->addr2str(a, addr_str, sizeof(addr_str)))

		ret = tipc_snprintf(buf, len, "%s(%s)", m_ptr->name, addr_str);

		ret = scnprintf(buf, len, "%s(%s)", m_ptr->name, addr_str);

	else {

		u32 i;



		ret = tipc_snprintf(buf, len, "UNKNOWN(%u)", a->media_id);

		ret = scnprintf(buf, len, "UNKNOWN(%u)", a->media_id);

		for (i = 0; i < sizeof(a->value); i++)

			ret += tipc_snprintf(buf - ret, len + ret,

			ret += scnprintf(buf - ret, len + ret,

					    "-%02x", a->value[i]);

	}

}



/**

 * tipc_media_get_names - record names of registered media in buffer

 */

struct sk_buff *tipc_media_get_names(void)

{

	struct sk_buff *buf;

	int i;



	buf = tipc_cfg_reply_alloc(MAX_MEDIA * TLV_SPACE(TIPC_MAX_MEDIA_NAME));

	if (!buf)

		return NULL;



	for (i = 0; media_info_array[i] != NULL; i++) {

		tipc_cfg_append_tlv(buf, TIPC_TLV_MEDIA_NAME,

				    media_info_array[i]->name,

				    strlen(media_info_array[i]->name) + 1);

	}

	return buf;

}



/**

 * bearer_name_validate - validate & (optionally) deconstruct bearer name

 * @name: ptr to bearer name string
@@ -205,35 +184,6 @@ struct tipc_bearer *tipc_bearer_find(struct net *net, const char *name) 
	return NULL;

}



/**

 * tipc_bearer_get_names - record names of bearers in buffer

 */

struct sk_buff *tipc_bearer_get_names(struct net *net)

{

	struct tipc_net *tn = net_generic(net, tipc_net_id);

	struct sk_buff *buf;

	struct tipc_bearer *b;

	int i, j;



	buf = tipc_cfg_reply_alloc(MAX_BEARERS * TLV_SPACE(TIPC_MAX_BEARER_NAME));

	if (!buf)

		return NULL;



	for (i = 0; media_info_array[i] != NULL; i++) {

		for (j = 0; j < MAX_BEARERS; j++) {

			b = rtnl_dereference(tn->bearer_list[j]);

			if (!b)

				continue;

			if (b->media == media_info_array[i]) {

				tipc_cfg_append_tlv(buf, TIPC_TLV_BEARER_NAME,

						    b->name,

						    strlen(b->name) + 1);

			}

		}

	}

	return buf;

}



void tipc_bearer_add_dest(struct net *net, u32 bearer_id, u32 dest)

{

	struct tipc_net *tn = net_generic(net, tipc_net_id);
@@ -265,8 +215,8 @@ void tipc_bearer_remove_dest(struct net *net, u32 bearer_id, u32 dest) 
/**

 * tipc_enable_bearer - enable bearer with the given name

 */

int tipc_enable_bearer(struct net *net, const char *name, u32 disc_domain,

		       u32 priority)

static int tipc_enable_bearer(struct net *net, const char *name,

			      u32 disc_domain, u32 priority)

{

	struct tipc_net *tn = net_generic(net, tipc_net_id);

	struct tipc_bearer *b_ptr;
@@ -422,22 +372,6 @@ static void bearer_disable(struct net *net, struct tipc_bearer *b_ptr, 
	kfree_rcu(b_ptr, rcu);

}



int tipc_disable_bearer(struct net *net, const char *name)

{

	struct tipc_bearer *b_ptr;

	int res;



	b_ptr = tipc_bearer_find(net, name);

	if (b_ptr == NULL) {

		pr_warn("Attempt to disable unknown bearer <%s>\n", name);

		res = -EINVAL;

	} else {

		bearer_disable(net, b_ptr, false);

		res = 0;

	}

	return res;

}



int tipc_enable_l2_media(struct net *net, struct tipc_bearer *b)

{

	struct net_device *dev;
@@ -658,7 +592,7 @@ static int __tipc_nl_add_bearer(struct tipc_nl_msg *msg, 
	struct nlattr *attrs;

	struct nlattr *prop;



	hdr = genlmsg_put(msg->skb, msg->portid, msg->seq, &tipc_genl_v2_family,

	hdr = genlmsg_put(msg->skb, msg->portid, msg->seq, &tipc_genl_family,

			  NLM_F_MULTI, TIPC_NL_BEARER_GET);

	if (!hdr)

		return -EMSGSIZE;
@@ -785,7 +719,7 @@ int tipc_nl_bearer_disable(struct sk_buff *skb, struct genl_info *info) 
	char *name;

	struct tipc_bearer *bearer;

	struct nlattr *attrs[TIPC_NLA_BEARER_MAX + 1];

	struct net *net = genl_info_net(info);

	struct net *net = sock_net(skb->sk);



	if (!info->attrs[TIPC_NLA_BEARER])

		return -EINVAL;
@@ -816,11 +750,11 @@ int tipc_nl_bearer_disable(struct sk_buff *skb, struct genl_info *info) 


int tipc_nl_bearer_enable(struct sk_buff *skb, struct genl_info *info)

{

	struct net *net = genl_info_net(info);

	struct tipc_net *tn = net_generic(net, tipc_net_id);

	int err;

	char *bearer;

	struct nlattr *attrs[TIPC_NLA_BEARER_MAX + 1];

	struct net *net = sock_net(skb->sk);

	struct tipc_net *tn = net_generic(net, tipc_net_id);

	u32 domain;

	u32 prio;
@@ -924,7 +858,7 @@ static int __tipc_nl_add_media(struct tipc_nl_msg *msg, 
	struct nlattr *attrs;

	struct nlattr *prop;



	hdr = genlmsg_put(msg->skb, msg->portid, msg->seq, &tipc_genl_v2_family,

	hdr = genlmsg_put(msg->skb, msg->portid, msg->seq, &tipc_genl_family,

			  NLM_F_MULTI, TIPC_NL_MEDIA_GET);

	if (!hdr)

		return -EMSGSIZE;

CRA Git | Maintained and supported by SUSTech CRA and CCSE