Re: [rvm-research] Implementation of setPriority/getPriority for RVMThread (RVM-750)

Carl Ritson <[email protected]>
Newsgroups gmane.comp.java.jikes.rvm.devel
Message-ID <CAKDym5NxRf9CCu-pEm6d9e2jKNnXrSoaYHCsn2KoOmoOsVvqoA@mail.gmail.com>
> Please find attached a patch which implements thread priority on
> RVMThreads at the OS level.  This addresses RVM-750.

Following on from this, classpath does not appropriately query the VM
thread when creating children.  This means that new threads do not
inherit priority (if it is set while they are running).  This applies
equally to unmodified JikesRVM.

The attached patch (applied to classpath) fixes this.

Carl

------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk

_______________________________________________
Jikesrvm-researchers mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/jikesrvm-researchers
classpath.ThreadPriority.patch (application/octet-stream, 495 B)
--- java/lang/Thread.java	2013-07-25 14:08:12.000000000 +0100
+++ java/lang/Thread.java	2013-07-25 14:04:48.000000000 +0100
@@ -374,8 +374,8 @@
         this.threadId = ++totalThreadsCreated;
       }
 
-    priority = current.priority;
-    daemon = current.daemon;
+    priority = current.getPriority();
+    daemon = current.isDaemon();
     contextClassLoader = current.contextClassLoader;
     contextClassLoaderIsSystemClassLoader =
         current.contextClassLoaderIsSystemClassLoader;
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.