Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-08 Thread Justin Pettit
> On Aug 7, 2018, at 10:19 PM, Gregory Rose wrote: > > > On 8/7/2018 8:31 PM, Justin Pettit wrote: >> Thanks, Greg. I actually have this queued up with another patch that will >> disable meters entirely on broken kernels. I plan to send that out tomorrow. >> >> --Justin > > Oops - perhaps

Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-08 Thread Justin Pettit
That's fine. I had just been holding off because I thought you'd wanted the probe to disable meters on kernels with a broken implementation. Regardless, I did get that patch out earlier today: https://mail.openvswitch.org/pipermail/ovs-dev/2018-August/350885.html Thanks, --Justin >

Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-08 Thread Ben Pfaff
Justin, I already applied this as a straightforward backport. Hope it doesn't disrupt your work. On Tue, Aug 07, 2018 at 08:31:35PM -0700, Justin Pettit wrote: > Thanks, Greg. I actually have this queued up with another patch that will > disable meters entirely on broken kernels. I plan to send

Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-07 Thread Gregory Rose
On 8/7/2018 8:31 PM, Justin Pettit wrote: Thanks, Greg. I actually have this queued up with another patch that will disable meters entirely on broken kernels. I plan to send that out tomorrow. --Justin Oops - perhaps I was a bit hasty.  I was just trying to get caught up with upstream. Ap

Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-07 Thread Justin Pettit
Thanks, Greg. I actually have this queued up with another patch that will disable meters entirely on broken kernels. I plan to send that out tomorrow. --Justin > On Aug 7, 2018, at 4:45 PM, Greg Rose wrote: > > From: Justin Pettit > > Upstream commit: >From: Justin Pettit >Date: S

Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-07 Thread Gregory Rose
On 8/7/2018 4:55 PM, Ben Pfaff wrote: On Tue, Aug 07, 2018 at 04:45:26PM -0700, Greg Rose wrote: From: Justin Pettit Upstream commit: From: Justin Pettit Date: Sat, 28 Jul 2018 15:26:01 -0700 Subject: [PATCH] openvswitch: meter: Fix setting meter id for new entries The me

Re: [ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-07 Thread Ben Pfaff
On Tue, Aug 07, 2018 at 04:45:26PM -0700, Greg Rose wrote: > From: Justin Pettit > > Upstream commit: > From: Justin Pettit > Date: Sat, 28 Jul 2018 15:26:01 -0700 > Subject: [PATCH] openvswitch: meter: Fix setting meter id for new entries > > The meter code would create an entr

[ovs-dev] [PATCH] datapath: meter: Fix setting meter id for new entries

2018-08-07 Thread Greg Rose
From: Justin Pettit Upstream commit: From: Justin Pettit Date: Sat, 28 Jul 2018 15:26:01 -0700 Subject: [PATCH] openvswitch: meter: Fix setting meter id for new entries The meter code would create an entry for each new meter. However, it would not set the meter id in the ne