Hi Severin,

Updated: http://cr.openjdk.java.net/~jiefu/8242480/webrev.02/

Please review it.

Thanks a lot.
Best regards,
Jie

On 2020/4/16, 11:40 PM, "Severin Gehwolf" <sgehw...@redhat.com> wrote:
    
    Since you've added a new test, please move them to the jdk docker tests
    in: 
    test/jdk/jdk/internal/platform/docker/

Fixed.
        
    test/hotspot/jtreg/containers/docker/TestGetFreeSwapSpaceSize.java
    
    + * @build sun.hotspot.WhiteBox GetFreeSwapSpaceSize
    + * @run driver ClassFileInstaller -jar whitebox.jar sun.hotspot.WhiteBox 
sun.hotspot.WhiteBox$WhiteBoxPermission
    
    I don't see any reason why WhiteBox would be needed for this test. Is
    that an oversight or am I missing something?

It's my oversight. Thanks for correcting me.
     

Reply via email to