[ovs-dev] [PATCH v2] datapath-windows: Prevent ct-counters from getting redundantly incremented
Alin Gabriel Serdean
aserdean at ovn.org
Mon Apr 30 20:47:43 UTC 2018
> On 30 Apr 2018, at 23:36, Ben Pfaff <blp at ovn.org> wrote:
>
> On Mon, Apr 30, 2018 at 11:26:52PM +0300, Alin Gabriel Serdean wrote:
>>
>>> On 28 Apr 2018, at 21:26, Ben Pfaff <blp at ovn.org> wrote:
>>>
>>> On Fri, Apr 27, 2018 at 08:03:43PM +0300, Alin Gabriel Serdean wrote:
>>>>
>>>>> On 27 Apr 2018, at 20:00, Anand Kumar <kumaranand at vmware.com> wrote:
>>>>>
>>>>> The conntrack-counters ought to be incremented only if it's a new lookup
>>>>> or if it's recirculated through a different zone for the first time.
>>>>>
>>>>> Signed-off-by: Anand Kumar <kumaranand at vmware.com>
>>>>> ---
>>>>> datapath-windows/ovsext/Conntrack.c | 7 ++++---
>>>>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>>>>
>>>>> diff --git a/datapath-windows/ovsext/Conntrack.c b/datapath-windows/ovsext/Conntrack.c
>>>>> index 678bedb..add1491 100644
>>>>> --- a/datapath-windows/ovsext/Conntrack.c
>>>>> +++ b/datapath-windows/ovsext/Conntrack.c
>>>>> @@ -886,10 +886,11 @@ OvsCtExecute_(OvsForwardingContext *fwdCtx,
>>>>> return NDIS_STATUS_RESOURCES;
>>>>> }
>>>>>
>>>>> - /* Increment the counters soon after the lookup, since we set ct.state
>>>>> - * to OVS_CS_F_TRACKED after processing the ct entry.
>>>>> + /* Increment stats for the entry if it wasn't tracked previously or
>>>>> + * if they are on different zones
>>>>> */
>>>>> - if (entry && (!(key->ct.state & OVS_CS_F_TRACKED))) {
>>>>> + if (entry && (entry->key.zone != key->ct.zone ||
>>>>> + (!(key->ct.state & OVS_CS_F_TRACKED)))) {
>>>>> OvsCtIncrementCounters(entry, ctx.reply, curNbl);
>>>>> }
>>>>>
>>>>> --
>>>>> 2.9.3.windows.1
>>>>>
>>>>> _______________________________________________
>>>>> dev mailing list
>>>>> dev at openvswitch.org
>>>>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev <https://mail.openvswitch.org/mailman/listinfo/ovs-dev>
>>>>
>>>> Acked-by: Alin Gabriel Serdean <aserdean at ovn.org <mailto:aserdean at ovn.org>>
>>>
>>> Do you want to apply this to the tree?
>>
>> I will apply them thanks for asking Ben. I am on vacation this week so response will be increased :).
>
> I'll be in Copenhagen this week (leaving for the airport in about 90
> minutes actually), so me too.
I saw that you are going to kubecon! I hope you enjoy your time over there :).
I hope you and Alin Balutoiu can come and say hi!
- Alin.
More information about the dev
mailing list