Need reviewer: fix to prtconf on solaris builds

David Katleman david.katleman at oracle.com
Wed Jan 4 12:03:23 PST 2012


On 1/4/2012 11:27 AM, Kelly O'Hair wrote:
> Openjdk6 change... creates annoying error messages during builds on solaris without it.
>
> Don't really need a reviewer since the same exact change was pushed into jdk7 a while back by Katleman.
>
> 7022473: JDK7 still runs /etc/prtconf to find memory size
> http://cr.openjdk.java.net/~ohair/openjdk6/openjdk6-prtconf/webrev/
>
>
> David? This look ok?

It looks identical to the original fix, which I am just now noticing 
that while it works, it should use ":=" rather than "=".

Unless we really need to keep looking up the memory every time 
$(MB_OF_MEMORY) is referenced, which isn't a trival number.

Side note, I know it's just a port, but I've noticed of late that 
prtconf doesn't function in Solaris zones, returning "prtconf: devinfo 
facility not available"

             Dave


More information about the jdk6-dev mailing list