Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-04 Thread Jiri Pirko
Wed, Oct 04, 2017 at 09:07:17PM CEST, simon.hor...@netronome.com wrote: >On Wed, Oct 04, 2017 at 08:07:15PM +0200, Jiri Pirko wrote: >> Wed, Oct 04, 2017 at 05:52:54PM CEST, t...@herbertland.com wrote: >> >On Wed, Oct 4, 2017 at 1:15 AM, Jiri Pirko wrote: >> >> Wed, Oct 04, 2017

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-04 Thread Simon Horman
On Wed, Oct 04, 2017 at 08:07:15PM +0200, Jiri Pirko wrote: > Wed, Oct 04, 2017 at 05:52:54PM CEST, t...@herbertland.com wrote: > >On Wed, Oct 4, 2017 at 1:15 AM, Jiri Pirko wrote: > >> Wed, Oct 04, 2017 at 10:08:57AM CEST, simon.hor...@netronome.com wrote: > >>>On Tue, Oct 03,

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-04 Thread Jiri Pirko
Wed, Oct 04, 2017 at 05:52:54PM CEST, t...@herbertland.com wrote: >On Wed, Oct 4, 2017 at 1:15 AM, Jiri Pirko wrote: >> Wed, Oct 04, 2017 at 10:08:57AM CEST, simon.hor...@netronome.com wrote: >>>On Tue, Oct 03, 2017 at 11:17:46AM -0700, Tom Herbert wrote: On Tue, Oct 3,

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-04 Thread Tom Herbert
On Wed, Oct 4, 2017 at 1:15 AM, Jiri Pirko wrote: > Wed, Oct 04, 2017 at 10:08:57AM CEST, simon.hor...@netronome.com wrote: >>On Tue, Oct 03, 2017 at 11:17:46AM -0700, Tom Herbert wrote: >>> On Tue, Oct 3, 2017 at 2:40 AM, Simon Horman >>> wrote:

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-04 Thread Jiri Pirko
Wed, Oct 04, 2017 at 10:08:57AM CEST, simon.hor...@netronome.com wrote: >On Tue, Oct 03, 2017 at 11:17:46AM -0700, Tom Herbert wrote: >> On Tue, Oct 3, 2017 at 2:40 AM, Simon Horman >> wrote: >> > On Mon, Oct 02, 2017 at 01:37:55PM -0700, Tom Herbert wrote: >> >> On

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-04 Thread Simon Horman
On Tue, Oct 03, 2017 at 11:17:46AM -0700, Tom Herbert wrote: > On Tue, Oct 3, 2017 at 2:40 AM, Simon Horman > wrote: > > On Mon, Oct 02, 2017 at 01:37:55PM -0700, Tom Herbert wrote: > >> On Mon, Oct 2, 2017 at 1:41 AM, Simon Horman > >>

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-03 Thread Tom Herbert
On Tue, Oct 3, 2017 at 2:40 AM, Simon Horman wrote: > On Mon, Oct 02, 2017 at 01:37:55PM -0700, Tom Herbert wrote: >> On Mon, Oct 2, 2017 at 1:41 AM, Simon Horman >> wrote: >> > Move dissection of tunnel info from the flower classifier to

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-03 Thread Simon Horman
On Mon, Oct 02, 2017 at 01:37:55PM -0700, Tom Herbert wrote: > On Mon, Oct 2, 2017 at 1:41 AM, Simon Horman > wrote: > > Move dissection of tunnel info from the flower classifier to the flow > > dissector where all other dissection occurs. This should not have any >

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-02 Thread Tom Herbert
On Mon, Oct 2, 2017 at 1:41 AM, Simon Horman wrote: > Move dissection of tunnel info from the flower classifier to the flow > dissector where all other dissection occurs. This should not have any > behavioural affect on other users of the flow dissector. > >

Re: [oss-drivers] Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-02 Thread Simon Horman
On Mon, Oct 02, 2017 at 12:36:33PM -0700, Tom Herbert wrote: > On Mon, Oct 2, 2017 at 1:41 AM, Simon Horman > wrote: > > Move dissection of tunnel info from the flower classifier to the flow > > dissector where all other dissection occurs. This should not have any >

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-02 Thread Tom Herbert
On Mon, Oct 2, 2017 at 1:41 AM, Simon Horman wrote: > Move dissection of tunnel info from the flower classifier to the flow > dissector where all other dissection occurs. This should not have any > behavioural affect on other users of the flow dissector. > >

Re: [PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-02 Thread Jiri Pirko
Mon, Oct 02, 2017 at 10:41:16AM CEST, simon.hor...@netronome.com wrote: >Move dissection of tunnel info from the flower classifier to the flow >dissector where all other dissection occurs. This should not have any >behavioural affect on other users of the flow dissector. > >Signed-off-by: Simon

[PATCH net-next 2/2] flow_dissector: dissect tunnel info

2017-10-02 Thread Simon Horman
Move dissection of tunnel info from the flower classifier to the flow dissector where all other dissection occurs. This should not have any behavioural affect on other users of the flow dissector. Signed-off-by: Simon Horman Reviewed-by: Jakub Kicinski