> On Aug 17, 2018, at 1:08 PM, Ben Pfaff <[email protected]> wrote:
> 
> On Fri, Aug 17, 2018 at 01:05:48PM -0700, Justin Pettit wrote:
>> Commit 92d0d515d ("dpif-netlink: Probe for broken Linux meter
>> implementations.") introduced a deadlock on the 'once' structure
>> declared in probe_broken_meters() with the following callstack:
>> 
>>        probe_broken_meters()
>>        probe_broken_meters__()
>>        dpif_netlink_meter_set()
>>        probe_broken_meters()
>> 
>> This commit introduce a modified version of dpif_netlink_meter_set()
>> that sets a meter without calling the probe.
>> 
>> Reported-by: Numan Siddique <[email protected]>
>> Signed-off-by: Justin Pettit <[email protected]>
> 
> I didn't test this; I assume you did.
> 
> Acked-by: Ben Pfaff <[email protected]>

Thanks for the quick review!  I pushed it to master and branch-2.10.

--Justin


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to