Big Switch: Consistency hash table lock causing deadlocks

[Kevin Benton]

The new consistency hash table seems to be causing deadlocks.

http://openstack-ci-gw.bigswitch.com/logs/refs-changes-13-92013-4/BSN_PLUGIN/logs/screen/screen-q-svc.log.gz

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/103874

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/103874
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=ee9fe2458f813b5d367bc7263c7a30a3af46aa2b
Submitter: Jenkins
Branch: master

commit ee9fe2458f813b5d367bc7263c7a30a3af46aa2b
Author: Kevin Benton <email address hidden>
Date: Tue Jul 1 06:03:19 2014 -0700

    BSN: Remove db lock and add missing contexts

    Adds context tracking decorators that were missing
    from router interface methods. Without them, new
    sessions were being created instead of using the
    existing context which was causing transaction
    issues.

    Modifies the servermanager to store context references
    as weakrefs so if multiple functions are called before
    the rest functions are called, the first one doesn't steal
    the only context reference with a pop() call.

    Removes a DB lock for update in the server manager that occured
    during rest calls that was triggering deadlocks due to the
    file lock synchronization for the rest calls.

    Closes-Bug: #1336251
    Change-Id: Iad3d61e2c23832b3ad760a999fbab7feaa13f805

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/109150

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/109150
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=f9379ef697b6a23588e674dd1b33ca2453adfad9
Submitter: Jenkins
Branch: stable/icehouse

commit f9379ef697b6a23588e674dd1b33ca2453adfad9
Author: Kevin Benton <email address hidden>
Date: Tue Jul 1 06:03:19 2014 -0700

    BSN: Remove db lock and add missing contexts

    Adds context tracking decorators that were missing
    from router interface methods. Without them, new
    sessions were being created instead of using the
    existing context which was causing transaction
    issues.

    Modifies the servermanager to store context references
    as weakrefs so if multiple functions are called before
    the rest functions are called, the first one doesn't steal
    the only context reference with a pop() call.

    Removes a DB lock for update in the server manager that occured
    during rest calls that was triggering deadlocks due to the
    file lock synchronization for the rest calls.

    Closes-Bug: #1336251
    Change-Id: Iad3d61e2c23832b3ad760a999fbab7feaa13f805
    (cherry picked from commit ee9fe2458f813b5d367bc7263c7a30a3af46aa2b)