RFR: 8065213 Specify and implement PlatformMBeanProvider for looking for all platform MBeans

shanliang shanliang.jiang at oracle.com
Tue Feb 3 13:16:28 UTC 2015


Hi,

Hope this is last time :)
    http://cr.openjdk.java.net/~sjiang/JDK-8065213/02/

1)
Mandy Chung wrote:
> You may want to consider using limited doPrivileged (that can be done 
> in the second phase).
Done as in ManagementFactory:
 878             // get all providers
 879             List<PlatformMBeanProvider> providers
 880                 = 
AccessController.doPrivileged((PrivilegedAction<List<PlatformMBeanProvider>>) 
() -> {
 881                     List<PlatformMBeanProvider> all = new 
ArrayList<>();
 882                     for (PlatformMBeanProvider provider : 
ServiceLoader.loadInstalled(PlatformMBeanProvider.class)) {
 883                         all.add(provider);
 884                     }
 885                     all.add(new DefaultPlatformMBeanProvider());
 886                     return all;
 887                 },
 888                 null,
 889                 new FilePermission("<<ALL FILES>>", "read"),
 890                 new 
RuntimePermission("sun.management.spi.PlatformMBeanProvider"));

2)
The modification to Flag is removed, we get another solution to know 
whether commercial feature is enabled.

3) some mis minors modifications.

Thanks,
Shanliang

   



More information about the serviceability-dev mailing list