Comment 21 for bug 1572397

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/40945
Committed: http://github.com/Juniper/contrail-vrouter/commit/9cb7dbdb52e74b7ac6212e5c0f9bf82b087d9f86
Submitter: Zuul (<email address hidden>)
Branch: R4.0

commit 9cb7dbdb52e74b7ac6212e5c0f9bf82b087d9f86
Author: Divakar <email address hidden>
Date: Fri May 27 15:58:31 2016 +0530

Keeping the mirror metadata entry in Flow entry

Currently mirror metadata is stored in index table using flow index
as key. Before the Flow eviction, the addition and deletion of this meta
data to Index table is always from Agent and hence no parallel
manipulation of Index table. After eviction, the metadata deletion from
index table can happen in parallel and hence leading to memory
corruption/leak in index table manipulation.

The right fix is making the index table SMP ready. As a stop gap fix,
the mirror metada data pointer is stored in the flow entry itself
directly.

Conflicts:
 include/vr_flow.h

closes-bug: #1572397, #1744369
Change-Id: I216d8b6c55025a6053b808fd2807338265b307e0
(cherry picked from commit a437faa4bb2b2c52ffb06c5e3635766d89e4ef60)