]> git.proxmox.com Git - ovs.git/commit
conntrack: Fix alg expectation cleanup.
authorDarrell Ball <dlu998@gmail.com>
Tue, 9 Jan 2018 23:44:54 +0000 (15:44 -0800)
committerBen Pfaff <blp@ovn.org>
Wed, 10 Jan 2018 17:17:47 +0000 (09:17 -0800)
commit4417ca3d95120d6585b6bcd2ff89db95988d3d8c
treed59e3f1a6c969bee7be455357efb93e3e8c72490
parentb99654b0ceeaa3a33059675ff218c2047eba0b91
conntrack: Fix alg expectation cleanup.

Presently, alg expectations are removed by being time expired.
This was intended to happen before the control connections and
was intended to minimize the extra work involved for tracking and
removing the expectations.  This is not the best option since it
should be possible to remove expectations when a control connection
is removed and a new api is in the works to do this. Also, conceptually
an expectation should not exist without a control connection context
and it can be argued that this should be a strict requirement.

The approach is changed to remove the expectations when the control
connections are removed.  The previous code to expire the expectations
is removed at the same time.

Fixes: bd5e81a0e ("Userspace Datapath: Add ALG infra and FTP.")
Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2017-December/341683.html
Signed-off-by: Darrell Ball <dlu998@gmail.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
lib/conntrack-private.h
lib/conntrack.c
lib/conntrack.h