[AMQ-4297] stomp deadlock on commit

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.8.0
• Fix Version/s: 5.8.0
• Component/s: STOMP
• Regression: Unit Test Broken
• Assignee: Gary Tully
• Reporter: Gary Tully
• Created: 05/Feb/13 18:27
• Updated: 05/Feb/13 18:32
• Resolved: 05/Feb/13 18:32

Description

Found one Java-level deadlock:
=============================
"ActiveMQ Transport: ssl://27.0.0.1:63934":
  waiting for ownable synchronizer 7f40f40b0, (a java.util.concurrent.locks.ReentrantLock$NonfairSync),
  which is held by "ActiveMQ BrokerService[localhost] Task-2"
"ActiveMQ BrokerService[localhost] Task-2":
  waiting to lock monitor 7f90ca08ea30 (object 7f35b97b0, a org.apache.activemq.transport.stomp.StompSubscription),
  which is held by "ActiveMQ Transport: ssl://27.0.0.1:63934"

Java stack information for the threads listed above:
===================================================
"ActiveMQ Transport: ssl://27.0.0.1:63934":
	at sun.misc.Unsafe.park(Native Method)
	- parking to wait for  <7f40f40b0> (a java.util.concurrent.locks.ReentrantLock$NonfairSync)
	at java.util.concurrent.locks.LockSupport.park(LockSupport.java:156)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.parkAndCheckInterrupt(AbstractQueuedSynchronizer.java:811)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquireQueued(AbstractQueuedSynchronizer.java:842)
	at java.util.concurrent.locks.AbstractQueuedSynchronizer.acquire(AbstractQueuedSynchronizer.java:1178)
	at java.util.concurrent.locks.ReentrantLock$NonfairSync.lock(ReentrantLock.java:186)
	at java.util.concurrent.locks.ReentrantLock.lock(ReentrantLock.java:262)
	at org.apache.activemq.transport.MutexTransport.onCommand(MutexTransport.java:43)
	at org.apache.activemq.transport.AbstractInactivityMonitor.onCommand(AbstractInactivityMonitor.java:288)
	at org.apache.activemq.transport.stomp.StompTransportFilter.sendToActiveMQ(StompTransportFilter.java:84)
	at org.apache.activemq.transport.stomp.StompSubscription.onStompCommit(StompSubscription.java:122)
	- locked <7f35b97b0> (a org.apache.activemq.transport.stomp.StompSubscription)
	at org.apache.activemq.transport.stomp.ProtocolConverter.onStompCommit(ProtocolConverter.java:502)
	at org.apache.activemq.transport.stomp.ProtocolConverter.onStompCommand(ProtocolConverter.java:241)
	at org.apache.activemq.transport.stomp.StompTransportFilter.onCommand(StompTransportFilter.java:73)
	at org.apache.activemq.transport.TransportSupport.doConsume(TransportSupport.java:83)
	at org.apache.activemq.transport.tcp.SslTransport.doConsume(SslTransport.java:91)
	at org.apache.activemq.transport.stomp.StompSslTransportFactory$1$1.doConsume(StompSslTransportFactory.java:70)
	at org.apache.activemq.transport.tcp.TcpTransport.doRun(TcpTransport.java:214)
	at org.apache.activemq.transport.tcp.TcpTransport.run(TcpTransport.java:196)
	at java.lang.Thread.run(Thread.java:680)
"ActiveMQ BrokerService[localhost] Task-2":
	at org.apache.activemq.transport.stomp.StompSubscription.onMessageDispatch(StompSubscription.java:70)
	- waiting to lock <7f35b97b0> (a org.apache.activemq.transport.stomp.StompSubscription)
	at org.apache.activemq.transport.stomp.ProtocolConverter.onActiveMQCommand(ProtocolConverter.java:857)
	at org.apache.activemq.transport.stomp.StompTransportFilter.oneway(StompTransportFilter.java:61)
	at org.apache.activemq.transport.AbstractInactivityMonitor.doOnewaySend(AbstractInactivityMonitor.java:322)
	at org.apache.activemq.transport.AbstractInactivityMonitor.oneway(AbstractInactivityMonitor.java:304)
	at org.apache.activemq.transport.MutexTransport.oneway(MutexTransport.java:68)
	at org.apache.activemq.broker.TransportConnection.dispatch(TransportConnection.java:1378)
	at org.apache.activemq.broker.TransportConnection.processDispatch(TransportConnection.java:897)
	at org.apache.activemq.broker.TransportConnection.iterate(TransportConnection.java:943)
	at org.apache.activemq.thread.PooledTaskRunner.runTask(PooledTaskRunner.java:129)
	at org.apache.activemq.thread.PooledTaskRunner$1.run(PooledTaskRunner.java:47)
	at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
	at java.lang.Thread.run(Thread.java:680)

Found 1 deadlock.

from org.apache.activemq.transport.stomp.StompTest#testPrefetchSize