Comment 12 for bug 1572397

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

Reviewed: https://review.opencontrail.org/39801
Committed: http://github.com/Juniper/contrail-vrouter/commit/a437faa4bb2b2c52ffb06c5e3635766d89e4ef60
Submitter: Zuul (<email address hidden>)
Branch: R3.2

commit a437faa4bb2b2c52ffb06c5e3635766d89e4ef60
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