Re: [patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-03-06 Thread Jiri Pirko
Tue, Feb 21, 2017 at 07:50:53PM CET, t...@herbertland.com wrote: >On Tue, Feb 21, 2017 at 6:33 AM, Jiri Pirko wrote: >> From: Jiri Pirko >> >> Make the main flow_dissect function a bit smaller and move the ARP >> dissection into a separate function. Along with that, do the ARP header >> processin

Re: [patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-02-22 Thread Simon Horman
On Tue, Feb 21, 2017 at 01:32:35PM -0500, David Miller wrote: > From: Jiri Pirko > Date: Tue, 21 Feb 2017 15:33:18 +0100 > > > From: Jiri Pirko > > > > Make the main flow_dissect function a bit smaller and move the ARP > > dissection into a separate function. Along with that, do the ARP header

Re: [patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-02-22 Thread Simon Horman
On Tue, Feb 21, 2017 at 03:33:18PM +0100, Jiri Pirko wrote: > From: Jiri Pirko > > Make the main flow_dissect function a bit smaller and move the ARP > dissection into a separate function. Along with that, do the ARP header > processing only in case the flow dissection user requires it. > > Sign

Re: [patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-02-21 Thread Jiri Pirko
Tue, Feb 21, 2017 at 07:50:53PM CET, t...@herbertland.com wrote: >On Tue, Feb 21, 2017 at 6:33 AM, Jiri Pirko wrote: >> From: Jiri Pirko >> >> Make the main flow_dissect function a bit smaller and move the ARP >> dissection into a separate function. Along with that, do the ARP header >> processin

Re: [patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-02-21 Thread Tom Herbert
On Tue, Feb 21, 2017 at 6:33 AM, Jiri Pirko wrote: > From: Jiri Pirko > > Make the main flow_dissect function a bit smaller and move the ARP > dissection into a separate function. Along with that, do the ARP header > processing only in case the flow dissection user requires it. > Acked-by: Tom H

Re: [patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-02-21 Thread David Miller
From: Jiri Pirko Date: Tue, 21 Feb 2017 15:33:18 +0100 > From: Jiri Pirko > > Make the main flow_dissect function a bit smaller and move the ARP > dissection into a separate function. Along with that, do the ARP header > processing only in case the flow dissection user requires it. > > Signed-

[patch net-next RFC 1/2] flow_dissecror: Move ARP dissection into a separate function

2017-02-21 Thread Jiri Pirko
From: Jiri Pirko Make the main flow_dissect function a bit smaller and move the ARP dissection into a separate function. Along with that, do the ARP header processing only in case the flow dissection user requires it. Signed-off-by: Jiri Pirko --- net/core/flow_dissector.c | 111 ++