From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4EC05C6FA82 for ; Wed, 21 Sep 2022 01:28:33 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 369EE6B0072; Tue, 20 Sep 2022 21:28:32 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 319AD940008; Tue, 20 Sep 2022 21:28:32 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 208C9940007; Tue, 20 Sep 2022 21:28:32 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0012.hostedemail.com [216.40.44.12]) by kanga.kvack.org (Postfix) with ESMTP id 1128A6B0072 for ; Tue, 20 Sep 2022 21:28:32 -0400 (EDT) Received: from smtpin29.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id DADE91A0C2C for ; Wed, 21 Sep 2022 01:28:31 +0000 (UTC) X-FDA: 79934357622.29.D6FC92C Received: from out30-43.freemail.mail.aliyun.com (out30-43.freemail.mail.aliyun.com [115.124.30.43]) by imf18.hostedemail.com (Postfix) with ESMTP id E900C1C0058 for ; Wed, 21 Sep 2022 01:28:29 +0000 (UTC) X-Alimail-AntiSpam:AC=PASS;BC=-1|-1;BR=01201311R191e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=ay29a033018046059;MF=xhao@linux.alibaba.com;NM=1;PH=DS;RN=5;SR=0;TI=SMTPD_---0VQLXQDo_1663723704; Received: from 30.240.97.89(mailfrom:xhao@linux.alibaba.com fp:SMTPD_---0VQLXQDo_1663723704) by smtp.aliyun-inc.com; Wed, 21 Sep 2022 09:28:26 +0800 Message-ID: <74773dd0-583a-afb4-ad73-9b324c715d08@linux.alibaba.com> Date: Wed, 21 Sep 2022 09:28:24 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.2.2 Subject: Re: [PATCH v3] mm/damon/sysfs: return 'err' value when call kstrtoul() failed To: SeongJae Park Cc: akpm@linux-foundation.org, damon@lists.linux.dev, linux-mm@kvack.org, linux-kernel@vger.kernel.org References: <20220920025158.70293-1-xhao@linux.alibaba.com> From: haoxin In-Reply-To: <20220920025158.70293-1-xhao@linux.alibaba.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1663723711; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=VRSeWxr62qKEU1IYoF8uxf+FmEvmApHbUMnOzalWgFo=; b=7E7rZ7wBg1NXfZk7TqfXKhwkT7lKlzDOxJZXUSSST4vxuuSvZzFtNUgl26rRvjzUnx/pJO P0kVCTHIuFlF1jd0DUCCly9uiGBrscNC2B57xOFUzxxA93D3Y8gyr1RhCXUVPF5Xeor7p8 lTrfqBc4xY9kOYqcRxJNld+AMPIvuvQ= ARC-Authentication-Results: i=1; imf18.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=alibaba.com; spf=pass (imf18.hostedemail.com: domain of xhao@linux.alibaba.com designates 115.124.30.43 as permitted sender) smtp.mailfrom=xhao@linux.alibaba.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1663723711; a=rsa-sha256; cv=none; b=uKGkEtT33YW0CLiR4vQZOm68Znp4GyuBKeXIZN8OFkTtYxaEdZv7tnHxibzwikhf0dJZFD Jz8gy98DiBJi0+7KBxowB9VGgcpnQ3hUUysYEGR14cCwuSDG+vPQHc5DPD2rLRVFWiO3tH wDlJLCdXlUa4mgZu6kNVBLMknJzu+Ns= Authentication-Results: imf18.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=alibaba.com; spf=pass (imf18.hostedemail.com: domain of xhao@linux.alibaba.com designates 115.124.30.43 as permitted sender) smtp.mailfrom=xhao@linux.alibaba.com X-Rspam-User: X-Rspamd-Server: rspam01 X-Stat-Signature: ru37d5irr95iqemzndduy4g9my7oqxcg X-Rspamd-Queue-Id: E900C1C0058 X-HE-Tag: 1663723709-460416 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: Hi SJ, 在 2022/9/21 上午12:35, SeongJae Park 写道: > From: Xin Hao > > 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. > > Suggested-by: SeongJae Park > Signed-off-by: Xin Hao > Reviewed-by: SeongJae Park > Signed-off-by: SeongJae Park > > --- > Changes from v2 > (https://lore.kernel.org/damon/20220920025158.70293-1-xhao@linux.alibaba.com/) > - Move patch changelog outside of the commit message area Thanks for fix this stupid bug，i will nerver make it happen once again. Reviewed-by: Xin Hao > > Changes from v1 > (https://lore.kernel.org/linux-mm/20220919170305.61335-1-sj@kernel.org/T/) > - keep the 'err' variable, and return the 'err' value when failed. > > mm/damon/sysfs.c | 46 ++++++++++++++-------------------------------- > 1 file changed, 14 insertions(+), 32 deletions(-) > > diff --git a/mm/damon/sysfs.c b/mm/damon/sysfs.c > index 0cca1909bf67..455215a5c059 100644 > --- 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, ®ion->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, ®ion->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; > -- > 2.31.0