gpu: nvgpu: Allow sysfs write only to root

Allow write access only to root to gk20a sysfs files.

Bug 200001241

Change-Id: Ibafb84ed703dd32743b520e01a57ffc82f8b4ac4
Signed-off-by: Terje Bergstrom <tbergstrom@nvidia.com>
Reviewed-on: http://git-master/r/405028
(cherry picked from commit dc0f977fe8fcacd260bf61ab658c166b004c1fcd)
Reviewed-on: http://git-master/r/406898
Reviewed-by: Mandar Padmawar <mpadmawar@nvidia.com>
Tested-by: Mandar Padmawar <mpadmawar@nvidia.com>

1 files changed, 8 insertions, 6 deletions

diff --git a/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c b/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c
index 14629fbe..a5267eac 100644
--- a/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c
+++ b/drivers/gpu/nvgpu/gk20a/gk20a_sysfs.c
@@ -35,6 +35,7 @@
    the resolution of ptimer. */
 #define PTIMER_REF_FREQ_HZ                      31250000
 
+#define ROOTRW (S_IRWXU|S_IRGRP|S_IROTH)
 
 static ssize_t elcg_enable_store(struct device *device,
         struct device_attribute *attr, const char *buf, size_t count)
@@ -73,7 +74,7 @@ static ssize_t elcg_enable_read(struct device *device,
         return sprintf(buf, "%d\n", g->elcg_enabled ? 1 : 0);
 }
 
-static DEVICE_ATTR(elcg_enable, S_IRWXUGO, elcg_enable_read, elcg_enable_store);
+static DEVICE_ATTR(elcg_enable, ROOTRW, elcg_enable_read, elcg_enable_store);
 
 static ssize_t blcg_enable_store(struct device *device,
         struct device_attribute *attr, const char *buf, size_t count)
@@ -109,7 +110,8 @@ static ssize_t blcg_enable_read(struct device *device,
         return sprintf(buf, "%d\n", g->blcg_enabled ? 1 : 0);
 }
 
-static DEVICE_ATTR(blcg_enable, S_IRWXUGO, blcg_enable_read, blcg_enable_store);
+
+static DEVICE_ATTR(blcg_enable, ROOTRW, blcg_enable_read, blcg_enable_store);
 
 static ssize_t slcg_enable_store(struct device *device,
         struct device_attribute *attr, const char *buf, size_t count)
@@ -151,7 +153,7 @@ static ssize_t slcg_enable_read(struct device *device,
         return sprintf(buf, "%d\n", g->slcg_enabled ? 1 : 0);
 }
 
-static DEVICE_ATTR(slcg_enable, S_IRWXUGO, slcg_enable_read, slcg_enable_store);
+static DEVICE_ATTR(slcg_enable, ROOTRW, slcg_enable_read, slcg_enable_store);
 
 static ssize_t ptimer_scale_factor_show(struct device *dev,
                                                 struct device_attribute *attr,
@@ -203,7 +205,7 @@ static ssize_t railgate_delay_show(struct device *dev,
         struct gk20a_platform *platform = dev_get_drvdata(dev);
         return snprintf(buf, PAGE_SIZE, "%d\n", platform->railgate_delay);
 }
-static DEVICE_ATTR(railgate_delay, S_IRWXUGO, railgate_delay_show,
+static DEVICE_ATTR(railgate_delay, ROOTRW, railgate_delay_show,
                    railgate_delay_store);
 
 static ssize_t clockgate_delay_store(struct device *dev,
@@ -229,7 +231,7 @@ static ssize_t clockgate_delay_show(struct device *dev,
         struct gk20a_platform *platform = dev_get_drvdata(dev);
         return snprintf(buf, PAGE_SIZE, "%d\n", platform->clockgate_delay);
 }
-static DEVICE_ATTR(clockgate_delay, S_IRWXUGO, clockgate_delay_show,
+static DEVICE_ATTR(clockgate_delay, ROOTRW, clockgate_delay_show,
                    clockgate_delay_store);
 
 static ssize_t counters_show(struct device *dev,
@@ -305,7 +307,7 @@ static ssize_t elpg_enable_read(struct device *device,
         return sprintf(buf, "%d\n", g->elpg_enabled ? 1 : 0);
 }
 
-static DEVICE_ATTR(elpg_enable, S_IRWXUGO, elpg_enable_read, elpg_enable_store);
+static DEVICE_ATTR(elpg_enable, ROOTRW, elpg_enable_read, elpg_enable_store);
 
 void gk20a_remove_sysfs(struct device *dev)
 {