diff options
Diffstat (limited to 'net/core')
| -rw-r--r-- | net/core/dev.c | 34 | ||||
| -rw-r--r-- | net/core/dev_ioctl.c | 19 | ||||
| -rw-r--r-- | net/core/ethtool.c | 6 | ||||
| -rw-r--r-- | net/core/netpoll.c | 1 | ||||
| -rw-r--r-- | net/core/skbuff.c | 20 | ||||
| -rw-r--r-- | net/core/sock.c | 17 | ||||
| -rw-r--r-- | net/core/utils.c | 22 | 
7 files changed, 56 insertions, 63 deletions
| diff --git a/net/core/dev.c b/net/core/dev.c index fc1e289397f5..faebb398fb46 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -792,6 +792,40 @@ struct net_device *dev_get_by_index(struct net *net, int ifindex)  EXPORT_SYMBOL(dev_get_by_index);  /** + *	netdev_get_name - get a netdevice name, knowing its ifindex. + *	@net: network namespace + *	@name: a pointer to the buffer where the name will be stored. + *	@ifindex: the ifindex of the interface to get the name from. + * + *	The use of raw_seqcount_begin() and cond_resched() before + *	retrying is required as we want to give the writers a chance + *	to complete when CONFIG_PREEMPT is not set. + */ +int netdev_get_name(struct net *net, char *name, int ifindex) +{ +	struct net_device *dev; +	unsigned int seq; + +retry: +	seq = raw_seqcount_begin(&devnet_rename_seq); +	rcu_read_lock(); +	dev = dev_get_by_index_rcu(net, ifindex); +	if (!dev) { +		rcu_read_unlock(); +		return -ENODEV; +	} + +	strcpy(name, dev->name); +	rcu_read_unlock(); +	if (read_seqcount_retry(&devnet_rename_seq, seq)) { +		cond_resched(); +		goto retry; +	} + +	return 0; +} + +/**   *	dev_getbyhwaddr_rcu - find a device by its hardware address   *	@net: the applicable net namespace   *	@type: media type of device diff --git a/net/core/dev_ioctl.c b/net/core/dev_ioctl.c index 6cc0481faade..5b7d0e1d0664 100644 --- a/net/core/dev_ioctl.c +++ b/net/core/dev_ioctl.c @@ -19,9 +19,8 @@  static int dev_ifname(struct net *net, struct ifreq __user *arg)  { -	struct net_device *dev;  	struct ifreq ifr; -	unsigned seq; +	int error;  	/*  	 *	Fetch the caller's info block. @@ -30,19 +29,9 @@ static int dev_ifname(struct net *net, struct ifreq __user *arg)  	if (copy_from_user(&ifr, arg, sizeof(struct ifreq)))  		return -EFAULT; -retry: -	seq = read_seqcount_begin(&devnet_rename_seq); -	rcu_read_lock(); -	dev = dev_get_by_index_rcu(net, ifr.ifr_ifindex); -	if (!dev) { -		rcu_read_unlock(); -		return -ENODEV; -	} - -	strcpy(ifr.ifr_name, dev->name); -	rcu_read_unlock(); -	if (read_seqcount_retry(&devnet_rename_seq, seq)) -		goto retry; +	error = netdev_get_name(net, ifr.ifr_name, ifr.ifr_ifindex); +	if (error) +		return error;  	if (copy_to_user(arg, &ifr, sizeof(struct ifreq)))  		return -EFAULT; diff --git a/net/core/ethtool.c b/net/core/ethtool.c index 22efdaa76ebf..ce91766eeca9 100644 --- a/net/core/ethtool.c +++ b/net/core/ethtool.c @@ -60,10 +60,10 @@ static const char netdev_features_strings[NETDEV_FEATURE_COUNT][ETH_GSTRING_LEN]  	[NETIF_F_IPV6_CSUM_BIT] =        "tx-checksum-ipv6",  	[NETIF_F_HIGHDMA_BIT] =          "highdma",  	[NETIF_F_FRAGLIST_BIT] =         "tx-scatter-gather-fraglist", -	[NETIF_F_HW_VLAN_CTAG_TX_BIT] =  "tx-vlan-ctag-hw-insert", +	[NETIF_F_HW_VLAN_CTAG_TX_BIT] =  "tx-vlan-hw-insert", -	[NETIF_F_HW_VLAN_CTAG_RX_BIT] =  "rx-vlan-ctag-hw-parse", -	[NETIF_F_HW_VLAN_CTAG_FILTER_BIT] = "rx-vlan-ctag-filter", +	[NETIF_F_HW_VLAN_CTAG_RX_BIT] =  "rx-vlan-hw-parse", +	[NETIF_F_HW_VLAN_CTAG_FILTER_BIT] = "rx-vlan-filter",  	[NETIF_F_HW_VLAN_STAG_TX_BIT] =  "tx-vlan-stag-hw-insert",  	[NETIF_F_HW_VLAN_STAG_RX_BIT] =  "rx-vlan-stag-hw-parse",  	[NETIF_F_HW_VLAN_STAG_FILTER_BIT] = "rx-vlan-stag-filter", diff --git a/net/core/netpoll.c b/net/core/netpoll.c index cec074be8c43..35a9f0804b6f 100644 --- a/net/core/netpoll.c +++ b/net/core/netpoll.c @@ -12,6 +12,7 @@  #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt  #include <linux/moduleparam.h> +#include <linux/kernel.h>  #include <linux/netdevice.h>  #include <linux/etherdevice.h>  #include <linux/string.h> diff --git a/net/core/skbuff.c b/net/core/skbuff.c index cfd777bd6bd0..1c1738cc4538 100644 --- a/net/core/skbuff.c +++ b/net/core/skbuff.c @@ -483,15 +483,8 @@ EXPORT_SYMBOL(skb_add_rx_frag);  static void skb_drop_list(struct sk_buff **listp)  { -	struct sk_buff *list = *listp; - +	kfree_skb_list(*listp);  	*listp = NULL; - -	do { -		struct sk_buff *this = list; -		list = list->next; -		kfree_skb(this); -	} while (list);  }  static inline void skb_drop_fraglist(struct sk_buff *skb) @@ -651,6 +644,17 @@ void kfree_skb(struct sk_buff *skb)  }  EXPORT_SYMBOL(kfree_skb); +void kfree_skb_list(struct sk_buff *segs) +{ +	while (segs) { +		struct sk_buff *next = segs->next; + +		kfree_skb(segs); +		segs = next; +	} +} +EXPORT_SYMBOL(kfree_skb_list); +  /**   *	skb_tx_error - report an sk_buff xmit error   *	@skb: buffer that triggered an error diff --git a/net/core/sock.c b/net/core/sock.c index 88868a9d21da..d6d024cfaaaf 100644 --- a/net/core/sock.c +++ b/net/core/sock.c @@ -571,9 +571,7 @@ static int sock_getbindtodevice(struct sock *sk, char __user *optval,  	int ret = -ENOPROTOOPT;  #ifdef CONFIG_NETDEVICES  	struct net *net = sock_net(sk); -	struct net_device *dev;  	char devname[IFNAMSIZ]; -	unsigned seq;  	if (sk->sk_bound_dev_if == 0) {  		len = 0; @@ -584,20 +582,9 @@ static int sock_getbindtodevice(struct sock *sk, char __user *optval,  	if (len < IFNAMSIZ)  		goto out; -retry: -	seq = read_seqcount_begin(&devnet_rename_seq); -	rcu_read_lock(); -	dev = dev_get_by_index_rcu(net, sk->sk_bound_dev_if); -	ret = -ENODEV; -	if (!dev) { -		rcu_read_unlock(); +	ret = netdev_get_name(net, devname, sk->sk_bound_dev_if); +	if (ret)  		goto out; -	} - -	strcpy(devname, dev->name); -	rcu_read_unlock(); -	if (read_seqcount_retry(&devnet_rename_seq, seq)) -		goto retry;  	len = strlen(devname) + 1; diff --git a/net/core/utils.c b/net/core/utils.c index 3c7f5b51b979..aa88e23fc87a 100644 --- a/net/core/utils.c +++ b/net/core/utils.c @@ -338,25 +338,3 @@ void inet_proto_csum_replace16(__sum16 *sum, struct sk_buff *skb,  				  csum_unfold(*sum)));  }  EXPORT_SYMBOL(inet_proto_csum_replace16); - -int mac_pton(const char *s, u8 *mac) -{ -	int i; - -	/* XX:XX:XX:XX:XX:XX */ -	if (strlen(s) < 3 * ETH_ALEN - 1) -		return 0; - -	/* Don't dirty result unless string is valid MAC. */ -	for (i = 0; i < ETH_ALEN; i++) { -		if (!isxdigit(s[i * 3]) || !isxdigit(s[i * 3 + 1])) -			return 0; -		if (i != ETH_ALEN - 1 && s[i * 3 + 2] != ':') -			return 0; -	} -	for (i = 0; i < ETH_ALEN; i++) { -		mac[i] = (hex_to_bin(s[i * 3]) << 4) | hex_to_bin(s[i * 3 + 1]); -	} -	return 1; -} -EXPORT_SYMBOL(mac_pton); |