Re: RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-04-04 Thread Kevin Walls
On Fri, 31 Mar 2023 08:24:19 GMT, Kevin Walls wrote: > The Permissions in DCmds relate to remote usage over JMX. > "monitor" is generally for reading information, and "control" is generally > for making changes. > The DCmds for changing compiler directives should have "control" as the > requir

Re: RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-04-04 Thread Serguei Spitsyn
On Fri, 31 Mar 2023 08:24:19 GMT, Kevin Walls wrote: > The Permissions in DCmds relate to remote usage over JMX. > "monitor" is generally for reading information, and "control" is generally > for making changes. > The DCmds for changing compiler directives should have "control" as the > requir

Re: RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-04-03 Thread Chris Plummer
On Fri, 31 Mar 2023 08:24:19 GMT, Kevin Walls wrote: > The Permissions in DCmds relate to remote usage over JMX. > "monitor" is generally for reading information, and "control" is generally > for making changes. > The DCmds for changing compiler directives should have "control" as the > requir

Re: RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-04-02 Thread Kevin Walls
On Fri, 31 Mar 2023 08:24:19 GMT, Kevin Walls wrote: > The Permissions in DCmds relate to remote usage over JMX. > "monitor" is generally for reading information, and "control" is generally > for making changes. > The DCmds for changing compiler directives should have "control" as the > requir

Re: RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-03-31 Thread Chris Plummer
On Fri, 31 Mar 2023 08:24:19 GMT, Kevin Walls wrote: > The Permissions in DCmds relate to remote usage over JMX. > "monitor" is generally for reading information, and "control" is generally > for making changes. > The DCmds for changing compiler directives should have "control" as the > requir

Re: RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-03-31 Thread Kevin Walls
On Fri, 31 Mar 2023 08:24:19 GMT, Kevin Walls wrote: > The Permissions in DCmds relate to remote usage over JMX. > "monitor" is generally for reading information, and "control" is generally > for making changes. > The DCmds for changing compiler directives should have "control" as the > requir

RFR: 8305237: CompilerDirectives DCmds permissions correction

2023-03-31 Thread Kevin Walls
The Permissions in DCmds relate to remote usage over JMX. "monitor" is generally for reading information, and "control" is generally for making changes. The DCmds for changing compiler directives should have "control" as the required permission. Tests in test/hotspot/jtreg/serviceability/dcmd/c