https://jira.opendaylight.org This file is an XML representation of an issue en-us 8.20.10 820010 22-06-2022 https://jira.opendaylight.org/browse/NETVIRT-614 netvirt <p>Right now BgpConfigurationManager uses its intrinsic lock in lot of its methods unnecessarily which can potentially lead to thread blocking (or) even thread deadlock since it has other inner DCN listener classes which add/update/update methods are also synchronized.</p> <p>For more details look at <a href="https://bugs.opendaylight.org/show_bug.cgi?id=8142" class="external-link" target="_blank" rel="nofollow noopener">https://bugs.opendaylight.org/show_bug.cgi?id=8142</a> <br/> For logs: <a href="https://drive.google.com/open?id=0B_Rr7XjF0yoHc2EwT1psVGQwRVE" class="external-link" target="_blank" rel="nofollow noopener">https://drive.google.com/open?id=0B_Rr7XjF0yoHc2EwT1psVGQwRVE</a></p> <p>Operating System: All<br/> Platform: All</p> NETVIRT-614

Unnecessary usage of java lock in BgpConfigurationManager which leads to thread blocking indefinitely

Bug Medium Resolved Cannot Reproduce Vyshakh Krishnan Periyasamy Palanisamy Mon, 10 Apr 2017 10:03:02 +0000 Fri, 6 Apr 2018 00:08:03 +0000 Fri, 6 Apr 2018 00:08:03 +0000 Carbon General 0 5 <p>In another issue, we are seeing that many threads are blocked due to the lock at the below location</p> <p>"AsyncDataTreeChangeListenerBase-DataTreeChangeHandler-0" #561 daemon prio=5 os_prio=0 tid=0x00007ff240012800 nid=0x2b9a runnable <span class="error">&#91;0x00007ff15adb7000&#93;</span><br/> java.lang.Thread.State: RUNNABLE<br/> at java.net.SocketInputStream.socketRead0(Native Method)<br/> at java.net.SocketInputStream.socketRead(SocketInputStream.java:116)<br/> at java.net.SocketInputStream.read(SocketInputStream.java:170)<br/> at java.net.SocketInputStream.read(SocketInputStream.java:141)<br/> at java.io.BufferedInputStream.fill(BufferedInputStream.java:246)<br/> at java.io.BufferedInputStream.read1(BufferedInputStream.java:286)<br/> at java.io.BufferedInputStream.read(BufferedInputStream.java:345)</p> <ul class="alternate" type="square"> <li>locked &lt;0x00000006ccacc490&gt; (a java.io.BufferedInputStream)<br/> at org.apache.thrift.transport.TIOStreamTransport.read(TIOStreamTransport.java:127)<br/> at org.apache.thrift.transport.TTransport.readAll(TTransport.java:84)<br/> at org.apache.thrift.protocol.TBinaryProtocol.readAll(TBinaryProtocol.java:362)<br/> at org.apache.thrift.protocol.TBinaryProtocol.readI32(TBinaryProtocol.java:284)<br/> at org.apache.thrift.protocol.TBinaryProtocol.readMessageBegin(TBinaryProtocol.java:191)<br/> at org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:69)<br/> at org.opendaylight.netvirt.bgpmanager.thrift.gen.BgpConfigurator$Client.recv_addVrf(BgpConfigurator.java:296)<br/> at org.opendaylight.netvirt.bgpmanager.thrift.gen.BgpConfigurator$Client.addVrf(BgpConfigurator.java:280)<br/> at org.opendaylight.netvirt.bgpmanager.thrift.client.BgpRouter.dispatch(BgpRouter.java:199)<br/> at org.opendaylight.netvirt.bgpmanager.thrift.client.BgpRouter.addVrf(BgpRouter.java:317)</li> <li>locked &lt;0x00000006cca0c458&gt; (a org.opendaylight.netvirt.bgpmanager.thrift.client.BgpRouter)<br/> at org.opendaylight.netvirt.bgpmanager.BgpConfigurationManager$VrfsReactor.add(BgpConfigurationManager.java:1177)</li> <li>locked &lt;0x00000006ca6e6318&gt; (a org.opendaylight.netvirt.bgpmanager.BgpConfigurationManager)</li> <li>locked &lt;0x00000006cbe4b560&gt; (a org.opendaylight.netvirt.bgpmanager.BgpConfigurationManager$VrfsReactor)<br/> at org.opendaylight.netvirt.bgpmanager.BgpConfigurationManager$VrfsReactor.add(BgpConfigurationManager.java:1152)<br/> at org.opendaylight.genius.datastoreutils.AsyncDataTreeChangeListenerBase$DataTreeChangeHandler.run(AsyncDataTreeChangeListenerBase.java:155)<br/> at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)<br/> at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)<br/> at java.lang.Thread.run(Thread.java:745)</li> </ul> Development External issue ID 8195 External issue URL Rank 0|i01svz: