On 3 August 2017 at 14:35, Ben Pfaff <b...@ovn.org> wrote:
> On Mon, Jul 31, 2017 at 04:54:22PM -0700, Joe Stringer wrote:
>> This call is operating on messages generated by the datapath. If a
>> datapath implementation sends improperly formatted netlink attributes,
>> then it's possible for a revalidator thread to end up trapped in an
>> infinite loop iterating across these attributes. Rather than using the
>> UNSAFE variation of this iterator, use the regular version.
>>
>> Fixes: 994fcc5a15d3 ("upcall: Check for recirc_id in
>> ukey_create_from_dpif_flow()")
>> Signed-off-by: Joe Stringer <j...@ovn.org>
>
> I wonder whether there's enough benefit in the _UNSAFE variants to keep
> them around.
Hmm. It's used in odp-execute to iterate generated actions, that's the
primary place that could have some sensitivity to this as it's used in
the userdp fastpath. If we can't measure an impact there with a decent
average actions list length, then perhaps it could be removed
altogether.
> Acked-by: Ben Pfaff <b...@ovn.org>
Thanks for the reviews, applied to master and branches 2.5-2.8.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
