[ovs-dev] [ovs-discuss] [ACLv2 04/19] ofproto: Add ofproto_has_matching_flow function.

Ben Pfaff blp at nicira.com
Fri Sep 4 21:26:09 UTC 2009


Jesse Gross <jesse at nicira.com> writes:

> Ben Pfaff wrote:
>> Jesse Gross <jesse at nicira.com> writes:
>>
>>   
>>> Checks whether one or more currently installed OpenFlow flows
>>> match the given flow and wildcards.
>>>     
>>
>> I don't see anything wrong here, but man! it is expensive to
>> iterate through all the flows to see whether there is a match.  I
>> hope that this is not going to be done too often.
>>   
>
> I realized that this wasn't strictly necessary, so I eliminated this
> commit.  I now record events that could change the result and use that
> cached value.

Thanks, that sounds like a better approach.




More information about the dev mailing list