From c2bf14deb99602780a71f9290d0936d143fb1a38 Mon Sep 17 00:00:00 2001 From: Florian Fainelli Date: Fri, 24 Apr 2009 14:35:19 +0000 Subject: remove 2.6.24, no target is relying on it SVN-Revision: 15388 --- .../patches-2.6.24/601-br2684-routed-support.patch | 728 --------------------- 1 file changed, 728 deletions(-) delete mode 100644 target/linux/generic-2.6/patches-2.6.24/601-br2684-routed-support.patch (limited to 'target/linux/generic-2.6/patches-2.6.24/601-br2684-routed-support.patch') diff --git a/target/linux/generic-2.6/patches-2.6.24/601-br2684-routed-support.patch b/target/linux/generic-2.6/patches-2.6.24/601-br2684-routed-support.patch deleted file mode 100644 index 3bc2e26..0000000 --- a/target/linux/generic-2.6/patches-2.6.24/601-br2684-routed-support.patch +++ /dev/null @@ -1,728 +0,0 @@ ---- a/include/linux/atmbr2684.h -+++ b/include/linux/atmbr2684.h -@@ -14,6 +14,9 @@ - #define BR2684_MEDIA_FDDI (3) - #define BR2684_MEDIA_802_6 (4) /* 802.6 */ - -+ /* used only at device creation: */ -+#define BR2684_FLAG_ROUTED (1<<16) /* payload is routed, not bridged */ -+ - /* - * Is there FCS inbound on this VC? This currently isn't supported. - */ -@@ -36,15 +39,22 @@ - #define BR2684_ENCAPS_AUTODETECT (2) /* Unsuported */ - - /* -+ * Is this VC bridged or routed? -+ */ -+ -+#define BR2684_PAYLOAD_ROUTED (0) -+#define BR2684_PAYLOAD_BRIDGED (1) -+ -+/* - * This is for the ATM_NEWBACKENDIF call - these are like socket families: - * the first element of the structure is the backend number and the rest - * is per-backend specific - */ - struct atm_newif_br2684 { -- atm_backend_t backend_num; /* ATM_BACKEND_BR2684 */ -- int media; /* BR2684_MEDIA_* */ -- char ifname[IFNAMSIZ]; -- int mtu; -+ atm_backend_t backend_num; /* ATM_BACKEND_BR2684 */ -+ int media; /* BR2684_MEDIA_*, flags in upper bits */ -+ char ifname[IFNAMSIZ]; -+ int mtu; - }; - - /* -@@ -55,10 +65,10 @@ struct atm_newif_br2684 { - #define BR2684_FIND_BYNUM (1) - #define BR2684_FIND_BYIFNAME (2) - struct br2684_if_spec { -- int method; /* BR2684_FIND_* */ -+ int method; /* BR2684_FIND_* */ - union { -- char ifname[IFNAMSIZ]; -- int devnum; -+ char ifname[IFNAMSIZ]; -+ int devnum; - } spec; - }; - -@@ -68,16 +78,16 @@ struct br2684_if_spec { - * is per-backend specific - */ - struct atm_backend_br2684 { -- atm_backend_t backend_num; /* ATM_BACKEND_BR2684 */ -+ atm_backend_t backend_num; /* ATM_BACKEND_BR2684 */ - struct br2684_if_spec ifspec; -- int fcs_in; /* BR2684_FCSIN_* */ -- int fcs_out; /* BR2684_FCSOUT_* */ -- int fcs_auto; /* 1: fcs_{in,out} disabled if no FCS rx'ed */ -- int encaps; /* BR2684_ENCAPS_* */ -- int has_vpiid; /* 1: use vpn_id - Unsupported */ -- __u8 vpn_id[7]; -- int send_padding; /* unsupported */ -- int min_size; /* we will pad smaller packets than this */ -+ int fcs_in; /* BR2684_FCSIN_* */ -+ int fcs_out; /* BR2684_FCSOUT_* */ -+ int fcs_auto; /* 1: fcs_{in,out} disabled if no FCS rx'ed */ -+ int encaps; /* BR2684_ENCAPS_* */ -+ int has_vpiid; /* 1: use vpn_id - Unsupported */ -+ __u8 vpn_id[7]; -+ int send_padding; /* unsupported */ -+ int min_size; /* we will pad smaller packets than this */ - }; - - /* -@@ -86,8 +96,8 @@ struct atm_backend_br2684 { - * efficient per-if in/out filters, this support will be removed - */ - struct br2684_filter { -- __be32 prefix; /* network byte order */ -- __be32 netmask; /* 0 = disable filter */ -+ __be32 prefix; /* network byte order */ -+ __be32 netmask; /* 0 = disable filter */ - }; - - struct br2684_filter_set { -@@ -95,7 +105,13 @@ struct br2684_filter_set { - struct br2684_filter filter; - }; - -+enum br2684_payload { -+ p_routed = BR2684_PAYLOAD_ROUTED, -+ p_bridged = BR2684_PAYLOAD_BRIDGED, -+}; -+ - #define BR2684_SETFILT _IOW( 'a', ATMIOC_BACKEND + 0, \ - struct br2684_filter_set) - - #endif /* _LINUX_ATMBR2684_H */ -+ ---- a/net/atm/br2684.c -+++ b/net/atm/br2684.c -@@ -1,8 +1,10 @@ - /* --Experimental ethernet netdevice using ATM AAL5 as underlying carrier --(RFC1483 obsoleted by RFC2684) for Linux 2.4 --Author: Marcell GAL, 2000, XDSL Ltd, Hungary --*/ -+ * Ethernet netdevice using ATM AAL5 as underlying carrier -+ * (RFC1483 obsoleted by RFC2684) for Linux -+ * -+ * Authors: Marcell GAL, 2000, XDSL Ltd, Hungary -+ * Eric Kinzie, 2006-2007, US Naval Research Laboratory -+ */ - - #include - #include -@@ -39,21 +41,35 @@ static void skb_debug(const struct sk_bu - #define skb_debug(skb) do {} while (0) - #endif - -+#define BR2684_ETHERTYPE_LEN 2 -+#define BR2684_PAD_LEN 2 -+ -+#define LLC 0xaa, 0xaa, 0x03 -+#define SNAP_BRIDGED 0x00, 0x80, 0xc2 -+#define SNAP_ROUTED 0x00, 0x00, 0x00 -+#define PID_ETHERNET 0x00, 0x07 -+#define ETHERTYPE_IPV4 0x08, 0x00 -+#define ETHERTYPE_IPV6 0x86, 0xdd -+#define PAD_BRIDGED 0x00, 0x00 -+ -+static unsigned char ethertype_ipv4[] = { ETHERTYPE_IPV4 }; -+static unsigned char ethertype_ipv6[] = { ETHERTYPE_IPV6 }; - static unsigned char llc_oui_pid_pad[] = -- { 0xAA, 0xAA, 0x03, 0x00, 0x80, 0xC2, 0x00, 0x07, 0x00, 0x00 }; --#define PADLEN (2) -+ { LLC, SNAP_BRIDGED, PID_ETHERNET, PAD_BRIDGED }; -+static unsigned char llc_oui_ipv4[] = { LLC, SNAP_ROUTED, ETHERTYPE_IPV4 }; -+static unsigned char llc_oui_ipv6[] = { LLC, SNAP_ROUTED, ETHERTYPE_IPV6 }; - - enum br2684_encaps { -- e_vc = BR2684_ENCAPS_VC, -+ e_vc = BR2684_ENCAPS_VC, - e_llc = BR2684_ENCAPS_LLC, - }; - - struct br2684_vcc { -- struct atm_vcc *atmvcc; -+ struct atm_vcc *atmvcc; - struct net_device *device; -- /* keep old push,pop functions for chaining */ -- void (*old_push)(struct atm_vcc *vcc,struct sk_buff *skb); -- /* void (*old_pop)(struct atm_vcc *vcc,struct sk_buff *skb); */ -+ /* keep old push, pop functions for chaining */ -+ void (*old_push) (struct atm_vcc * vcc, struct sk_buff * skb); -+ /* void (*old_pop)(struct atm_vcc *vcc, struct sk_buff *skb); */ - enum br2684_encaps encaps; - struct list_head brvccs; - #ifdef CONFIG_ATM_BR2684_IPFILTER -@@ -66,9 +82,10 @@ struct br2684_dev { - struct net_device *net_dev; - struct list_head br2684_devs; - int number; -- struct list_head brvccs; /* one device <=> one vcc (before xmas) */ -+ struct list_head brvccs; /* one device <=> one vcc (before xmas) */ - struct net_device_stats stats; - int mac_was_set; -+ enum br2684_payload payload; - }; - - /* -@@ -84,7 +101,7 @@ static LIST_HEAD(br2684_devs); - - static inline struct br2684_dev *BRPRIV(const struct net_device *net_dev) - { -- return (struct br2684_dev *) net_dev->priv; -+ return (struct br2684_dev *)net_dev->priv; - } - - static inline struct net_device *list_entry_brdev(const struct list_head *le) -@@ -94,7 +111,7 @@ static inline struct net_device *list_en - - static inline struct br2684_vcc *BR2684_VCC(const struct atm_vcc *atmvcc) - { -- return (struct br2684_vcc *) (atmvcc->user_back); -+ return (struct br2684_vcc *)(atmvcc->user_back); - } - - static inline struct br2684_vcc *list_entry_brvcc(const struct list_head *le) -@@ -132,10 +149,11 @@ static struct net_device *br2684_find_de - * otherwise false - */ - static int br2684_xmit_vcc(struct sk_buff *skb, struct br2684_dev *brdev, -- struct br2684_vcc *brvcc) -+ struct br2684_vcc *brvcc) - { - struct atm_vcc *atmvcc; - int minheadroom = (brvcc->encaps == e_llc) ? 10 : 2; -+ - if (skb_headroom(skb) < minheadroom) { - struct sk_buff *skb2 = skb_realloc_headroom(skb, minheadroom); - brvcc->copies_needed++; -@@ -146,23 +164,48 @@ static int br2684_xmit_vcc(struct sk_buf - } - skb = skb2; - } -- skb_push(skb, minheadroom); -- if (brvcc->encaps == e_llc) -- skb_copy_to_linear_data(skb, llc_oui_pid_pad, 10); -- else -- memset(skb->data, 0, 2); -+ -+ if (brvcc->encaps == e_llc) { -+ if (brdev->payload == p_bridged) { -+ skb_push(skb, sizeof(llc_oui_pid_pad)); -+ skb_copy_to_linear_data(skb, llc_oui_pid_pad, -+ sizeof(llc_oui_pid_pad)); -+ } else if (brdev->payload == p_routed) { -+ unsigned short prot = ntohs(skb->protocol); -+ -+ skb_push(skb, sizeof(llc_oui_ipv4)); -+ switch (prot) { -+ case ETH_P_IP: -+ skb_copy_to_linear_data(skb, llc_oui_ipv4, -+ sizeof(llc_oui_ipv4)); -+ break; -+ case ETH_P_IPV6: -+ skb_copy_to_linear_data(skb, llc_oui_ipv6, -+ sizeof(llc_oui_ipv6)); -+ break; -+ default: -+ dev_kfree_skb(skb); -+ return 0; -+ } -+ } -+ } else { -+ skb_push(skb, 2); -+ if (brdev->payload == p_bridged) -+ memset(skb->data, 0, 2); -+ } - skb_debug(skb); - - ATM_SKB(skb)->vcc = atmvcc = brvcc->atmvcc; - pr_debug("atm_skb(%p)->vcc(%p)->dev(%p)\n", skb, atmvcc, atmvcc->dev); - if (!atm_may_send(atmvcc, skb->truesize)) { -- /* we free this here for now, because we cannot know in a higher -- layer whether the skb point it supplied wasn't freed yet. -- now, it always is. -- */ -+ /* -+ * We free this here for now, because we cannot know in a higher -+ * layer whether the skb pointer it supplied wasn't freed yet. -+ * Now, it always is. -+ */ - dev_kfree_skb(skb); - return 0; -- } -+ } - atomic_add(skb->truesize, &sk_atm(atmvcc)->sk_wmem_alloc); - ATM_SKB(skb)->atm_options = atmvcc->atm_options; - brdev->stats.tx_packets++; -@@ -172,10 +215,9 @@ static int br2684_xmit_vcc(struct sk_buf - } - - static inline struct br2684_vcc *pick_outgoing_vcc(struct sk_buff *skb, -- struct br2684_dev *brdev) -+ struct br2684_dev *brdev) - { -- return list_empty(&brdev->brvccs) ? NULL : -- list_entry_brvcc(brdev->brvccs.next); /* 1 vcc/dev right now */ -+ return list_empty(&brdev->brvccs) ? NULL : list_entry_brvcc(brdev->brvccs.next); /* 1 vcc/dev right now */ - } - - static int br2684_start_xmit(struct sk_buff *skb, struct net_device *dev) -@@ -199,11 +241,10 @@ static int br2684_start_xmit(struct sk_b - /* - * We should probably use netif_*_queue() here, but that - * involves added complication. We need to walk before -- * we can run -+ * we can run. -+ * -+ * Don't free here! this pointer might be no longer valid! - */ -- /* don't free here! this pointer might be no longer valid! -- dev_kfree_skb(skb); -- */ - brdev->stats.tx_errors++; - brdev->stats.tx_fifo_errors++; - } -@@ -217,12 +258,11 @@ static struct net_device_stats *br2684_g - return &BRPRIV(dev)->stats; - } - -- - /* - * We remember when the MAC gets set, so we don't override it later with - * the ESI of the ATM card of the first VC - */ --static int (*my_eth_mac_addr)(struct net_device *, void *); -+static int (*my_eth_mac_addr) (struct net_device *, void *); - static int br2684_mac_addr(struct net_device *dev, void *p) - { - int err = my_eth_mac_addr(dev, p); -@@ -233,7 +273,7 @@ static int br2684_mac_addr(struct net_de - - #ifdef CONFIG_ATM_BR2684_IPFILTER - /* this IOCTL is experimental. */ --static int br2684_setfilt(struct atm_vcc *atmvcc, void __user *arg) -+static int br2684_setfilt(struct atm_vcc *atmvcc, void __user * arg) - { - struct br2684_vcc *brvcc; - struct br2684_filter_set fs; -@@ -243,13 +283,12 @@ static int br2684_setfilt(struct atm_vcc - if (fs.ifspec.method != BR2684_FIND_BYNOTHING) { - /* - * This is really a per-vcc thing, but we can also search -- * by device -+ * by device. - */ - struct br2684_dev *brdev; - read_lock(&devs_lock); - 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; - else - brvcc = list_entry_brvcc(brdev->brvccs.next); -@@ -267,15 +306,16 @@ static inline int - packet_fails_filter(__be16 type, struct br2684_vcc *brvcc, struct sk_buff *skb) - { - if (brvcc->filter.netmask == 0) -- return 0; /* no filter in place */ -+ return 0; /* no filter in place */ - if (type == htons(ETH_P_IP) && -- (((struct iphdr *) (skb->data))->daddr & brvcc->filter. -+ (((struct iphdr *)(skb->data))->daddr & brvcc->filter. - netmask) == brvcc->filter.prefix) - return 0; - if (type == htons(ETH_P_ARP)) - return 0; -- /* TODO: we should probably filter ARPs too.. don't want to have -- * them returning values that don't make sense, or is that ok? -+ /* -+ * TODO: we should probably filter ARPs too.. don't want to have -+ * them returning values that don't make sense, or is that ok? - */ - return 1; /* drop */ - } -@@ -299,7 +339,6 @@ static void br2684_push(struct atm_vcc * - struct br2684_vcc *brvcc = BR2684_VCC(atmvcc); - struct net_device *net_dev = brvcc->device; - struct br2684_dev *brdev = BRPRIV(net_dev); -- int plen = sizeof(llc_oui_pid_pad) + ETH_HLEN; - - pr_debug("br2684_push\n"); - -@@ -320,35 +359,58 @@ static void br2684_push(struct atm_vcc * - atm_return(atmvcc, skb->truesize); - pr_debug("skb from brdev %p\n", brdev); - if (brvcc->encaps == e_llc) { -- /* let us waste some time for checking the encapsulation. -- Note, that only 7 char is checked so frames with a valid FCS -- are also accepted (but FCS is not checked of course) */ -- if (memcmp(skb->data, llc_oui_pid_pad, 7)) { -+ -+ if (skb->len > 7 && skb->data[7] == 0x01) -+ __skb_trim(skb, skb->len - 4); -+ -+ /* accept packets that have "ipv[46]" in the snap header */ -+ if ((skb->len >= (sizeof(llc_oui_ipv4))) -+ && -+ (memcmp -+ (skb->data, llc_oui_ipv4, -+ sizeof(llc_oui_ipv4) - BR2684_ETHERTYPE_LEN) == 0)) { -+ if (memcmp -+ (skb->data + 6, ethertype_ipv6, -+ sizeof(ethertype_ipv6)) == 0) -+ skb->protocol = __constant_htons(ETH_P_IPV6); -+ else if (memcmp -+ (skb->data + 6, ethertype_ipv4, -+ sizeof(ethertype_ipv4)) == 0) -+ skb->protocol = __constant_htons(ETH_P_IP); -+ else { -+ brdev->stats.rx_errors++; -+ dev_kfree_skb(skb); -+ return; -+ } -+ skb_pull(skb, sizeof(llc_oui_ipv4)); -+ skb_reset_network_header(skb); -+ skb->pkt_type = PACKET_HOST; -+ /* -+ * Let us waste some time for checking the encapsulation. -+ * Note, that only 7 char is checked so frames with a valid FCS -+ * are also accepted (but FCS is not checked of course). -+ */ -+ } else if ((skb->len >= sizeof(llc_oui_pid_pad)) && -+ (memcmp(skb->data, llc_oui_pid_pad, 7) == 0)) { -+ skb_pull(skb, sizeof(llc_oui_pid_pad)); -+ skb->protocol = eth_type_trans(skb, net_dev); -+ } else { - brdev->stats.rx_errors++; - dev_kfree_skb(skb); - return; - } - -- /* Strip FCS if present */ -- if (skb->len > 7 && skb->data[7] == 0x01) -- __skb_trim(skb, skb->len - 4); - } else { -- plen = PADLEN + ETH_HLEN; /* pad, dstmac,srcmac, ethtype */ - /* first 2 chars should be 0 */ - if (*((u16 *) (skb->data)) != 0) { - brdev->stats.rx_errors++; - dev_kfree_skb(skb); - return; - } -- } -- if (skb->len < plen) { -- brdev->stats.rx_errors++; -- dev_kfree_skb(skb); /* dev_ not needed? */ -- return; -+ skb_pull(skb, BR2684_PAD_LEN + ETH_HLEN); /* pad, dstmac, srcmac, ethtype */ -+ skb->protocol = eth_type_trans(skb, net_dev); - } - -- skb_pull(skb, plen - ETH_HLEN); -- skb->protocol = eth_type_trans(skb, net_dev); - #ifdef CONFIG_ATM_BR2684_IPFILTER - if (unlikely(packet_fails_filter(skb->protocol, brvcc, skb))) { - brdev->stats.rx_dropped++; -@@ -372,11 +434,12 @@ static void br2684_push(struct atm_vcc * - netif_rx(skb); - } - --static int br2684_regvcc(struct atm_vcc *atmvcc, void __user *arg) -+/* -+ * Assign a vcc to a dev -+ * Note: we do not have explicit unassign, but look at _push() -+ */ -+static int br2684_regvcc(struct atm_vcc *atmvcc, void __user * arg) - { --/* assign a vcc to a dev --Note: we do not have explicit unassign, but look at _push() --*/ - int err; - struct br2684_vcc *brvcc; - struct sk_buff *skb; -@@ -395,7 +458,7 @@ Note: we do not have explicit unassign, - net_dev = br2684_find_dev(&be.ifspec); - if (net_dev == NULL) { - printk(KERN_ERR -- "br2684: tried to attach to non-existant device\n"); -+ "br2684: tried to attach to non-existant device\n"); - err = -ENXIO; - goto error; - } -@@ -411,13 +474,15 @@ Note: we do not have explicit unassign, - } - if (be.fcs_in != BR2684_FCSIN_NO || be.fcs_out != BR2684_FCSOUT_NO || - be.fcs_auto || be.has_vpiid || be.send_padding || (be.encaps != -- BR2684_ENCAPS_VC && be.encaps != BR2684_ENCAPS_LLC) || -- be.min_size != 0) { -+ BR2684_ENCAPS_VC -+ && be.encaps != -+ BR2684_ENCAPS_LLC) -+ || be.min_size != 0) { - err = -EINVAL; - goto error; - } -- pr_debug("br2684_regvcc vcc=%p, encaps=%d, brvcc=%p\n", atmvcc, be.encaps, -- brvcc); -+ pr_debug("br2684_regvcc vcc=%p, encaps=%d, brvcc=%p\n", atmvcc, -+ be.encaps, brvcc); - if (list_empty(&brdev->brvccs) && !brdev->mac_was_set) { - unsigned char *esi = atmvcc->dev->esi; - if (esi[0] | esi[1] | esi[2] | esi[3] | esi[4] | esi[5]) -@@ -430,7 +495,7 @@ Note: we do not have explicit unassign, - brvcc->device = net_dev; - brvcc->atmvcc = atmvcc; - atmvcc->user_back = brvcc; -- brvcc->encaps = (enum br2684_encaps) be.encaps; -+ brvcc->encaps = (enum br2684_encaps)be.encaps; - brvcc->old_push = atmvcc->push; - barrier(); - atmvcc->push = br2684_push; -@@ -461,7 +526,7 @@ Note: we do not have explicit unassign, - } - __module_get(THIS_MODULE); - return 0; -- error: -+ error: - write_unlock_irq(&devs_lock); - kfree(brvcc); - return err; -@@ -482,25 +547,52 @@ static void br2684_setup(struct net_devi - INIT_LIST_HEAD(&brdev->brvccs); - } - --static int br2684_create(void __user *arg) -+static void br2684_setup_routed(struct net_device *netdev) -+{ -+ struct br2684_dev *brdev = BRPRIV(netdev); -+ brdev->net_dev = netdev; -+ -+ netdev->hard_header_len = 0; -+ my_eth_mac_addr = netdev->set_mac_address; -+ netdev->set_mac_address = br2684_mac_addr; -+ netdev->hard_start_xmit = br2684_start_xmit; -+ netdev->get_stats = br2684_get_stats; -+ netdev->addr_len = 0; -+ netdev->mtu = 1500; -+ netdev->type = ARPHRD_PPP; -+ netdev->flags = IFF_POINTOPOINT | IFF_NOARP | IFF_MULTICAST; -+ netdev->tx_queue_len = 100; -+ INIT_LIST_HEAD(&brdev->brvccs); -+} -+ -+static int br2684_create(void __user * arg) - { - int err; - struct net_device *netdev; - struct br2684_dev *brdev; - struct atm_newif_br2684 ni; -+ enum br2684_payload payload; - - pr_debug("br2684_create\n"); - - if (copy_from_user(&ni, arg, sizeof ni)) { - return -EFAULT; - } -+ -+ if (ni.media & BR2684_FLAG_ROUTED) -+ payload = p_routed; -+ else -+ payload = p_bridged; -+ ni.media &= 0xffff; /* strip flags */ -+ - if (ni.media != BR2684_MEDIA_ETHERNET || ni.mtu != 1500) { - return -EINVAL; - } - - netdev = alloc_netdev(sizeof(struct br2684_dev), - ni.ifname[0] ? ni.ifname : "nas%d", -- br2684_setup); -+ (payload == p_routed) ? -+ br2684_setup_routed : br2684_setup); - if (!netdev) - return -ENOMEM; - -@@ -516,6 +608,7 @@ static int br2684_create(void __user *ar - } - - write_lock_irq(&devs_lock); -+ brdev->payload = payload; - brdev->number = list_empty(&br2684_devs) ? 1 : - BRPRIV(list_entry_brdev(br2684_devs.prev))->number + 1; - list_add_tail(&brdev->br2684_devs, &br2684_devs); -@@ -528,16 +621,16 @@ static int br2684_create(void __user *ar - * -ENOIOCTLCMD for any unrecognized ioctl - */ - static int br2684_ioctl(struct socket *sock, unsigned int cmd, -- unsigned long arg) -+ unsigned long arg) - { - struct atm_vcc *atmvcc = ATM_SD(sock); - void __user *argp = (void __user *)arg; -+ atm_backend_t b; - - int err; -- switch(cmd) { -+ switch (cmd) { - case ATM_SETBACKEND: -- case ATM_NEWBACKENDIF: { -- atm_backend_t b; -+ case ATM_NEWBACKENDIF: - err = get_user(b, (atm_backend_t __user *) argp); - if (err) - return -EFAULT; -@@ -549,7 +642,6 @@ static int br2684_ioctl(struct socket *s - return br2684_regvcc(atmvcc, argp); - else - return br2684_create(argp); -- } - #ifdef CONFIG_ATM_BR2684_IPFILTER - case BR2684_SETFILT: - if (atmvcc->push != br2684_push) -@@ -557,6 +649,7 @@ static int br2684_ioctl(struct socket *s - if (!capable(CAP_NET_ADMIN)) - return -EPERM; - err = br2684_setfilt(atmvcc, argp); -+ - return err; - #endif /* CONFIG_ATM_BR2684_IPFILTER */ - } -@@ -564,24 +657,25 @@ static int br2684_ioctl(struct socket *s - } - - static struct atm_ioctl br2684_ioctl_ops = { -- .owner = THIS_MODULE, -- .ioctl = br2684_ioctl, -+ .owner = THIS_MODULE, -+ .ioctl = br2684_ioctl, - }; - -- - #ifdef CONFIG_PROC_FS --static void *br2684_seq_start(struct seq_file *seq, loff_t *pos) -+static void *br2684_seq_start(struct seq_file *seq, loff_t * pos) -+ __acquires(devs_lock) - { - read_lock(&devs_lock); - return seq_list_start(&br2684_devs, *pos); - } - --static void *br2684_seq_next(struct seq_file *seq, void *v, loff_t *pos) -+static void *br2684_seq_next(struct seq_file *seq, void *v, loff_t * pos) - { - return seq_list_next(v, &br2684_devs, pos); - } - - static void br2684_seq_stop(struct seq_file *seq, void *v) -+ __releases(devs_lock) - { - read_unlock(&devs_lock); - } -@@ -589,7 +683,7 @@ static void br2684_seq_stop(struct seq_f - static int br2684_seq_show(struct seq_file *seq, void *v) - { - const struct br2684_dev *brdev = list_entry(v, struct br2684_dev, -- br2684_devs); -+ br2684_devs); - const struct net_device *net_dev = brdev->net_dev; - const struct br2684_vcc *brvcc; - DECLARE_MAC_BUF(mac); -@@ -601,21 +695,19 @@ static int br2684_seq_show(struct seq_fi - brdev->mac_was_set ? "set" : "auto"); - - list_for_each_entry(brvcc, &brdev->brvccs, brvccs) { -- seq_printf(seq, " vcc %d.%d.%d: encaps=%s" -- ", failed copies %u/%u" -- "\n", brvcc->atmvcc->dev->number, -- brvcc->atmvcc->vpi, brvcc->atmvcc->vci, -- (brvcc->encaps == e_llc) ? "LLC" : "VC" -- , brvcc->copies_failed -- , brvcc->copies_needed -- ); -+ seq_printf(seq, " vcc %d.%d.%d: encaps=%s payload=%s" -+ ", failed copies %u/%u" -+ "\n", brvcc->atmvcc->dev->number, -+ brvcc->atmvcc->vpi, brvcc->atmvcc->vci, -+ (brvcc->encaps == e_llc) ? "LLC" : "VC", -+ (brdev->payload == p_bridged) ? "bridged" : "routed", -+ brvcc->copies_failed, brvcc->copies_needed); - #ifdef CONFIG_ATM_BR2684_IPFILTER - #define b1(var, byte) ((u8 *) &brvcc->filter.var)[byte] - #define bs(var) b1(var, 0), b1(var, 1), b1(var, 2), b1(var, 3) -- if (brvcc->filter.netmask != 0) -- seq_printf(seq, " filter=%d.%d.%d.%d/" -- "%d.%d.%d.%d\n", -- bs(prefix), bs(netmask)); -+ if (brvcc->filter.netmask != 0) -+ seq_printf(seq, " filter=%d.%d.%d.%d/" -+ "%d.%d.%d.%d\n", bs(prefix), bs(netmask)); - #undef bs - #undef b1 - #endif /* CONFIG_ATM_BR2684_IPFILTER */ -@@ -625,9 +717,9 @@ static int br2684_seq_show(struct seq_fi - - static const struct seq_operations br2684_seq_ops = { - .start = br2684_seq_start, -- .next = br2684_seq_next, -- .stop = br2684_seq_stop, -- .show = br2684_seq_show, -+ .next = br2684_seq_next, -+ .stop = br2684_seq_stop, -+ .show = br2684_seq_show, - }; - - static int br2684_proc_open(struct inode *inode, struct file *file) -@@ -636,26 +728,28 @@ static int br2684_proc_open(struct inode - } - - static const struct file_operations br2684_proc_ops = { -- .owner = THIS_MODULE, -- .open = br2684_proc_open, -- .read = seq_read, -- .llseek = seq_lseek, -+ .owner = THIS_MODULE, -+ .open = br2684_proc_open, -+ .read = seq_read, -+ .llseek = seq_lseek, - .release = seq_release, - }; - - extern struct proc_dir_entry *atm_proc_root; /* from proc.c */ --#endif -+#endif /* CONFIG_PROC_FS */ - - static int __init br2684_init(void) - { - #ifdef CONFIG_PROC_FS - struct proc_dir_entry *p; -+ - if ((p = create_proc_entry("br2684", 0, atm_proc_root)) == NULL) -- return -ENOMEM; -- p->proc_fops = &br2684_proc_ops; -+ return -ENOMEM; -+ p->proc_fops = &br2684_proc_ops; - #endif -- register_atm_ioctl(&br2684_ioctl_ops); -- return 0; -+ register_atm_ioctl(&br2684_ioctl_ops); -+ return 0; -+ - } - - static void __exit br2684_exit(void) -@@ -689,3 +783,4 @@ module_exit(br2684_exit); - MODULE_AUTHOR("Marcell GAL"); - MODULE_DESCRIPTION("RFC2684 bridged protocols over ATM/AAL5"); - MODULE_LICENSE("GPL"); -+ -- cgit v1.1