Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-12 Thread Daniele Di Proietto
st, 2016 02:59 > *To:* Jan Scheurich > *Cc:* dev@openvswitch.org; antonio.fische...@intel.com > *Subject:* Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with > sorted subtables > > > > Hi Jan, > > thanks for the patch! > > I tried benchmarking with flow

Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-11 Thread Jan Scheurich
-netdev: dpcls per in_port with sorted subtables Hi Jan, thanks for the patch! I tried benchmarking with flow tables from utilities/ovs-pipegen.py and the results look very good. [Jan] Do you have any numbers? Setting up pipelines is one thing. Is there are standard test setup for measuring DPDK

Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-10 Thread Daniele Di Proietto
Hi Jan, thanks for the patch! I tried benchmarking with flow tables from utilities/ovs-pipegen.py and the results look very good. I had to manually edit the patch to apply: some whitespaces are trimmed, some lines are truncated and the form feeds('^L') are two separate characters. Would you min

Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-10 Thread Jan Scheurich (POP)
Hi Antonio, Thanks for the review. In the present v4 patch the cpvector-related changes are already removed. They are taken care of in Jarno's revert patch. BR, Jan > Am 10.08.2016 um 16:51 schrieb Fischetti, Antonio > : > > Hi Jan, > I don't know if you're going to split this in different patc

Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-10 Thread Fischetti, Antonio
Hi Jan, I don't know if you're going to split this in different patches to separate cpvector stuff, in any case all the content looks good to me. Thanks. Acked-by: Antonio Fischetti > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Jan > Scheurich > Sen