RFR (XXS) CR 8006997: ContendedPaddingWidth should be uintx

[Aleksey Shipilev]

On 02/08/2013 05:22 PM, David Holmes wrote:
> On 8/02/2013 8:39 PM, Aleksey Shipilev wrote:
>> Ok then. This is the new webrev:
>>   http://cr.openjdk.java.net/~shade/8006997/webrev.01/
>>
>> I haven't found another good place to stick the guarantee() to, so opted
>> to stick it before the actual use. I would be happy to stick it
>> somewhere else where the options are generally checked, but found no
>> non-platform/non-OS specific location.
> 
> arguments.cpp - Arguments::parse_each_vm_init_arg - see
> -XX:MaxDirectMemorySize as an example.

Ok, great, thanks! I will update the webrev and give it another jprt spin.

> Are there any other constraints on this value?

Nope. The upper bound makes no sense anyway, and only affects the
instance size (which, I'd guess, should be checked on it's own?). Going
over sensible padding width is not as catastrophic as going below zero.
It might be even fun to allocate the object with field-per-page placement.

-Aleksey.