RFR 8046588: test for SO_FLOW_SLA availability does not check for EACCESS

2014-06-12 Thread Michael McMahon
Could I get this change reviewed please? We need to check if the current process has permission to use the SO_FLOW_SLA socket option as well as test if the feature is installed. The problem would cause the new test to fail on Solaris machines with S11.2 unless running with the required privile

Re: RFR 8046588: test for SO_FLOW_SLA availability does not check for EACCESS

2014-06-12 Thread Alan Bateman
On 12/06/2014 18:04, Michael McMahon wrote: Could I get this change reviewed please? We need to check if the current process has permission to use the SO_FLOW_SLA socket option as well as test if the feature is installed. The problem would cause the new test to fail on Solaris machines with

Re: RFR 8046588: test for SO_FLOW_SLA availability does not check for EACCESS

2014-06-12 Thread Michael McMahon
On 12/06/14 18:47, Alan Bateman wrote: On 12/06/2014 18:04, Michael McMahon wrote: Could I get this change reviewed please? We need to check if the current process has permission to use the SO_FLOW_SLA socket option as well as test if the feature is installed. The problem would cause the new

Re: RFR 8046588: test for SO_FLOW_SLA availability does not check for EACCESS

2014-06-12 Thread Alan Bateman
On 12/06/2014 20:15, Michael McMahon wrote: It would be possible to change the error back, but what about supportedOptions() - what should that return? It doesn't seem right that it would include an option that cannot be used and how do we test it, if we can't tell whether the option is usabl

Re: RFR 8046588: test for SO_FLOW_SLA availability does not check for EACCESS

2014-06-12 Thread Michael McMahon
On 12/06/14 20:35, Alan Bateman wrote: On 12/06/2014 20:15, Michael McMahon wrote: It would be possible to change the error back, but what about supportedOptions() - what should that return? It doesn't seem right that it would include an option that cannot be used and how do we test it, if w