Pls review 6898160 (S)
Mandy Chung
Mandy.Chung at Sun.COM
Wed Nov 4 12:36:52 PST 2009
Looks okay to me. Minor nit:
attachListener.cpp line 267: "unsigned integer" in the error message
should be "unsigned 64-bit value" or something like that.
Mandy
Paul Hohensee wrote:
> 6898160: Need serviceability support for new vm argument type 'uint64_t'.
>
> Webrev here
>
> http://cr.openjdk.java.net/~phh/6898160/webrev.01/
>
> A new vm argument type, 'uint64_t', was added as part of 6887571. The
> jdk library
> code in jdk/share/native/sun/management/Flag.c that fetches all the vm
> product flags
> throws an exception if it doesn't understand the type of an argument.
> The change
> for 6887571 didn't include full serviceability support for uint64_t.
>
> The fix is to add serviceability support for uint64_t. In addition,
> if a flag has a type
> unknown to the jdk library, we now assert in debug builds and ignore
> it in product
> builds. Thus, no jdk library change is necessary.
>
> The failing test,
>
> jdk/test/com/sun/management/HotSpotDiagnosticMXBean/GetDiagnosticOptions.java,
>
>
> now passes.
>
> Thanks,
>
> Paul
>
More information about the serviceability-dev
mailing list