如何从skb里得到vlan id?

happy_flying

假设有一个设备,有两个接口,都是trunk口,不打tag,也不剥tag,但是在bridge里想获得其vlan id.写了下面一个函数,不知道对不对,请各位帮忙看看.

unsigned int br_get_vlan_id(struct sk_buff *skb) {     struct vlan_ethhdr *veth;     unsigned short vlan_TCI;                    unsigned short vlanid = 0;    //if no config VLAN, default is 0     if (skb->protocol == htons(ETH_P_8021Q)) {         veth = (struct vlan_ethhdr *)(skb->data - VLAN_ETH_HLEN);         vlan_TCI = ntohs(veth->h_vlan_TCI);                     vlanid = (vlan_TCI & VLAN_VID_MASK);                 }         return vlanid; }

其实关键问题是我没有搞清楚在bridge module里,skb->data指向哪儿?二层头还是三层头?
请知道的帮个忙.

qps104

个人觉得是指向二层头.不敢肯定

qtdszws

eth_type_trans

skb->mac.raw=skb->data;//设置mac指针
//跳过硬件头dst mac6,src mac,protocol 6+6+2=14
skb_pull(skb,dev->hard_header_len);//data指向ip协议头
eth= skb->mac.ethernet;

指向三层头

happy_flying

非常感谢qtdszws ,又看了一遍code,确实如你所说,我上面写的不对,应该是:

unsigned short br_get_vlan_id(struct sk_buff *skb) {     struct vlan_hdr *vhdr;     unsigned short vlan_TCI;                    unsigned short vlanid = 0;    //if no config VLAN, default is 0         if (skb->protocol == ETH_P_8021Q) {    // or htons(ETH_P_8021Q) ??????         vhdr = (struct vlan_hdr *)(skb->data);            vlan_TCI = vhdr->h_vlan_TCI;    // or ntohs(vhdr->h_vlan_TCI) ??????                 vlanid = (vlan_TCI & VLAN_VID_MASK);                 }         return vlanid; }

呵呵,还有一个很弱的问题,如果在mips cpu上,由于是大字节序的,是不是就不需要字节序转换了?

qtdszws

需要ntohs(type)

happy_flying

再次感谢qtdszws.
刚弄清楚,这儿是应该加.虽然对于字节序为big-endian的mips cpu来讲( 此时ntoh(x)等价于(x) ),不加也可以.
但是为了可移植性,应该加,这样不管是big-endian还是little-endian,都ok.
PS :mips的字节序好像是可以随意配置的,我们用的是big-endian

呵呵,再借这个机会请教一个bridge module的问题.
当本机自己要往外发一个包时,如果目的设备是bridge,在dev_queue_xmit()里调用dev->hard_start_xmit(skb, dev)就相当于
调用br_dev_xmit(),接着会调用__br_dev_xmit(),在这个函数里,有
skb_pull(skb, ETH_HLEN);
接着依次调用
br_deliver()-->__br_deliver()-->__br_forward_finish()-->__dev_queue_push_xmit()-->dev_queue_xmit(),把包真正发送出去.
在__dev_queue_push_xmit()这个函数里,有
skb_push(skb, ETH_HLEN);
我的问题是,为什么要先pull,再push,看code好像没看出来这么做有什么用?

[ 本帖最后由 happy_flying 于 2008-3-7 16:09 编辑 ]

qtdszws

至于为何调用skb_pull(skb, ETH_HLEN);因为ebtables.c要求skb->data指向三层头.例如ebt_ip.c中
ih = skb_header_pointer(skb, 0, sizeof(_iph), &_iph);
而后面调用skb_push(skb, ETH_HLEN);是因为真的要从接口发送出去了

happy_flying

果然如此，我看得还是不仔细,佩服!
不过还有一个疑问，对于vlan包，pull 14 bytes后，指向的是下面struct里的h_vlan_TCI，并不是三层头呀，为什么不pull 18 bytes? 呵呵，问题比较多，还望见谅。

struct vlan_ethhdr {    unsigned char        h_dest[ETH_ALEN];           /* destination eth addr        */    unsigned char        h_source[ETH_ALEN];           /* source ether addr        */    unsigned short       h_vlan_proto;              /* Should always be 0x8100 */    unsigned short       h_vlan_TCI;                /* Encapsulates priority and VLAN ID */    unsigned short        h_vlan_encapsulated_proto; /* packet type ID field (or len) */ };

qtdszws

好像没有考虑这种情况

处理bridge在前

        if (handle_bridge(&skb, &pt_prev, &ret, orig_dev))
                goto out;

而对vlan的处理在这里
        type = skb->protocol;
        list_for_each_entry_rcu(ptype, &ptype_base[ntohs(type)&15], list) {
                if (ptype->type == type &&
                    (!ptype->dev || ptype->dev == skb->dev)) {
                        if (pt_prev)
                                ret = deliver_skb(skb, pt_prev, orig_dev);
                        pt_prev = ptype;
                }
        }