Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-11 Thread Guillaume Nault
On Mon, Jan 11, 2021 at 09:02:04AM -0700, David Ahern wrote: > On 1/11/21 8:44 AM, Guillaume Nault wrote: > > On Mon, Jan 11, 2021 at 08:30:32AM -0700, David Ahern wrote: > >> On 1/11/21 3:57 AM, Guillaume Nault wrote: > >>> Okay, but, in the end, should I repost this patch? > >> > >> I think your

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-11 Thread David Ahern
On 1/11/21 8:44 AM, Guillaume Nault wrote: > On Mon, Jan 11, 2021 at 08:30:32AM -0700, David Ahern wrote: >> On 1/11/21 3:57 AM, Guillaume Nault wrote: >>> Okay, but, in the end, should I repost this patch? >> >> I think your patches are covered, but you should check the repo to make >> sure. > >

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-11 Thread Guillaume Nault
On Mon, Jan 11, 2021 at 08:30:32AM -0700, David Ahern wrote: > On 1/11/21 3:57 AM, Guillaume Nault wrote: > > Okay, but, in the end, should I repost this patch? > > I think your patches are covered, but you should check the repo to make > sure. This patch ("tc: flower: fix json output with mpls l

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-11 Thread David Ahern
On 1/11/21 3:57 AM, Guillaume Nault wrote: > Okay, but, in the end, should I repost this patch? I think your patches are covered, but you should check the repo to make sure.

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-11 Thread Guillaume Nault
On Thu, Jan 07, 2021 at 10:39:03AM -0700, David Ahern wrote: > On 1/7/21 10:13 AM, Jakub Kicinski wrote: > > On Thu, 7 Jan 2021 17:48:56 +0100 Guillaume Nault wrote: > >> On Fri, Dec 18, 2020 at 11:25:32PM +0100, Guillaume Nault wrote: > >>> The json output of the TCA_FLOWER_KEY_MPLS_OPTS attribute

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-07 Thread David Ahern
On 1/7/21 10:13 AM, Jakub Kicinski wrote: > On Thu, 7 Jan 2021 17:48:56 +0100 Guillaume Nault wrote: >> On Fri, Dec 18, 2020 at 11:25:32PM +0100, Guillaume Nault wrote: >>> The json output of the TCA_FLOWER_KEY_MPLS_OPTS attribute was invalid. >>> >>> Example: >>> >>> $ tc filter add dev eth0 ing

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-07 Thread Jakub Kicinski
On Thu, 7 Jan 2021 17:48:56 +0100 Guillaume Nault wrote: > On Fri, Dec 18, 2020 at 11:25:32PM +0100, Guillaume Nault wrote: > > The json output of the TCA_FLOWER_KEY_MPLS_OPTS attribute was invalid. > > > > Example: > > > > $ tc filter add dev eth0 ingress protocol mpls_uc flower mpls \ > >

Re: [PATCH iproute2] tc: flower: fix json output with mpls lse

2021-01-07 Thread Guillaume Nault
On Fri, Dec 18, 2020 at 11:25:32PM +0100, Guillaume Nault wrote: > The json output of the TCA_FLOWER_KEY_MPLS_OPTS attribute was invalid. > > Example: > > $ tc filter add dev eth0 ingress protocol mpls_uc flower mpls \ > lse depth 1 label 100 \ >

[PATCH iproute2] tc: flower: fix json output with mpls lse

2020-12-18 Thread Guillaume Nault
The json output of the TCA_FLOWER_KEY_MPLS_OPTS attribute was invalid. Example: $ tc filter add dev eth0 ingress protocol mpls_uc flower mpls \ lse depth 1 label 100 \ lse depth 2 label 200 $ tc -json filter show dev eth0 ingress ...{"eth_t