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 X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0FC2DC54FCC for ; Wed, 22 Apr 2020 00:36:07 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id CCE9E2071C for ; Wed, 22 Apr 2020 00:36:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CCE9E2071C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 5A2A98E0005; Tue, 21 Apr 2020 20:36:06 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 52BF88E0003; Tue, 21 Apr 2020 20:36:06 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 3F42C8E0005; Tue, 21 Apr 2020 20:36:06 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0022.hostedemail.com [216.40.44.22]) by kanga.kvack.org (Postfix) with ESMTP id 2374D8E0003 for ; Tue, 21 Apr 2020 20:36:06 -0400 (EDT) Received: from smtpin21.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with ESMTP id D9EB3181AEF1A for ; Wed, 22 Apr 2020 00:36:05 +0000 (UTC) X-FDA: 76733623890.21.spy96_6f5debd966435 X-HE-Tag: spy96_6f5debd966435 X-Filterd-Recvd-Size: 3320 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by imf15.hostedemail.com (Postfix) with ESMTP for ; Wed, 22 Apr 2020 00:36:05 +0000 (UTC) IronPort-SDR: rCLw2D50Dizs1l7rLhCpfZLVJnHO8Buvw3obrna0fV4OKN3KklG7RYFqkJYmoHlWGnNvIwZOZv UsGu9EQ0jxpg== X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by fmsmga106.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Apr 2020 17:36:02 -0700 IronPort-SDR: VK+prjcz24F4zDfuPl/MNTubRPj3dw6PxJaaHisBS9GDnCGNh5MVUwe8moasHTK/IVvQaFv3aM rc32sl7Glnzg== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.72,412,1580803200"; d="scan'208";a="456303528" Received: from yhuang-dev.sh.intel.com (HELO yhuang-dev) ([10.239.159.23]) by fmsmga005.fm.intel.com with ESMTP; 21 Apr 2020 17:36:02 -0700 From: "Huang\, Ying" To: Wei Yang Cc: , , , Subject: Re: [Patch v2 1/3] mm/swapfile.c: found_free could be represented by (tmp < max) References: <20200421213824.8099-1-richard.weiyang@gmail.com> Date: Wed, 22 Apr 2020 08:36:01 +0800 In-Reply-To: <20200421213824.8099-1-richard.weiyang@gmail.com> (Wei Yang's message of "Tue, 21 Apr 2020 21:38:22 +0000") Message-ID: <87r1wgqrv2.fsf@yhuang-dev.intel.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=ascii 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: Wei Yang writes: > This is not necessary to use the variable found_free to record the > status. Just check tmp and max is enough. > > Signed-off-by: Wei Yang All 3 patches looks good to me. Thanks! Reviewed-by: "Huang, Ying" Best Regards, Huang, Ying > --- > mm/swapfile.c | 11 +++-------- > 1 file changed, 3 insertions(+), 8 deletions(-) > > diff --git a/mm/swapfile.c b/mm/swapfile.c > index 469ab417ed43..d203cdc6750a 100644 > --- a/mm/swapfile.c > +++ b/mm/swapfile.c > @@ -601,7 +601,6 @@ static bool scan_swap_map_try_ssd_cluster(struct swap_info_struct *si, > { > struct percpu_cluster *cluster; > struct swap_cluster_info *ci; > - bool found_free; > unsigned long tmp, max; > > new_cluster: > @@ -623,8 +622,6 @@ static bool scan_swap_map_try_ssd_cluster(struct swap_info_struct *si, > return false; > } > > - found_free = false; > - > /* > * Other CPUs can use our cluster if they can't find a free cluster, > * check if there is still free entry in the cluster > @@ -638,21 +635,19 @@ static bool scan_swap_map_try_ssd_cluster(struct swap_info_struct *si, > } > ci = lock_cluster(si, tmp); > while (tmp < max) { > - if (!si->swap_map[tmp]) { > - found_free = true; > + if (!si->swap_map[tmp]) > break; > - } > tmp++; > } > unlock_cluster(ci); > - if (!found_free) { > + if (tmp >= max) { > cluster_set_null(&cluster->index); > goto new_cluster; > } > cluster->next = tmp + 1; > *offset = tmp; > *scan_base = tmp; > - return found_free; > + return tmp < max; > } > > static void __del_from_avail_list(struct swap_info_struct *p)