RFR: 8034944: (process) Improve subprocess handling on Solaris

[Martin Buchholz]

Hi Rob,

Thanks for doing this.

In future, could you include a link to the bug URL?

This is not really a "backport", but a ... sideport (?) from Linux to
Solaris.  My intent was always that such a change is valuable, but I didn't
have the computing/testing resources to do it myself.

Can you explain why Basic.java no longer needs to test the deferred close
handling on Solaris?


On Fri, Mar 21, 2014 at 10:43 AM, Rob McKenna <rob.mckenna at oracle.com>wrote:

> Hi folks,
>
> I'd like to push this change into JDK8 & 7. There is other work going on
> in 9 which will supersede this fix so there is likely no point in pushing
> to that release.
>
> This is backport of the threadpool specific changes from
> https://bugs.openjdk.java.net/browse/JDK-6944584 along with some cosmetic
> changes in order to keep the code somewhat similar.
>
> In a nutshell a new process reaper thread was spawned for every Process
> created by the JDK. This fix runs these reaper threads in a thread pool to
> save on thread creation when creating a lot of new processes.
>
> http://cr.openjdk.java.net/~robm/8034944/webrev.01/
>
>     -Rob
>
>