[OVSDB-345] thread leak when connecting/disconnecting ovs nodes in a loop Created: 27/May/16 Updated: 15/Jun/16 Resolved: 15/Jun/16 |
|
| Status: | Resolved |
| Project: | ovsdb |
| Component/s: | Library |
| Affects Version/s: | unspecified |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Jamo Luhrsen | Assignee: | Dileep Ranganathan |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 5976 |
| Description |
|
just do this in a loop: ovs-vsctl set-manager ${ODL_IP} 6640 each connection is generating some threads but they are not cleaned feature installed is odl-ovsdb-southbound-impl-rest |
| Comments |
| Comment by Anil Vishnoi [ 27/May/16 ] |
|
Adding more details from the mail discussion. This is what i did: 1) Started the controller with ovsdb You can look at these thread dump at following urls This is what i see: Total Threads in First Dump : 98 I see three category of threads Thread "PassiveConnection-X" (9 threads) – These are expected. Thread "pool-XX-thread-Y" (197 threads) – These threads are problem. Each thread has the same stack. This stack related to is executor pool threads. As of now I am not sure whether we are creating these threads or any of the core controller component is causing these. I am working on a patch, that actually name all executor pools in the ovsdb code, so that we can figure out whether these "pool-xxx-thread-y" are created by us or someone else. BUT indeed it's a thread leak somewhere. |
| Comment by Dileep Ranganathan [ 07/Jun/16 ] |
| Comment by Sam Hague [ 14/Jun/16 ] |
|
be: https://git.opendaylight.org/gerrit/#/c/39986/ |
| Comment by Jamo Luhrsen [ 14/Jun/16 ] |
|
I'm going to bring a CSIT test case to check for this. Once I do, I'll |
| Comment by Anil Vishnoi [ 14/Jun/16 ] |
|
I was about to ask this and you sensed it |
| Comment by Sam Hague [ 15/Jun/16 ] |
|
Anil, Jamo, add the test cases to the spreadsheet Venkat and Josh started: https://docs.google.com/spreadsheets/d/1n4yCFc9kogkkRrWwFjrVJyDPeO4lu79PZtwjuourDWg/edit#gid=0 |