> thanks for the patches. We appreciate your contributions.
> There was an implementation of RVM-750 in MRP (see
> Could you comment on the different approaches?
The approach in the linked patch is the same as my patch for systems
where pthread static priority is possible; however, it will have no
effect on Linux. The linked implementation is incomplete as the
priority of new threads will not be set when they are created, and
getPriority will not return the OS priority of a thread if it has been
set correctly or otherwise. This is a point for discussion in many
implementations setPriority always succeeds and the result of
getPriority is always the value passed to the last setPriority,
regardless of the OS priority of the thread. My approach will always
return the OS priority.
> I looked briefly at your patch and have the following questions/suggestions:
> -is the code portable enough? IIRC AIX is the only operating system that
> we support and that you did not test.
It should be portable, although I don't have access to an AIX system to test.
> -Would debugging variables for thread priority (e.g. something
> comparable to "traceAcct" in RVMThread) be useful?
Perhaps, although existing trace variables might be equally applicable.
> -the code of setPriority() in RVMThread would probably be more readable
> if the result of the SysCall was saved to its own variable.
True, and this would be desirable for debugging output.
> -the patch contains trailing whitespace (which leads to checkstyle
Ah, sorry about that.
Would you like me to fix this and the above, and provide new patches?
> We need contributor statements to apply the patches. You can also
> provide us with a contributor statement for all contributions (text can
> be found on the bottom of the "Contributions" wiki page with the other
STATEMENT OF ORIGIN FOR A SINGLE CONTRIBUTOR
I, Carl Ritson:
(a) represent that either:
(i) I am the only author and owner of the contributed software
(described as/entitled setPriority() implementation for RVMThread),
which was neither derived nor copied from any other software,
(ii) that any exception to (i) is software which was obtained under the
EPL (Eclipse Public License),
(b) hereby agree to license this contributed software under the EPL.