mm/damon/sysfs: return 'err' value when call kstrtoul() failed

We had better return the 'err' value when calling kstrtoul() failed, so
the user will know why it really fails, there do little change, let it
return the 'err' value when failed.

Link: https://lkml.kernel.org/r/6329ebe0.050a0220.ec4bd.297cSMTPIN_ADDED_BROKEN@mx.google.com
Suggested-by: SeongJae Park <sj@kernel.org>
Signed-off-by: Xin Hao <xhao@linux.alibaba.com>
Reviewed-by: SeongJae Park <sj@kernel.org>
Signed-off-by: SeongJae Park <sj@kernel.org>
Reviewed-by: Xin Hao <xhao@linux.alibaba.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c
index 0cca1909bf67bc56b8f47d59062bfbcaf2ccf2ee..455215a5c05981c9ef61f5a94747b21e8aac5c9c 100644 (file)
--- a/mm/damon/sysfs.c
+++ b/mm/damon/sysfs.c
@@ -58,7 +58,7 @@ static ssize_t min_store(struct kobject *kobj, struct kobj_attribute *attr,
 
        err = kstrtoul(buf, 0, &min);
        if (err)
-               return -EINVAL;
+               return err;
 
        range->min = min;
        return count;
@@ -83,7 +83,7 @@ static ssize_t max_store(struct kobject *kobj, struct kobj_attribute *attr,
 
        err = kstrtoul(buf, 0, &max);
        if (err)
-               return -EINVAL;
+               return err;
 
        range->max = max;
        return count;
@@ -291,9 +291,7 @@ static ssize_t interval_us_store(struct kobject *kobj,
                        struct damon_sysfs_watermarks, kobj);
        int err = kstrtoul(buf, 0, &watermarks->interval_us);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static ssize_t high_show(struct kobject *kobj,
@@ -312,9 +310,7 @@ static ssize_t high_store(struct kobject *kobj,
                        struct damon_sysfs_watermarks, kobj);
        int err = kstrtoul(buf, 0, &watermarks->high);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static ssize_t mid_show(struct kobject *kobj,
@@ -333,9 +329,7 @@ static ssize_t mid_store(struct kobject *kobj,
                        struct damon_sysfs_watermarks, kobj);
        int err = kstrtoul(buf, 0, &watermarks->mid);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static ssize_t low_show(struct kobject *kobj,
@@ -354,9 +348,7 @@ static ssize_t low_store(struct kobject *kobj,
                        struct damon_sysfs_watermarks, kobj);
        int err = kstrtoul(buf, 0, &watermarks->low);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static void damon_sysfs_watermarks_release(struct kobject *kobj)
@@ -437,9 +429,7 @@ static ssize_t sz_permil_store(struct kobject *kobj,
                        struct damon_sysfs_weights, kobj);
        int err = kstrtouint(buf, 0, &weights->sz);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static ssize_t nr_accesses_permil_show(struct kobject *kobj,
@@ -458,9 +448,7 @@ static ssize_t nr_accesses_permil_store(struct kobject *kobj,
                        struct damon_sysfs_weights, kobj);
        int err = kstrtouint(buf, 0, &weights->nr_accesses);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static ssize_t age_permil_show(struct kobject *kobj,
@@ -479,9 +467,7 @@ static ssize_t age_permil_store(struct kobject *kobj,
                        struct damon_sysfs_weights, kobj);
        int err = kstrtouint(buf, 0, &weights->age);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static void damon_sysfs_weights_release(struct kobject *kobj)
@@ -1111,9 +1097,7 @@ static ssize_t start_store(struct kobject *kobj, struct kobj_attribute *attr,
                        struct damon_sysfs_region, kobj);
        int err = kstrtoul(buf, 0, &region->start);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static ssize_t end_show(struct kobject *kobj, struct kobj_attribute *attr,
@@ -1132,9 +1116,7 @@ static ssize_t end_store(struct kobject *kobj, struct kobj_attribute *attr,
                        struct damon_sysfs_region, kobj);
        int err = kstrtoul(buf, 0, &region->end);
 
-       if (err)
-               return -EINVAL;
-       return count;
+       return err ? err : count;
 }
 
 static void damon_sysfs_region_release(struct kobject *kobj)
@@ -1528,7 +1510,7 @@ static ssize_t sample_us_store(struct kobject *kobj,
        int err = kstrtoul(buf, 0, &us);
 
        if (err)
-               return -EINVAL;
+               return err;
 
        intervals->sample_us = us;
        return count;
@@ -1552,7 +1534,7 @@ static ssize_t aggr_us_store(struct kobject *kobj, struct kobj_attribute *attr,
        int err = kstrtoul(buf, 0, &us);
 
        if (err)
-               return -EINVAL;
+               return err;
 
        intervals->aggr_us = us;
        return count;
@@ -1576,7 +1558,7 @@ static ssize_t update_us_store(struct kobject *kobj,
        int err = kstrtoul(buf, 0, &us);
 
        if (err)
-               return -EINVAL;
+               return err;
 
        intervals->update_us = us;
        return count;