Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-04 Thread Jesse Gross
On Fri, Oct 4, 2013 at 6:30 AM, Kyle Mestery (kmestery) wrote: > On Oct 3, 2013, at 2:31 PM, Jesse Gross wrote: >> On Thu, Oct 3, 2013 at 9:46 AM, Kyle Mestery (kmestery) >> wrote: >>> So, we realize the need to add the NSH code upstream into the kernel. >>> But in parallel to this work, we're w

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-04 Thread Kyle Mestery (kmestery)
On Oct 3, 2013, at 2:31 PM, Jesse Gross wrote: > On Thu, Oct 3, 2013 at 9:46 AM, Kyle Mestery (kmestery) > wrote: >> So, we realize the need to add the NSH code upstream into the kernel. >> But in parallel to this work, we're wondering if it would be ok to add a new >> vport-nsh in the data path

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-03 Thread Jesse Gross
On Thu, Oct 3, 2013 at 9:46 AM, Kyle Mestery (kmestery) wrote: > So, we realize the need to add the NSH code upstream into the kernel. > But in parallel to this work, we're wondering if it would be ok to add a new > vport-nsh in the data path code. This would allow for stacking NSH headers > on wh

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-03 Thread Jesse Gross
On Tue, Oct 1, 2013 at 2:44 PM, Jarno Rajahalme wrote: > On Sep 20, 2013, at 1:04 AM, pritesh wrote: >> diff --git a/datapath/linux/compat/include/net/vxlan.h >> b/datapath/linux/compat/include/net/vxlan.h >> index 3ac816b..1c15dfb 100644 >> --- a/datapath/linux/compat/include/net/vxlan.h >> +++

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-03 Thread Kyle Mestery (kmestery)
On Oct 1, 2013, at 4:44 PM, Jarno Rajahalme wrote: > > On Sep 20, 2013, at 1:04 AM, pritesh wrote: > >> This patch adds support for Network Service Headers (nsh) over VXLAN >> as mentioned in [1]. Here changes are made to datapath to add nsh >> headers whenever a vxlan port with destination po

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Pritesh Kothari (pritkoth)
>> vxlan.c:118: if (vxh->vx_flags != htonl(VXLAN_FLAGS) || (vxh->vx_vni & >> htonl(0xff))) { >> >> or if (unlikely(nsh->b.svc_path & htonl(NSH_M_NSI))) { // NSH_M_NSI ==> >> 0x00ff which is same as 0xff above? >> am i missing something here? >> > > I did not check if the NSH spec tells to

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Jarno Rajahalme
> On Oct 1, 2013, at 6:36 PM, "Pritesh Kothari (pritkoth)" > wrote: > >> Also, the usual treatment of reserved bits is ignore on reception, rather >> than >> dropping. > > hmm. i didn't quite get what you are saying here, vxlan_vni and svc_path both > use upper 24bits > and their detection s

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Pritesh Kothari (pritkoth)
btw, first and foremost, thanks for the review, i greatly appreciate it :) >> diff --git a/datapath/flow.c b/datapath/flow.c >> index 29122af..4f47a48 100644 >> --- a/datapath/flow.c >> +++ b/datapath/flow.c >> @@ -1235,6 +1235,7 @@ int ovs_ipv4_tun_from_nlattr(const struct nlattr *attr, >> i

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Jarno Rajahalme
On Sep 20, 2013, at 1:04 AM, pritesh wrote: > This patch adds support for Network Service Headers (nsh) over VXLAN > as mentioned in [1]. Here changes are made to datapath to add nsh > headers whenever a vxlan port with destination port as 9030 is created. > IANA port allocation for nsh over vxl

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Pritesh Kothari (pritkoth)
On Oct 1, 2013, at 11:03 AM, Jarno Rajahalme wrote: > > On Oct 1, 2013, at 10:07 AM, Pritesh Kothari (pritkoth) > wrote: > >> >> On Sep 30, 2013, at 5:46 PM, Jesse Gross wrote: >> >>> On Fri, Sep 20, 2013 at 1:04 AM, pritesh wrote: This patch adds support for Network Service Headers (

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Pritesh Kothari (pritkoth)
On Oct 1, 2013, at 12:08 PM, Jesse Gross wrote: > On Tue, Oct 1, 2013 at 10:07 AM, Pritesh Kothari (pritkoth) > wrote: >> On Sep 30, 2013, at 5:46 PM, Jesse Gross wrote: >>> One other thing - would you mind using your full name in the signed-off-by >>> line? >> >> i generally prefer just my fi

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Jesse Gross
On Tue, Oct 1, 2013 at 10:07 AM, Pritesh Kothari (pritkoth) wrote: > On Sep 30, 2013, at 5:46 PM, Jesse Gross wrote: >> One other thing - would you mind using your full name in the signed-off-by >> line? > > i generally prefer just my first name, but if its needed i can use my full > name here.

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Jarno Rajahalme
On Oct 1, 2013, at 10:07 AM, Pritesh Kothari (pritkoth) wrote: > > On Sep 30, 2013, at 5:46 PM, Jesse Gross wrote: > >> On Fri, Sep 20, 2013 at 1:04 AM, pritesh wrote: >>> This patch adds support for Network Service Headers (nsh) over VXLAN >>> as mentioned in [1]. Here changes are made to d

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-10-01 Thread Pritesh Kothari (pritkoth)
On Sep 30, 2013, at 5:46 PM, Jesse Gross wrote: > On Fri, Sep 20, 2013 at 1:04 AM, pritesh wrote: >> This patch adds support for Network Service Headers (nsh) over VXLAN >> as mentioned in [1]. Here changes are made to datapath to add nsh >> headers whenever a vxlan port with destination port as

Re: [ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-09-30 Thread Jesse Gross
On Fri, Sep 20, 2013 at 1:04 AM, pritesh wrote: > This patch adds support for Network Service Headers (nsh) over VXLAN > as mentioned in [1]. Here changes are made to datapath to add nsh > headers whenever a vxlan port with destination port as 9030 is created. > IANA port allocation for nsh over v

[ovs-dev] [PATCH 1/8] nsh: datapath support for network service headers

2013-09-20 Thread pritesh
This patch adds support for Network Service Headers (nsh) over VXLAN as mentioned in [1]. Here changes are made to datapath to add nsh headers whenever a vxlan port with destination port as 9030 is created. IANA port allocation for nsh over vxlan is yet to be done. [1] http://tools.ietf.org/html/d