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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 661BAC433EF for ; Wed, 13 Oct 2021 16:27:16 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id DDF6761053 for ; Wed, 13 Oct 2021 16:27:15 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org DDF6761053 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.alibaba.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=kvack.org Received: by kanga.kvack.org (Postfix) id 4FBA5900003; Wed, 13 Oct 2021 12:27:15 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 4A9A2900002; Wed, 13 Oct 2021 12:27:15 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3BE52900003; Wed, 13 Oct 2021 12:27:15 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0079.hostedemail.com [216.40.44.79]) by kanga.kvack.org (Postfix) with ESMTP id 2C2BB900002 for ; Wed, 13 Oct 2021 12:27:15 -0400 (EDT) Received: from smtpin03.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id DFB951842ACC1 for ; Wed, 13 Oct 2021 16:27:14 +0000 (UTC) X-FDA: 78691943988.03.6BE5826 Received: from out30-130.freemail.mail.aliyun.com (out30-130.freemail.mail.aliyun.com [115.124.30.130]) by imf16.hostedemail.com (Postfix) with ESMTP id 23AEDF00008E for ; Wed, 13 Oct 2021 16:27:12 +0000 (UTC) X-Alimail-AntiSpam:AC=PASS;BC=-1|-1;BR=01201311R151e4;CH=green;DM=||false|;DS=||;FP=0|-1|-1|-1|0|-1|-1|-1;HT=e01e04357;MF=rongwei.wang@linux.alibaba.com;NM=1;PH=DS;RN=4;SR=0;TI=SMTPD_---0Uri79JY_1634142428; Received: from 30.30.71.190(mailfrom:rongwei.wang@linux.alibaba.com fp:SMTPD_---0Uri79JY_1634142428) by smtp.aliyun-inc.com(127.0.0.1); Thu, 14 Oct 2021 00:27:09 +0800 Message-ID: Date: Thu, 14 Oct 2021 00:27:08 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:93.0) Gecko/20100101 Thunderbird/93.0 Subject: Re: [PATCH] mm/damon/dbgfs: remove unnecessary variables Content-Language: en-US To: SeongJae Park Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, akpm@linux-foundation.org References: <20211013150413.23027-1-sj@kernel.org> From: Rongwei Wang In-Reply-To: <20211013150413.23027-1-sj@kernel.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: 23AEDF00008E X-Stat-Signature: pb6f6x8rxftqqkdptrszka9npby9wi9s Authentication-Results: imf16.hostedemail.com; dkim=none; dmarc=pass (policy=none) header.from=alibaba.com; spf=pass (imf16.hostedemail.com: domain of rongwei.wang@linux.alibaba.com designates 115.124.30.130 as permitted sender) smtp.mailfrom=rongwei.wang@linux.alibaba.com X-HE-Tag: 1634142432-253950 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: On 10/13/21 11:04 PM, SeongJae Park wrote: > Hi Rongwei, > > > Thank you for this patch! Looks good to me overall. I left a couple of > nitpicks below, though. > > On Wed, 13 Oct 2021 21:09:01 +0800 Rongwei Wang wrote: > > [...] >> @@ -352,7 +350,7 @@ static ssize_t dbgfs_target_ids_write(struct file *file, >> >> nrs = kbuf; >> >> - targets = str_to_target_ids(nrs, ret, &nr_targets); >> + targets = str_to_target_ids(nrs, count, &nr_targets); >> if (!targets) { >> ret = -ENOMEM; >> goto out; >> @@ -378,12 +376,12 @@ static ssize_t dbgfs_target_ids_write(struct file *file, >> goto unlock_out; >> } >> >> - err = damon_set_targets(ctx, targets, nr_targets); >> - if (err) { >> + ret = damon_set_targets(ctx, targets, nr_targets); >> + if (ret < 0) { > > I'd prefer 'if (ret) {', to be consistent with other part. Hi In fact, explicit style, such as 'if (ret < 0)', may be more understandable. But I agree with your advice that to be consistent with other part. And I will update it like your said. > >> if (targetid_is_pid(ctx)) >> dbgfs_put_pids(targets, nr_targets); >> - ret = err; >> - } >> + } else >> + ret = count; > > I'd prefer this to have braces: > https://docs.kernel.org/process/coding-style.html#placing-braces-and-spaces Agree! I will update here. Thanks! > >> >> unlock_out: >> mutex_unlock(&ctx->kdamond_lock); >> @@ -548,8 +546,7 @@ static ssize_t dbgfs_mk_context_write(struct file *file, > [...] > > > Thanks, > SJ >