UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

net/atm/br2684.c: checkpatch cleanups 

Convert #include <asm... to #include <linux...
Use print_hex_dump
Spacing cleanups
Move logical continuation tests to end of previous line
80 column wrapping
Move goto branch label to column 1
Remove unnecessary single statement braces

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Joe Perches 2010-01-26 11:40:04 +00:00 committed by David S. Miller
parent f0a6cb118d
commit 641d729eb6
1 changed files with 31 additions and 41 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

72
net/atm/br2684.c
View File

@ -17,7 +17,7 @@
#include <linux/etherdevice.h> #include <linux/etherdevice.h>
#include <linux/rtnetlink.h> #include <linux/rtnetlink.h>
#include <linux/ip.h> #include <linux/ip.h>
#include <asm/uaccess.h> #include <linux/uaccess.h>
#include <net/arp.h> #include <net/arp.h>
#include <linux/atm.h> #include <linux/atm.h>
#include <linux/atmdev.h> #include <linux/atmdev.h>
@ -28,20 +28,14 @@
#include "common.h" #include "common.h"
#ifdef SKB_DEBUG
static void skb_debug(const struct sk_buff *skb) static void skb_debug(const struct sk_buff *skb)
{ {
#ifdef SKB_DEBUG
#define NUM2PRINT 50 #define NUM2PRINT 50
char buf[NUM2PRINT * 3 + 1]; /* 3 chars per byte */ print_hex_dump(KERN_DEBUG, "br2684: skb: ", DUMP_OFFSET,
int i = 0; 16, 1, skb->data, min(NUM2PRINT, skb->len), true);
for (i = 0; i < skb->len && i < NUM2PRINT; i++) {
sprintf(buf + i * 3, "%2.2x ", 0xff & skb->data[i]);
}
printk(KERN_DEBUG "br2684: skb: %s\n", buf);
}
#else
#define skb_debug(skb) do {} while (0)
#endif #endif
}
#define BR2684_ETHERTYPE_LEN 2 #define BR2684_ETHERTYPE_LEN 2
#define BR2684_PAD_LEN 2 #define BR2684_PAD_LEN 2
@ -70,7 +64,7 @@ struct br2684_vcc {
struct atm_vcc *atmvcc; struct atm_vcc *atmvcc;
struct net_device *device; struct net_device *device;
/* keep old push, pop functions for chaining */ /* keep old push, pop functions for chaining */
void (*old_push) (struct atm_vcc * vcc, struct sk_buff * skb); void (*old_push)(struct atm_vcc *vcc, struct sk_buff *skb);
void (*old_pop)(struct atm_vcc *vcc, struct sk_buff *skb); void (*old_pop)(struct atm_vcc *vcc, struct sk_buff *skb);
enum br2684_encaps encaps; enum br2684_encaps encaps;
struct list_head brvccs; struct list_head brvccs;
@ -302,7 +296,8 @@ static int br2684_setfilt(struct atm_vcc *atmvcc, void __user * arg)
struct br2684_dev *brdev; struct br2684_dev *brdev;
read_lock(&devs_lock); read_lock(&devs_lock);
brdev = BRPRIV(br2684_find_dev(&fs.ifspec)); brdev = BRPRIV(br2684_find_dev(&fs.ifspec));
if (brdev == NULL || list_empty(&brdev->brvccs) || brdev->brvccs.next != brdev->brvccs.prev) /* >1 VCC */ if (brdev == NULL || list_empty(&brdev->brvccs) ||
brdev->brvccs.next != brdev->brvccs.prev) /* >1 VCC */
brvcc = NULL; brvcc = NULL;
else else
brvcc = list_entry_brvcc(brdev->brvccs.next); brvcc = list_entry_brvcc(brdev->brvccs.next);
@ -378,29 +373,25 @@ static void br2684_push(struct atm_vcc *atmvcc, struct sk_buff *skb)
__skb_trim(skb, skb->len - 4); __skb_trim(skb, skb->len - 4);
/* accept packets that have "ipv[46]" in the snap header */ /* accept packets that have "ipv[46]" in the snap header */
if ((skb->len >= (sizeof(llc_oui_ipv4))) if ((skb->len >= (sizeof(llc_oui_ipv4))) &&
&& (memcmp(skb->data, llc_oui_ipv4,
(memcmp sizeof(llc_oui_ipv4) - BR2684_ETHERTYPE_LEN) == 0)) {
(skb->data, llc_oui_ipv4, if (memcmp(skb->data + 6, ethertype_ipv6,
sizeof(llc_oui_ipv4) - BR2684_ETHERTYPE_LEN) == 0)) { sizeof(ethertype_ipv6)) == 0)
if (memcmp
(skb->data + 6, ethertype_ipv6,
sizeof(ethertype_ipv6)) == 0)
skb->protocol = htons(ETH_P_IPV6); skb->protocol = htons(ETH_P_IPV6);
else if (memcmp else if (memcmp(skb->data + 6, ethertype_ipv4,
(skb->data + 6, ethertype_ipv4, sizeof(ethertype_ipv4)) == 0)
sizeof(ethertype_ipv4)) == 0)
skb->protocol = htons(ETH_P_IP); skb->protocol = htons(ETH_P_IP);
else else
goto error; goto error;
skb_pull(skb, sizeof(llc_oui_ipv4)); skb_pull(skb, sizeof(llc_oui_ipv4));
skb_reset_network_header(skb); skb_reset_network_header(skb);
skb->pkt_type = PACKET_HOST; skb->pkt_type = PACKET_HOST;
/* /*
* Let us waste some time for checking the encapsulation. * Let us waste some time for checking the encapsulation.
* Note, that only 7 char is checked so frames with a valid FCS * Note, that only 7 char is checked so frames with a valid FCS
* are also accepted (but FCS is not checked of course). * are also accepted (but FCS is not checked of course).
*/ */
} else if ((skb->len >= sizeof(llc_oui_pid_pad)) && } else if ((skb->len >= sizeof(llc_oui_pid_pad)) &&
(memcmp(skb->data, llc_oui_pid_pad, 7) == 0)) { (memcmp(skb->data, llc_oui_pid_pad, 7) == 0)) {
skb_pull(skb, sizeof(llc_oui_pid_pad)); skb_pull(skb, sizeof(llc_oui_pid_pad));
@ -495,12 +486,12 @@ static int br2684_regvcc(struct atm_vcc *atmvcc, void __user * arg)
err = -EEXIST; err = -EEXIST;
goto error; goto error;
} }
if (be.fcs_in != BR2684_FCSIN_NO || be.fcs_out != BR2684_FCSOUT_NO || if (be.fcs_in != BR2684_FCSIN_NO ||
be.fcs_auto || be.has_vpiid || be.send_padding || (be.encaps != be.fcs_out != BR2684_FCSOUT_NO ||
BR2684_ENCAPS_VC be.fcs_auto || be.has_vpiid || be.send_padding ||
&& be.encaps != (be.encaps != BR2684_ENCAPS_VC &&
BR2684_ENCAPS_LLC) be.encaps != BR2684_ENCAPS_LLC) ||
|| be.min_size != 0) { be.min_size != 0) {
err = -EINVAL; err = -EINVAL;
goto error; goto error;
} }
@ -541,7 +532,8 @@ static int br2684_regvcc(struct atm_vcc *atmvcc, void __user * arg)
} }
__module_get(THIS_MODULE); __module_get(THIS_MODULE);
return 0; return 0;
error:
error:
write_unlock_irq(&devs_lock); write_unlock_irq(&devs_lock);
kfree(brvcc); kfree(brvcc);
return err; return err;
@ -587,7 +579,7 @@ static void br2684_setup_routed(struct net_device *netdev)
INIT_LIST_HEAD(&brdev->brvccs); INIT_LIST_HEAD(&brdev->brvccs);
} }
static int br2684_create(void __user * arg) static int br2684_create(void __user *arg)
{ {
int err; int err;
struct net_device *netdev; struct net_device *netdev;
@ -597,9 +589,8 @@ static int br2684_create(void __user * arg)
pr_debug("\n"); pr_debug("\n");
if (copy_from_user(&ni, arg, sizeof ni)) { if (copy_from_user(&ni, arg, sizeof ni))
return -EFAULT; return -EFAULT;
}
if (ni.media & BR2684_FLAG_ROUTED) if (ni.media & BR2684_FLAG_ROUTED)
payload = p_routed; payload = p_routed;
@ -607,9 +598,8 @@ static int br2684_create(void __user * arg)
payload = p_bridged; payload = p_bridged;
ni.media &= 0xffff; /* strip flags */ ni.media &= 0xffff; /* strip flags */
if (ni.media != BR2684_MEDIA_ETHERNET || ni.mtu != 1500) { if (ni.media != BR2684_MEDIA_ETHERNET || ni.mtu != 1500)
return -EINVAL; return -EINVAL;
}
netdev = alloc_netdev(sizeof(struct br2684_dev), netdev = alloc_netdev(sizeof(struct br2684_dev),
ni.ifname[0] ? ni.ifname : "nas%d", ni.ifname[0] ? ni.ifname : "nas%d",