Review request: 8025059: Metspace::should_expand mixes bytes and words in check against MaxMetaspaceSize

[Stefan Karlsson]

http://cr.openjdk.java.net/~stefank/8025059/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8025059

The following code in Metaspace::should_expand:

if (!FLAG_IS_DEFAULT(MaxMetaspaceSize)) {
   size_t real_allocated = Metaspace::space_list()->reserved_words() +
       MetaspaceAux::allocated_capacity_bytes(Metaspace::ClassType);
   if (real_allocated >= MaxMetaspaceSize) {
     return false;
   }

doesn't convert Metaspace::space_list()->reserved_words() to bytes, as 
it should.

Note that when JDK-8024547 gets fixed, this check will be rewritten to 
limit the amount of committed memory in both the Class and NonClass 
metaspaces. See:
https://bugs.openjdk.java.net/browse/JDK-8024547: MaxMetaspaceSize 
should limit the committed memory used by the metaspaces

Testing: I've run our internal testing that sets the MaxMetaspaceSize.

thanks,
StefanK