Re: [ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-28 Thread Ben Pfaff
On Fri, Jun 28, 2013 at 09:33:00AM -0700, Ben Pfaff wrote: > On Fri, Jun 28, 2013 at 04:28:40PM +, Rajahalme, Jarno (NSN - FI/Espoo) > wrote: > > > > On Jun 28, 2013, at 19:09 , ext Ben Pfaff wrote: > > > > > On Fri, Jun 28, 2013 at 07:17:09AM +, Rajahalme, Jarno (NSN - > > > FI/Espoo)

Re: [ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-28 Thread Rajahalme, Jarno (NSN - FI/Espoo)
On Jun 28, 2013, at 19:33 , ext Ben Pfaff wrote: > On Fri, Jun 28, 2013 at 04:28:40PM +, Rajahalme, Jarno (NSN - FI/Espoo) > wrote: >> >> On Jun 28, 2013, at 19:09 , ext Ben Pfaff wrote: >> >>> On Fri, Jun 28, 2013 at 07:17:09AM +, Rajahalme, Jarno (NSN - FI/Espoo) >>> wrote: >>

Re: [ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-28 Thread Ben Pfaff
On Fri, Jun 28, 2013 at 04:28:40PM +, Rajahalme, Jarno (NSN - FI/Espoo) wrote: > > On Jun 28, 2013, at 19:09 , ext Ben Pfaff wrote: > > > On Fri, Jun 28, 2013 at 07:17:09AM +, Rajahalme, Jarno (NSN - FI/Espoo) > > wrote: > >> > >> On Jun 28, 2013, at 1:19 , ext Ben Pfaff wrote: > >>> o

Re: [ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-28 Thread Rajahalme, Jarno (NSN - FI/Espoo)
On Jun 28, 2013, at 19:09 , ext Ben Pfaff wrote: > On Fri, Jun 28, 2013 at 07:17:09AM +, Rajahalme, Jarno (NSN - FI/Espoo) > wrote: >> >> On Jun 28, 2013, at 1:19 , ext Ben Pfaff wrote: >>> ofproto: Use another method to find meter_id. >> >> OK, with the rationale that actions lists are sh

Re: [ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-28 Thread Ben Pfaff
On Fri, Jun 28, 2013 at 07:17:09AM +, Rajahalme, Jarno (NSN - FI/Espoo) wrote: > > On Jun 28, 2013, at 1:19 , ext Ben Pfaff wrote: > > ofproto: Use another method to find meter_id. > > OK, with the rationale that actions lists are short on average, so > it does not matter scanning them twic

Re: [ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-28 Thread Rajahalme, Jarno (NSN - FI/Espoo)
On Jun 28, 2013, at 1:19 , ext Ben Pfaff wrote: > I spent some time looking at and playing with patch 4/5 "v2: ofproto: > Implement OpenFlow 1.3 meter table." and decided that there was a fair bit > that I wanted to tweak. This series represent what I came up with: > > IMPORTANT: In this series

[ovs-dev] [meters 0/9] Revised meters patch 4/5 proposal

2013-06-27 Thread Ben Pfaff
I spent some time looking at and playing with patch 4/5 "v2: ofproto: Implement OpenFlow 1.3 meter table." and decided that there was a fair bit that I wanted to tweak. This series represent what I came up with: IMPORTANT: In this series, I intend to squash all the patches after "Implement OpenFl