orb icon indicating copy to clipboard operation
orb copied to clipboard

100% load in POAImpl.acquireLock

Open glassfishrobot opened this issue 13 years ago • 6 comments

Glassfish threads can hang in an endless loop. This is cause by a bogus exit condition:

{{ // in acquireLock: do { ... try

{ ... // invokation of a function which detects thread interruption and throws InterruptedException }

catch(InterruptedException exc)

{ interrupted = true; }

if(interrupted) Thread.currentThread().interrupt(); } while(true); }}

If the current thread the InterruptedException is caught but does not break the loop.

=> 100% load for the thread. You can end up with many threads and thus all CPUs at 100%.

suggested fix: don't catch the InterruptedException. Only catch an Exception if you can handle it properly. (here it's not handly properly, catching an InterruptedException to interrupt() again is simply nonsens.)

Environment

any

Affected Versions

[current]

glassfishrobot avatar Apr 12 '12 09:04 glassfishrobot

  • Issue Imported From: https://github.com/javaee/glassfish-corba/issues/11
  • Original Issue Raised By:@glassfishrobot
  • Original Issue Assigned To: @russgold

glassfishrobot avatar Sep 05 '18 14:09 glassfishrobot

@glassfishrobot Commented Reported by bebbo

glassfishrobot avatar Apr 12 '12 09:04 glassfishrobot

@glassfishrobot Commented bebbo said: You can try my patched version and update the glassfish-corba-orb.jar (in modules).

glassfishrobot avatar Aug 02 '12 05:08 glassfishrobot

@glassfishrobot Commented File: POAImpl.class Attached By: bebbo

glassfishrobot avatar Aug 02 '12 05:08 glassfishrobot

@glassfishrobot Commented Was assigned to hvilekar

glassfishrobot avatar Apr 12 '12 09:04 glassfishrobot

@glassfishrobot Commented This issue was imported from java.net JIRA GLASSFISH_CORBA-11

glassfishrobot avatar Apr 24 '17 07:04 glassfishrobot