[RESEND PATCH] tools/power/x86/intel_pstate_tracer: Adjust directory permissions

From: Doug Smythies
Date: Sun Apr 16 2017 - 11:17:46 EST


Depending on what is being done, the intel_pstate_tracer.py script
needs to be run as root, or can be run as a regular user.
If run the first time as root the results directory will be
incorrect for any subsequent run as a regular user. For any run
as root the specific testname subdirectory will not allow any
subsequent file saves by a regular user. Typically, and for example,
the regular user might be attempting to save a .csv file converted to
a spreadsheet with added calculations or graphs.

Override the default folder permissions.

Signed-off-by: Doug Smythies <dsmythies@xxxxxxxxx>
---
tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py b/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py
index fd706ac..7c855508 100755
--- a/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py
+++ b/tools/power/x86/intel_pstate_tracer/intel_pstate_tracer.py
@@ -517,13 +517,19 @@ else:
cpu_mask[i] = 1

if not os.path.exists('results'):
+ # Setting permissions with the mode option doesn't work on all platforms
os.mkdir('results')
+ # so use chmod to set them. Needed because user may or may not be root.
+ os.chmod('results', 0777)

os.chdir('results')
if os.path.exists(testname):
print('The test name directory already exists. Please provide a unique test name. Test re-run not supported, yet.')
sys.exit()
+# Setting permissions with the mode option doesn't work on all platforms
os.mkdir(testname)
+os.chmod(testname, 0777)
+# so use chmod to set them. Needed because user may or may not be root.
os.chdir(testname)

# Temporary (or perhaps not)
--
2.7.4