Re: [PATCH net-next v2 2/3] mpls: move mpls_hdr to a common location

2016-09-30 Thread pravin shelar
On Fri, Sep 30, 2016 at 10:51 AM, Jiri Benc wrote: > On Fri, 30 Sep 2016 10:43:42 -0700, pravin shelar wrote: >> There is already struct mpls_label defined in uapi mpls header file. >> Is there reason for not using it here? > > I think this should be unified as well. But it's for a different patch

Re: [PATCH net-next v2 2/3] mpls: move mpls_hdr to a common location

2016-09-30 Thread Jiri Benc
On Fri, 30 Sep 2016 10:43:42 -0700, pravin shelar wrote: > There is already struct mpls_label defined in uapi mpls header file. > Is there reason for not using it here? I think this should be unified as well. But it's for a different patch not related to this patchset. This patch just moves the ex

Re: [PATCH net-next v2 2/3] mpls: move mpls_hdr to a common location

2016-09-30 Thread pravin shelar
On Fri, Sep 30, 2016 at 10:08 AM, Jiri Benc wrote: > This will be also used by openvswitch. > > Signed-off-by: Jiri Benc > --- > v2: new in v2 > --- > include/net/mpls.h | 9 + > net/mpls/internal.h | 10 +- > 2 files changed, 10 insertions(+), 9 deletions(-) > > diff --git a/i

Re: [PATCH net-next v2 2/3] mpls: move mpls_hdr to a common location

2016-09-30 Thread David Ahern
On 9/30/16 11:08 AM, Jiri Benc wrote: > This will be also used by openvswitch. > > Signed-off-by: Jiri Benc > --- > v2: new in v2 > --- > include/net/mpls.h | 9 + > net/mpls/internal.h | 10 +- > 2 files changed, 10 insertions(+), 9 deletions(-) > Acked-by: David Ahern

[PATCH net-next v2 2/3] mpls: move mpls_hdr to a common location

2016-09-30 Thread Jiri Benc
This will be also used by openvswitch. Signed-off-by: Jiri Benc --- v2: new in v2 --- include/net/mpls.h | 9 + net/mpls/internal.h | 10 +- 2 files changed, 10 insertions(+), 9 deletions(-) diff --git a/include/net/mpls.h b/include/net/mpls.h index 5b3b5addfb08..3ebbc0bb57ff