Re: [PATCH net-next V2 2/5] flow_dissector: Get vlan priority in addition to vlan id

2016-08-18 Thread Jiri Pirko
Wed, Aug 17, 2016 at 12:36:11PM CEST, had...@mellanox.com wrote: >Add vlan priority check to the flow dissector by adding new flow >dissector struct, flow_dissector_key_vlan which includes vlan tag >fields. > >vlan_id and flow_label fields were under the same struct >(flow_dissector_key_tags). It w

[PATCH net-next V2 2/5] flow_dissector: Get vlan priority in addition to vlan id

2016-08-17 Thread Hadar Hen Zion
Add vlan priority check to the flow dissector by adding new flow dissector struct, flow_dissector_key_vlan which includes vlan tag fields. vlan_id and flow_label fields were under the same struct (flow_dissector_key_tags). It was a convenient setting since struct flow_dissector_key_tags is used by

Re: [PATCH net-next 2/4] flow_dissector: Get vlan priority in addition to vlan id

2016-08-11 Thread Jiri Pirko
Thu, Aug 11, 2016 at 05:23:00PM CEST, had...@dev.mellanox.co.il wrote: >On Thu, Aug 11, 2016 at 12:58 AM, kbuild test robot wrote: >> Hi Hadar, >> >> [auto build test ERROR on net-next/master] >> >> url: >> https://github.com/0day-ci/linux/commits/Hadar-Hen-Zion/flow_dissector-Get-vlan-info-fr

Re: [PATCH net-next 2/4] flow_dissector: Get vlan priority in addition to vlan id

2016-08-11 Thread Hadar Hen Zion
On Thu, Aug 11, 2016 at 12:58 AM, kbuild test robot wrote: > Hi Hadar, > > [auto build test ERROR on net-next/master] > > url: > https://github.com/0day-ci/linux/commits/Hadar-Hen-Zion/flow_dissector-Get-vlan-info-from-skb-vlan_tci-instead-of-skb-data/20160811-042500 > config: cris-etrax-100lx

Re: [PATCH net-next 2/4] flow_dissector: Get vlan priority in addition to vlan id

2016-08-10 Thread kbuild test robot
Hi Hadar, [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Hadar-Hen-Zion/flow_dissector-Get-vlan-info-from-skb-vlan_tci-instead-of-skb-data/20160811-042500 config: cris-etrax-100lx_v2_defconfig (attached as .config) compiler: cris-linux-gcc (GCC) 4.6.3

Re: [PATCH net-next 2/4] flow_dissector: Get vlan priority in addition to vlan id

2016-08-10 Thread kbuild test robot
Hi Hadar, [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Hadar-Hen-Zion/flow_dissector-Get-vlan-info-from-skb-vlan_tci-instead-of-skb-data/20160811-042500 config: m68k-sun3_defconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 4.9.0 reproduce

[PATCH net-next 2/4] flow_dissector: Get vlan priority in addition to vlan id

2016-08-10 Thread Hadar Hen Zion
Add vlan priority check to the flow dissector by adding new flow dissector struct, flow_dissector_key_vlan which includes vlan tag fields. vlan_id and flow_label fields were under the same struct (flow_dissector_key_tags). It was a convenient setting since struct flow_dissector_key_tags is used by

flow_dissector: Get vlan priority in addition to vlan id

2016-08-07 Thread Hadar Hen Zion
Hi Tom and Jiri, I would like to add vlan priority to __skb_flow_dissect. In the current vlan tag implementation there isn't any room left to vlan priority next to the vlan id. struct flow_dissector_key_tags { u32 vlan_id:12, flow_label:20