hdlc bridge

From: Ferenci Daniel
Date: Mon Nov 15 2004 - 09:14:23 EST


Hi Mr. Eis,
Hi linux folks,

excuse the newcommer, this is my first attempt to make a contribution.
I tried to keep all advices from the Documentation.

Change log:

HDLC bridge is kind of a stack for hdlc bridging.
It can be usefull for sniffing hdlc traffic (within two hdlc interface).

- why the kernel needed patching

I didn't find such functionality within the kernel tree.

- the overall design approach in the patch

HDLC bridge is one of the network stacs.
Like other x25,lapb stacks also hdlc bridge stack is connected to the HW device via sethdlc command.

sethdlc hdlc0 hdlc-bridge
sethdlc hdlc1 hdlc-bridge

If you have two hdlc cards (like MOXA or TAHOE or any other) you can swith on bridging
via updated command sethdlc (new version is placed at htp://www.dafe.net/x25/) like:
and afterwards all traffic incoming to hdlc0 card is sent from hdlc1 card and vice versa.
If you have switched #define DEBUG_PKT you can found all passing hdlc traffic in dmesg.

- implementation details

Principles of hdlc bridge stack is are the same like x25 or lapb stack.

- testing results

I tested it several times on several ATM maschines and CISCO devices and it worked without problem.

Signed-off-by: Daniel Ferenci <dafe@xxxxxxxx>

---
diff --new-file -u --recursive linux-2.4.25/include/net/hdlcbridge.h linux-2.4.25.intel/include/net/hdlcbridge.h
--- linux-2.4.25/include/net/hdlcbridge.h Thu Jan 1 01:00:00 1970
+++ linux-2.4.25.intel/include/net/hdlcbridge.h Sat Nov 13 08:21:12 2004
@@ -0,0 +1,33 @@
+#ifndef _HDLCBRIDGE_H
+#define _HDLCBRIDGE_H
+
+/*
+ Definitions for hdlc bridge
+ + History
+ 12.11.2004 Daniel Ferenci creation
+*/
+
+
+typedef enum {
+ PHYS_OK,
+ PHYS_NOK
+} STAT;
+
+typedef struct phys_bridge_register_struct {
+ int (*tx)(struct sk_buff *skb, struct net_device *dev);
+} PHYS_BRIDGE_REGISTER_STRUCT;
+
+typedef struct bridge_pair
+{
+ hdlc_device *hdlc1;
+ struct phys_bridge_register_struct *cb1;
+ hdlc_device *hdlc2;
+ struct phys_bridge_register_struct *cb2;
+} BRIDGE_PAIR;
+
+int phys_bridge_register(hdlc_device *hdlc, struct phys_bridge_register_struct *callbacks);
+int phys_bridge_unregister(hdlc_device *hdlc);
+int phys_bridge_data_received(hdlc_device *hdlc, struct sk_buff *skb);
+
+#endif /* _HDLCBRIDGE_H */
--- linux-2.4.25/net/Config.in Fri Nov 28 19:26:21 2003
+++ linux-2.4.25.intel/net/Config.in Sat Nov 13 08:22:39 2004
@@ -1,6 +1,8 @@
#
# Network configuration
#
+# 12.11.2004 Daniel Ferenci added parts for hdlcbridge
+
mainmenu_option next_comment
comment 'Networking options'
tristate 'Packet socket' CONFIG_PACKET
@@ -9,6 +11,9 @@
fi

tristate 'Netlink device emulation' CONFIG_NETLINK_DEV
+tristate 'Netlink sockets' CONFIG_NETLINK
+
+tristate 'HDLC Bridging' CONFIG_HDLC_BRIDGE

bool 'Network packet filtering (replaces ipchains)' CONFIG_NETFILTER
if [ "$CONFIG_NETFILTER" = "y" ]; then
--- linux-2.4.25/net/Makefile Fri Nov 28 19:26:21 2003
+++ linux-2.4.25.intel/net/Makefile Sat Nov 13 08:22:14 2004
@@ -4,6 +4,9 @@
# 2 Sep 2000, Christoph Hellwig <hch@xxxxxxxxxxxxx>
# Rewritten to use lists instead of if-statements.
#
+# 12.11.2004 Daniel Ferenci added hdlcbridge files
+
+

O_TARGET := network.o

@@ -46,6 +49,7 @@
subdir-$(CONFIG_DECNET) += decnet
subdir-$(CONFIG_ECONET) += econet
subdir-$(CONFIG_VLAN_8021Q) += 8021q
+subdir-$(CONFIG_HDLC_BRIDGE) += hdlcbridge

ifeq ($(CONFIG_NETFILTER),y)
mod-subdirs += ipv4/ipvs
diff --new-file -u --recursive linux-2.4.25/net/hdlcbridge/Makefile linux-2.4.25.intel/net/hdlcbridge/Makefile
--- linux-2.4.25/net/hdlcbridge/Makefile Thu Jan 1 01:00:00 1970
+++ linux-2.4.25.intel/net/hdlcbridge/Makefile Sat Nov 13 08:24:48 2004
@@ -0,0 +1,22 @@
+#
+# Makefile for the Linux HDLC Bridge
+#
+# History:
+# 12.11.2004 Daniel Ferenci creation
+#
+# Note! Dependencies are done automagically by 'make dep', which also
+# removes any old dependencies. DON'T put your own dependencies here
+# unless it's something special (ie not a .c file).
+#
+# Note 2! The CFLAGS definition is now in the main makefile...
+
+
+O_TARGET := hdlcbridge.o
+
+export-objs := hdlc_bridge.o
+
+obj-y := hdlc_bridge.o
+obj-m := $(O_TARGET)
+
+include $(TOPDIR)/Rules.make
+
diff --new-file -u --recursive linux-2.4.25/net/hdlcbridge/hdlc_bridge.c linux-2.4.25.intel/net/hdlcbridge/hdlc_bridge.c
--- linux-2.4.25/net/hdlcbridge/hdlc_bridge.c Thu Jan 1 01:00:00 1970
+++ linux-2.4.25.intel/net/hdlcbridge/hdlc_bridge.c Sat Nov 13 08:24:12 2004
@@ -0,0 +1,123 @@
+#include <linux/module.h>
+#include <linux/kernel.h>
+#include <linux/slab.h>
+#include <linux/poll.h>
+#include <linux/errno.h>
+#include <linux/if_arp.h>
+#include <linux/init.h>
+#include <linux/skbuff.h>
+#include <linux/pkt_sched.h>
+#include <linux/inetdevice.h>
+#include <linux/lapb.h>
+#include <linux/rtnetlink.h>
+#include <linux/hdlc.h>
+#include <net/hdlcbridge.h>
+/* + Hdlc bridge main file
+ + hdlc bridge:
+ with sethdlc binary register first two hdlc devices
+ data will be bridged then
+ + History:
+ 12.11.2004 Daniel Ferenci creation
+ +*/
+
+
+/*
+ Register
+ register two first devices not more
+*/
+static struct bridge_pair my_pair = {NULL, NULL, NULL, NULL};
+
+int phys_bridge_register(hdlc_device *hdlc, struct phys_bridge_register_struct *callbacks)
+{
+ if (my_pair.hdlc1 == NULL )
+ {
+ /* register first device*/
+ my_pair.cb1 = (struct phys_bridge_register_struct *) kmalloc(sizeof(struct phys_bridge_register_struct), GFP_KERNEL);
+ my_pair.hdlc1 = hdlc;
+ my_pair.cb1->tx = callbacks->tx;
+ printk(KERN_ERR "First device %s registered.", hdlc_to_dev(hdlc)->name);
+ } else
+ if (my_pair.hdlc2 == NULL )
+ {
+ /* register second device*/
+ my_pair.cb2 = (struct phys_bridge_register_struct *) kmalloc(sizeof(struct phys_bridge_register_struct), GFP_KERNEL);
+ my_pair.hdlc2 = hdlc;
+ my_pair.cb2->tx = callbacks->tx;
+ printk(KERN_ERR "Second device %s registered.", hdlc_to_dev(hdlc)->name);
+ } else {
+ /* not more devices to register*/
+ printk(KERN_ERR "Not possible to register device %s.", hdlc_to_dev(hdlc)->name);
+ return PHYS_NOK;
+ }
+
+ return PHYS_OK;
+}
+
+int phys_bridge_unregister(hdlc_device *hdlc)
+{ + if (my_pair.hdlc1 == hdlc )
+ {
+ /* unregister first device*/
+ my_pair.hdlc1 = NULL;
+ kfree(my_pair.cb1);
+ my_pair.cb1 = NULL;
+ printk(KERN_ERR "First device %s unregistered.", hdlc_to_dev(hdlc)->name);
+ } else
+ if (my_pair.hdlc2 == hdlc )
+ {
+ /* unregister second device*/
+ my_pair.hdlc2 = NULL;
+ kfree(my_pair.cb2);
+ my_pair.cb2 = NULL;
+ printk(KERN_ERR "Second device %s unregistered.", hdlc_to_dev(hdlc)->name);
+ } else {
+ /* not more devices to register*/
+ printk(KERN_ERR "Not possible to unregister device %s.", hdlc_to_dev(hdlc)->name);
+ return PHYS_NOK;
+ }
+ return PHYS_OK;
+}
+
+int phys_bridge_data_received(hdlc_device *hdlc, struct sk_buff *skb)
+{
+ + printk(KERN_ERR "phys_bridge_data_received.");
+ + if (my_pair.hdlc1 == hdlc )
+ {
+ /* send the data to the side 2 */
+ my_pair.cb2->tx(skb, hdlc_to_dev(my_pair.hdlc2));
+ } else
+ if (my_pair.hdlc2 == hdlc )
+ {
+ /* send the data to the side 1 */
+ my_pair.cb1->tx(skb, hdlc_to_dev(my_pair.hdlc1));
+ } else {
+ /* not more devices to send */
+ return PHYS_NOK;
+ }
+ return PHYS_OK;
+}
+
+
+EXPORT_SYMBOL(phys_bridge_register);
+EXPORT_SYMBOL(phys_bridge_unregister);
+EXPORT_SYMBOL(phys_bridge_data_received);
+
+static char banner[] __initdata = KERN_ERR "HDLC Bridge for Linux version 0.0.1\n";
+
+static int __init bridge_init(void)
+{
+ printk(banner);
+ return 0;
+}
+
+MODULE_AUTHOR("Daniel Ferenci <dafe@xxxxxxxx>");
+MODULE_DESCRIPTION("HDLC bridge");
+MODULE_LICENSE("GPL");
+
+module_init(bridge_init);
--- linux-2.4.25/drivers/net/wan/hdlc_bridge.c Thu Jan 1 01:00:00 1970
+++ linux-2.4.25.intel/drivers/net/wan/hdlc_bridge.c Sat Nov 13 08:26:02 2004
@@ -0,0 +1,136 @@
+/*
+ * HDLC bridge support
+ *
+ * History:
+ * 12.11.2004 Daniel Ferenci creation
+ *
+ *
+ * This program is free software; you can redistribute it and/or modify it
+ * under the terms of version 2 of the GNU General Public License
+ * as published by the Free Software Foundation.
+ */
+
+#include <linux/module.h>
+#include <linux/kernel.h>
+#include <linux/slab.h>
+#include <linux/poll.h>
+#include <linux/errno.h>
+#include <linux/if_arp.h>
+#include <linux/init.h>
+#include <linux/skbuff.h>
+#include <linux/pkt_sched.h>
+#include <linux/inetdevice.h>
+#include <linux/lapb.h>
+#include <linux/rtnetlink.h>
+#include <linux/hdlc.h>
+#include <net/hdlcbridge.h>
+
+/*static int bridge_xmit(struct sk_buff *skb, struct net_device *dev);
+static int bridge_open(hdlc_device *hdlc);
+static void bridge_close(hdlc_device *hdlc);
+static void bridge_rx(struct sk_buff *skb);*/
+
+/* These functions are callbacks called by PHYSICAL BRIDGE layer */
+static int bridge_xmit(struct sk_buff *skb, struct net_device *dev)
+{
+ hdlc_device *hdlc = dev_to_hdlc(dev);
+ + printk(KERN_ERR "bridge_xmit device: %s", dev->name);
+ hdlc->xmit(skb, dev);
+ /*dev_kfree_skb(skb);*/
+ return 0;
+}
+
+
+
+static int bridge_open(hdlc_device *hdlc)
+{
+ struct phys_bridge_register_struct cb;
+ int result;
+ + printk(KERN_ERR "bridge_open device: %s", hdlc_to_dev(hdlc)->name);
+ cb.tx = bridge_xmit;
+ + result = phys_bridge_register(hdlc, &cb);
+ if (result != PHYS_NOK)
+ return result;
+ return 0;
+}
+
+
+
+static void bridge_close(hdlc_device *hdlc)
+{
+ printk(KERN_ERR "bridge_close device: %s", hdlc_to_dev(hdlc)->name);
+ phys_bridge_unregister(hdlc);
+}
+
+
+
+static void bridge_rx(struct sk_buff *skb)
+{
+ hdlc_device *hdlc = dev_to_hdlc(skb->dev);
+ + printk(KERN_ERR "bridge_rx device: %s", skb->dev->name);
+
+ if (phys_bridge_data_received(hdlc, skb) == PHYS_OK)
+ return;
+ hdlc->stats.rx_errors++;
+ dev_kfree_skb_any(skb);
+}
+
+
+
+int hdlc_bridge_ioctl(hdlc_device *hdlc, struct ifreq *ifr)
+{ + raw_hdlc_proto new_settings;
+ raw_hdlc_proto *raw_s = ifr->ifr_settings.ifs_ifsu.raw_hdlc;
+ const size_t size = sizeof(raw_hdlc_proto);
+ struct net_device *dev = hdlc_to_dev(hdlc);
+ int result;
+ + printk(KERN_DEBUG "hdlc_bridge_ioctl device: %s", hdlc_to_dev(hdlc)->name);
+ + switch (ifr->ifr_settings.type) {
+ case IF_GET_PROTO:
+ ifr->ifr_settings.type = IF_PROTO_X25;
+ return 0; /* return protocol only, no settable parameters */
+
+ case IF_PROTO_HDLC_BRIDGE:
+ if(!capable(CAP_NET_ADMIN))
+ return -EPERM;
+
+ if(dev->flags & IFF_UP)
+ return -EBUSY;
+ + if (copy_from_user(&new_settings, raw_s, size))
+ return -EFAULT;
+ + if (new_settings.encoding == ENCODING_DEFAULT)
+ new_settings.encoding = ENCODING_NRZ;
+
+ if (new_settings.parity == PARITY_DEFAULT)
+ new_settings.parity = PARITY_CRC16_PR1_CCITT;
+ + result = hdlc->attach(hdlc, new_settings.encoding,
+ new_settings.parity);
+ if (result)
+ return result;
+ + hdlc_proto_detach(hdlc);
+
+ hdlc->open = bridge_open;
+ hdlc->stop = bridge_close;
+ hdlc->netif_rx = bridge_rx;
+ hdlc->type_trans = NULL;
+ hdlc->proto = IF_PROTO_HDLC;
+ dev->hard_start_xmit = bridge_xmit;
+ dev->hard_header = NULL;
+ dev->type = ARPHRD_RAWHDLC;
+ dev->addr_len = 0;
+ + return 0;
+ }
+
+ return -EINVAL;
+}
--- linux-2.4.25/drivers/net/wan/hdlc_generic.c Wed Feb 18 14:36:31 2004
+++ linux-2.4.25.intel/drivers/net/wan/hdlc_generic.c Sat Oct 30 08:50:17 2004
@@ -107,6 +107,7 @@
case IF_PROTO_CISCO:
case IF_PROTO_FR:
case IF_PROTO_X25:
+ case IF_PROTO_HDLC_BRIDGE:
proto = ifr->ifr_settings.type;
break;

@@ -121,6 +122,8 @@
case IF_PROTO_CISCO: return hdlc_cisco_ioctl(hdlc, ifr);
case IF_PROTO_FR: return hdlc_fr_ioctl(hdlc, ifr);
case IF_PROTO_X25: return hdlc_x25_ioctl(hdlc, ifr);
+ case IF_PROTO_LAPB: return hdlc_x25_ioctl(hdlc, ifr);
+ case IF_PROTO_HDLC_BRIDGE: return hdlc_bridge_ioctl(hdlc, ifr);
default: return -EINVAL;
}
}
--- linux-2.4.25/include/linux/if.h Fri Nov 28 19:26:21 2003
+++ linux-2.4.25.intel/include/linux/if.h Sat Oct 2 08:09:35 2004
@@ -23,7 +23,9 @@
#include <linux/socket.h> /* for "struct sockaddr" et al */

#define IFNAMSIZ 16
+
#include <linux/hdlc/ioctl.h>
+#include <linux/lapb/ioctl.h>

/* Standard interface flags (netdevice->flags). */
#define IFF_UP 0x1 /* interface is up */
@@ -54,6 +56,7 @@

#define IF_GET_IFACE 0x0001 /* for querying only */
#define IF_GET_PROTO 0x0002
+#define IF_GET_ST3 0x0003

/* For definitions see hdlc.h */
#define IF_IFACE_V35 0x1000 /* V.35 serial interface */
@@ -76,6 +79,9 @@
#define IF_PROTO_FR_DEL_ETH_PVC 0x2009 /* Delete FR Ethernet-bridged PVC */
#define IF_PROTO_FR_PVC 0x200A /* for reading PVC status */
#define IF_PROTO_FR_ETH_PVC 0x200B
+#define IF_PROTO_HDLC_BRIDGE 0x200C /* hdlc bridge stack */
+#define IF_PROTO_HDLC_SEND 0x200D /* hdlc bridge stack */
+#define IF_PROTO_LAPB 0x200E /* lapb stack */


/*
@@ -110,10 +116,13 @@
fr_proto *fr;
fr_proto_pvc *fr_pvc;
fr_proto_pvc_info *fr_pvc_info;
-
+ struct lapb_parms_struct *lapb_stuff;
/* interface settings */
sync_serial_settings *sync;
te1_settings *te1;
+ st3_status *st3;
+ + buffer *data;
} ifs_ifsu;
};

--- linux-2.4.25/drivers/net/wan/Makefile Sat Nov 13 09:14:24 2004
+++ linux-2.4.25.intel/drivers/net/wan/Makefile Sat Oct 2 08:04:28 2004
@@ -82,6 +82,9 @@
obj-$(CONFIG_N2) += n2.o
obj-$(CONFIG_C101) += c101.o
obj-$(CONFIG_PCI200SYN) += pci200syn.o
+obj-$(CONFIG_TAHOE9XX) += tahoe9xx.o
+obj-$(CONFIG_X25TAP) += x25tap.o
+

include $(TOPDIR)/Rules.make



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/