From patchwork Sat Jul 29 17:43:50 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kemeng Shi X-Patchwork-Id: 128089 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp945806vqg; Sat, 29 Jul 2023 03:04:56 -0700 (PDT) X-Google-Smtp-Source: APBJJlHdrFUNHzi0EePCpKGl0dFz5EiMn81KrlSQNWsK1qyYPTKdtp/m0rFLJNPomyhF4EU0yIp5 X-Received: by 2002:a17:90a:17ec:b0:268:a26:d9ee with SMTP id q99-20020a17090a17ec00b002680a26d9eemr3644898pja.46.1690625095799; Sat, 29 Jul 2023 03:04:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690625095; cv=none; d=google.com; s=arc-20160816; b=MOMlwHntpUcMNV9z3pu++AfBSDJlhS73r3YpPlgijcUai7rhoL5TfpNlVFtsV0rK70 3e5APal9rxktdSCkGTKui8EvASgl0sArinNoVi/7NQWG0qg+G+rYM15gZgqqxGLH3K7n ePW3Ak7Hy/68wE2wakNfWtoP85L/BpQLDKbnYDyswIJ1LUeJhSt7LsCHFwtvACKBYI5D 9Gmbvfi8Xf4iCHKdL9xj6Sol4mnv4kH6v5ExXy6PExC1KMnj9Pm6JsvYqXmQetrbUw85 dVS5mfkIlNB+UYWfeOZZkALGHjspdRalVuYgSTwrZKRe0fPK3wbNNvknXgTeD6+aJUOy RzxQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=IZ/ybrQMSLoGamzR3ER5S8lsGGrYKsbtPQsucYnFdD8=; fh=62l4Cw9qWcpYrK61xUehDvMiFjGrQXJUwd3X3H+4A88=; b=ZpHps7CO7CHf5Sppk6lbN4Ov1Jjbd8ipoFq4F2iYjgiMTXSPW5lEurauZA4qPKvhuv rAqM05yQytC2PCgd7OOcZ9iF5TaVuEJCCk9wa3/s9PtckBOoNqhTt4e2Sa3FZTubLnve /xlgd219w0Z9CsXim0OGm6R+1xoe1KeEkxRbWo7poXaEqxfaO3ZhX+m8fAJvCarrFzYW rhKxsvVPH5YhrbYJlSGZSoTvjukZiiPy8pW6+4nex/ovO4ZuM9u9WEwis//GXnPOEIA1 Jkc3hNQErucFsadf+Gt92wqSfVLCJA0YZ3q2KpUXpwr1pIAwpsyJzuHdDxdTW1uN2oAj yFYQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id t18-20020a170902e85200b001b9ea592e75si637147plg.265.2023.07.29.03.04.33; Sat, 29 Jul 2023 03:04:55 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231772AbjG2Jno (ORCPT + 99 others); Sat, 29 Jul 2023 05:43:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50420 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231148AbjG2Jnk (ORCPT ); Sat, 29 Jul 2023 05:43:40 -0400 Received: from dggsgout11.his.huawei.com (dggsgout11.his.huawei.com [45.249.212.51]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 93B162727 for ; Sat, 29 Jul 2023 02:43:38 -0700 (PDT) Received: from mail02.huawei.com (unknown [172.30.67.153]) by dggsgout11.his.huawei.com (SkyGuard) with ESMTP id 4RCfhP6JR6z4f3p0y for ; Sat, 29 Jul 2023 17:43:33 +0800 (CST) Received: from huaweicloud.com (unknown [10.175.124.27]) by APP1 (Coremail) with SMTP id cCh0CgDHMy5D38RkAQcpOQ--.25635S3; Sat, 29 Jul 2023 17:43:34 +0800 (CST) From: Kemeng Shi To: linux-mm@kvack.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org Cc: baolin.wang@linux.alibaba.com, mgorman@techsingularity.net, willy@infradead.org, david@redhat.com, shikemeng@huaweicloud.com Subject: [PATCH 1/5] mm/compaction: allow blockpfn outside of pageblock for high order buddy page Date: Sun, 30 Jul 2023 01:43:50 +0800 Message-Id: <20230729174354.2239980-2-shikemeng@huaweicloud.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20230729174354.2239980-1-shikemeng@huaweicloud.com> References: <20230729174354.2239980-1-shikemeng@huaweicloud.com> MIME-Version: 1.0 X-CM-TRANSID: cCh0CgDHMy5D38RkAQcpOQ--.25635S3 X-Coremail-Antispam: 1UD129KBjvJXoW7KrW8Ar1kWryDXFyfWryxAFb_yoW8KryDpa 47K3W29F1DuayYvFy7JrWkuF1jkwsYyF4jyrW5Kr18ZF9Yg3Z2k3sFyr1jvryjqr1fAFZ0 vr4DKFZ7CrsrZa7anT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUBE14x267AKxVW5JVWrJwAFc2x0x2IEx4CE42xK8VAvwI8IcIk0 rVWrJVCq3wAFIxvE14AKwVWUJVWUGwA2jI8I6cxK62vIxIIY0VWUZVW8XwA2048vs2IY02 0E87I2jVAFwI0_Jr4l82xGYIkIc2x26xkF7I0E14v26r1I6r4UM28lY4IEw2IIxxk0rwA2 F7IY1VAKz4vEj48ve4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_Ar0_tr1l84ACjcxK6xIIjx v20xvEc7CjxVAFwI0_Gr1j6F4UJwA2z4x0Y4vEx4A2jsIE14v26rxl6s0DM28EF7xvwVC2 z280aVCY1x0267AKxVW0oVCq3wAS0I0E0xvYzxvE52x082IY62kv0487Mc02F40EFcxC0V AKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWUJVWUGwAv7VC2z280aVAFwI0_Jr0_Gr1l Ox8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcxkI7VAKI48JM4x0x7Aq67IIx4CEVc8vx2IErc IFxwCF04k20xvY0x0EwIxGrwCFx2IqxVCFs4IE7xkEbVWUJVW8JwC20s026c02F40E14v2 6r1j6r18MI8I3I0E7480Y4vE14v26r106r1rMI8E67AF67kF1VAFwI0_Jw0_GFylIxkGc2 Ij64vIr41lIxAIcVC0I7IYx2IY67AKxVWUJVWUCwCI42IY6xIIjxv20xvEc7CjxVAFwI0_ Gr0_Cr1lIxAIcVCF04k26cxKx2IYs7xG6r1j6r1xMIIF0xvEx4A2jsIE14v26r1j6r4UMI IF0xvEx4A2jsIEc7CjxVAFwI0_Gr0_Gr1UYxBIdaVFxhVjvjDU0xZFpf9x0pRxhLUUUUUU = X-CM-SenderInfo: 5vklyvpphqwq5kxd4v5lfo033gof0z/ X-CFilter-Loop: Reflected X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00,DATE_IN_FUTURE_06_12, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772748900564215015 X-GMAIL-MSGID: 1772748900564215015 Commit 9fcd6d2e052ee ("mm, compaction: skip compound pages by order in free scanner") skiped compound pages to save iterations and limit blockpfn to reach outside of page block in case of bogus compound_order. While this also limit pfnblock outside page block in case a buddy page with order higher than page block is found. After this, isolate_freepages_range will fail unexpectedly as it will fail to isolate the page block which was isolated successfully by high order buddy page in previous page block and abort the free page isolation. Fix this by allow blockpfn outside of pageblock in case of high order buddy page. Fixes: 9fcd6d2e052ee ("mm, compaction: skip compound pages by order in free scanner") Signed-off-by: Kemeng Shi --- mm/compaction.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/mm/compaction.c b/mm/compaction.c index 6841c0496223..d1d28d57e5bd 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -681,8 +681,10 @@ static unsigned long isolate_freepages_block(struct compact_control *cc, /* * There is a tiny chance that we have read bogus compound_order(), * so be careful to not go outside of the pageblock. + * Allow blockpfn outside pageblock in normal case that we isolate + * buddy page with order more than pageblock order. */ - if (unlikely(blockpfn > end_pfn)) + if (unlikely(blockpfn > end_pfn) && total_isolated <= pageblock_nr_pages) blockpfn = end_pfn; trace_mm_compaction_isolate_freepages(*start_pfn, blockpfn, @@ -1418,7 +1420,7 @@ fast_isolate_around(struct compact_control *cc, unsigned long pfn) isolate_freepages_block(cc, &start_pfn, end_pfn, &cc->freepages, 1, false); /* Skip this pageblock in the future as it's full or nearly full */ - if (start_pfn == end_pfn && !cc->no_set_skip_hint) + if (start_pfn >= end_pfn && !cc->no_set_skip_hint) set_pageblock_skip(page); } @@ -1687,7 +1689,7 @@ static void isolate_freepages(struct compact_control *cc) block_end_pfn, freelist, stride, false); /* Update the skip hint if the full pageblock was scanned */ - if (isolate_start_pfn == block_end_pfn) + if (isolate_start_pfn >= block_end_pfn) update_pageblock_skip(cc, page, block_start_pfn); /* Are enough freepages isolated? */ From patchwork Sat Jul 29 17:43:51 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kemeng Shi X-Patchwork-Id: 128091 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp945895vqg; Sat, 29 Jul 2023 03:05:09 -0700 (PDT) X-Google-Smtp-Source: APBJJlH/aJY6GePLTUMB/kMnXmlbfvMKAlXmG8hlMRyBevQUZT3XqeZ30enx6x0w4O2RSKMDueNp X-Received: by 2002:a05:6a20:9155:b0:122:ff52:7331 with SMTP id x21-20020a056a20915500b00122ff527331mr4267377pzc.52.1690625109114; Sat, 29 Jul 2023 03:05:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690625109; cv=none; d=google.com; s=arc-20160816; b=S+HMK/qRE6qusr3IwBPz5AyD8AByHqDBCdbKpUdB+ZhqFWb2YIPCh+lG6W51ntPRLa CyGCxfC0SMRi/laYREpgN0P7GUxHZtlfu4dLWgiSCiFTW9EaplWEgaHWrTcMHpmnh9S/ L2hbATQGTG+Wjfop5HYydPObyrJbQLXj7t0DGPmYS1KW8vQVT4/+sGfWP0E/peqpDwsN KtBtk8Ib2itD3MsuOoi9OfHhNwZnfigZ4Esj5Wblk7zUWuVqq/OV/YyUVDwfZfan9ruZ J96KS2YY6gdjLLAwIUyPYP8gocWZT+x9qBbZtgo3X0e8Dz0jD/d3AsMCKroWZ6EIuFUN NJ5A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=Z1HTIYk7VPg3eMqbi2GpDAqyLJa9v08gSasDici92kw=; fh=62l4Cw9qWcpYrK61xUehDvMiFjGrQXJUwd3X3H+4A88=; b=aH/3R3sfXcUFOTCyoTnRLjkHyl8En26d4wbyrFF8dWLm9A9GZbHiRaJdWJ7Nbh0Cjc VtZu59jRtCTHXzVlU8CYQ1PSKMHiILtlDhGQh2TXvMRg6VODZTjj5nzzdN8DzlSBSpLZ qyENxDW6htRMz1SdsekZJ/S5hni55Ugl5PIa2Fp59wyxf7QH8CJKOQKHIWT1NE4M6aob L9pZAuscMEV8M9IZYnaLgZp5g4ibeHDidh+diRcfNZkVsMBBVJMywHs6HT7tHR9n67W2 ZoDeiSLNrIU6/k0fwNIa+yhikmHhzYxvZlTVXdQH7MmxioOwpSI8LSNI8+s+sXt09a1O BHqw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id bv7-20020a056a00414700b00686d854cb0dsi1303200pfb.23.2023.07.29.03.04.48; Sat, 29 Jul 2023 03:05:09 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231781AbjG2Jnq (ORCPT + 99 others); Sat, 29 Jul 2023 05:43:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50422 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229997AbjG2Jnk (ORCPT ); Sat, 29 Jul 2023 05:43:40 -0400 Received: from dggsgout12.his.huawei.com (dggsgout12.his.huawei.com [45.249.212.56]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1F1E52728 for ; Sat, 29 Jul 2023 02:43:39 -0700 (PDT) Received: from mail02.huawei.com (unknown [172.30.67.153]) by dggsgout12.his.huawei.com (SkyGuard) with ESMTP id 4RCfhN4Fwsz4f3nwR for ; Sat, 29 Jul 2023 17:43:32 +0800 (CST) Received: from huaweicloud.com (unknown [10.175.124.27]) by APP1 (Coremail) with SMTP id cCh0CgDHMy5D38RkAQcpOQ--.25635S4; Sat, 29 Jul 2023 17:43:35 +0800 (CST) From: Kemeng Shi To: linux-mm@kvack.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org Cc: baolin.wang@linux.alibaba.com, mgorman@techsingularity.net, willy@infradead.org, david@redhat.com, shikemeng@huaweicloud.com Subject: [PATCH 2/5] mm/compaction: set compact_cached_free_pfn correctly in update_pageblock_skip Date: Sun, 30 Jul 2023 01:43:51 +0800 Message-Id: <20230729174354.2239980-3-shikemeng@huaweicloud.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20230729174354.2239980-1-shikemeng@huaweicloud.com> References: <20230729174354.2239980-1-shikemeng@huaweicloud.com> MIME-Version: 1.0 X-CM-TRANSID: cCh0CgDHMy5D38RkAQcpOQ--.25635S4 X-Coremail-Antispam: 1UD129KBjvdXoW7Jw1kCr17WFW3KFWxWrykXwb_yoWfGrb_GF 1IyFn8K3W2v39xZ3ZrJw4Syry7Kr48Crn5Ww15Jry7Z3yqvF4kXanrJwnxXrn8XFZrurnx Gw1kuF18Gr4a9jkaLaAFLSUrUUUUUb8apTn2vfkv8UJUUUU8Yxn0WfASr-VFAUDa7-sFnT 9fnUUIcSsGvfJTRUUUbqAYFVCjjxCrM7AC8VAFwI0_Wr0E3s1l1xkIjI8I6I8E6xAIw20E Y4v20xvaj40_Wr0E3s1l1IIY67AEw4v_Jr0_Jr4l87I20VAvwVAaII0Ic2I_JFv_Gryl82 xGYIkIc2x26280x7IE14v26r15M28IrcIa0xkI8VCY1x0267AKxVW8JVW5JwA2ocxC64kI II0Yj41l84x0c7CEw4AK67xGY2AK021l84ACjcxK6xIIjxv20xvE14v26F1j6w1UM28EF7 xvwVC0I7IYx2IY6xkF7I0E14v26r4UJVWxJr1l84ACjcxK6I8E87Iv67AKxVW0oVCq3wA2 z4x0Y4vEx4A2jsIEc7CjxVAFwI0_GcCE3s1le2I262IYc4CY6c8Ij28IcVAaY2xG8wAqx4 xG64xvF2IEw4CE5I8CrVC2j2WlYx0E2Ix0cI8IcVAFwI0_Jr0_Jr4lYx0Ex4A2jsIE14v2 6r1j6r4UMcvjeVCFs4IE7xkEbVWUJVW8JwACjcxG0xvY0x0EwIxGrwACjI8F5VA0II8E6I AqYI8I648v4I1l42xK82IYc2Ij64vIr41l4I8I3I0E4IkC6x0Yz7v_Jr0_Gr1lx2IqxVAq x4xG67AKxVWUJVWUGwC20s026x8GjcxK67AKxVWUGVWUWwC2zVAF1VAY17CE14v26r1q6r 43MIIYrxkI7VAKI48JMIIF0xvE2Ix0cI8IcVAFwI0_Jr0_JF4lIxAIcVC0I7IYx2IY6xkF 7I0E14v26r4j6F4UMIIF0xvE42xK8VAvwI8IcIk0rVWUJVWUCwCI42IY6I8E87Iv67AKxV WUJVW8JwCI42IY6I8E87Iv6xkF7I0E14v26r4j6r4UJbIYCTnIWIevJa73UjIFyTuYvjTR QNVDUUUUU X-CM-SenderInfo: 5vklyvpphqwq5kxd4v5lfo033gof0z/ X-CFilter-Loop: Reflected X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00,DATE_IN_FUTURE_06_12, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772748914385250835 X-GMAIL-MSGID: 1772748914385250835 We will set skip to page block of block_start_pfn, it's more reasonable to set compact_cached_free_pfn to page block before the block_start_pfn. Signed-off-by: Kemeng Shi Reviewed-by: Baolin Wang --- mm/compaction.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/mm/compaction.c b/mm/compaction.c index d1d28d57e5bd..4a784872565a 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -1690,7 +1690,8 @@ static void isolate_freepages(struct compact_control *cc) /* Update the skip hint if the full pageblock was scanned */ if (isolate_start_pfn >= block_end_pfn) - update_pageblock_skip(cc, page, block_start_pfn); + update_pageblock_skip(cc, page, block_start_pfn - + pageblock_nr_pages); /* Are enough freepages isolated? */ if (cc->nr_freepages >= cc->nr_migratepages) { From patchwork Sat Jul 29 17:43:52 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kemeng Shi X-Patchwork-Id: 128106 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp973985vqg; Sat, 29 Jul 2023 04:25:42 -0700 (PDT) X-Google-Smtp-Source: APBJJlEp6NNot/Ns23CRSMY+OccZPGhBqEMCpG7QEn/5opEveOpJQXd8ta9IJYMj0+KRuSJcCl37 X-Received: by 2002:a17:90a:13c9:b0:267:8012:b393 with SMTP id s9-20020a17090a13c900b002678012b393mr4257664pjf.34.1690629941635; Sat, 29 Jul 2023 04:25:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690629941; cv=none; d=google.com; s=arc-20160816; b=FY3xcn2455dna4s+XqBAqQEERwOU/gVYqfvQkjuKJpfcMvROwZPjVc1BZR3AU5/vDU p40IqbiJKy4nsJXWQSkE/Uva20DVrTaTnmt79T+fw44Z/Z7upEoZfqR8XxezP2WVlZPh hUZJUbp+MnSWvMXsFzgyLAfoaPJgsubxvoO2FGQJW4XNl/EzHFWfupNHkAqG1SrgiP0p wFrulPnB6HVwkvCYGbVIeTCpeVBVnVcoUY/vckaMoDwVnZReU5nTHkoSh3IhEeUHHHCN 8e8XNxyLtc1MwMUMmQ/glflaeNaPy51QU/NxsXr7SLSjLMCnnpjy3T8qFjv6M8RVruHR jn6Q== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=GY7ptctu2PcfFka/7M4exz/N4HZ/kFVT+wXlc0jUhmU=; fh=62l4Cw9qWcpYrK61xUehDvMiFjGrQXJUwd3X3H+4A88=; b=aO5g5pd0v1C4ZBywo0iHqKJb085TW5JbL/l2w1fQEb8wJslGIA210idofKac8oNETR TP3VIjBQ2Ry+MvzQC+u8uxuoO2W9XrZf+udImoLI2w67JBU9fHf7OzeEY4eHPZNG1a6q SKRaI5MMDuQvbEWgJLF1vW0hujOPKhKHGjb30oVexJdz4TLBoLpf36GctW9Cmh7Lr+ew h+0HsEU0MjX2MCbsK9p/bzScd/IY3q3VG5t4VavuUhgF7k91+3Zz1IKvKA5Lb98SOE0E b90fswDnDWxbg0e82nNSPSRmXdybitGaguTlhlqrDGukRlExFBHKQWVDzY3Nci+HKPJQ SxCA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id lh14-20020a170903290e00b001b8b48a5e81si4359442plb.314.2023.07.29.04.25.28; Sat, 29 Jul 2023 04:25:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231755AbjG2Jnm (ORCPT + 99 others); Sat, 29 Jul 2023 05:43:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50410 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229483AbjG2Jnj (ORCPT ); Sat, 29 Jul 2023 05:43:39 -0400 Received: from dggsgout11.his.huawei.com (dggsgout11.his.huawei.com [45.249.212.51]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 676922701 for ; Sat, 29 Jul 2023 02:43:38 -0700 (PDT) Received: from mail02.huawei.com (unknown [172.30.67.153]) by dggsgout11.his.huawei.com (SkyGuard) with ESMTP id 4RCfhR0xxwz4f403y for ; Sat, 29 Jul 2023 17:43:35 +0800 (CST) Received: from huaweicloud.com (unknown [10.175.124.27]) by APP1 (Coremail) with SMTP id cCh0CgDHMy5D38RkAQcpOQ--.25635S5; Sat, 29 Jul 2023 17:43:35 +0800 (CST) From: Kemeng Shi To: linux-mm@kvack.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org Cc: baolin.wang@linux.alibaba.com, mgorman@techsingularity.net, willy@infradead.org, david@redhat.com, shikemeng@huaweicloud.com Subject: [PATCH 3/5] mm/compaction: merge end_pfn boundary check in isolate_freepages_range Date: Sun, 30 Jul 2023 01:43:52 +0800 Message-Id: <20230729174354.2239980-4-shikemeng@huaweicloud.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20230729174354.2239980-1-shikemeng@huaweicloud.com> References: <20230729174354.2239980-1-shikemeng@huaweicloud.com> MIME-Version: 1.0 X-CM-TRANSID: cCh0CgDHMy5D38RkAQcpOQ--.25635S5 X-Coremail-Antispam: 1UD129KBjvJXoW7JF4kWr1rWF4kWryDJF4ruFg_yoW8Jrykp3 47GF1xWr1kuayYvFy7X3s8Zr15Cw4kKF4UArWUGF1rAFyFkF97K3sayryDZFyjqw1xZrZ0 vrsFkFyxt3WUZa7anT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUBE14x267AKxVWrJVCq3wAFc2x0x2IEx4CE42xK8VAvwI8IcIk0 rVWrJVCq3wAFIxvE14AKwVWUJVWUGwA2jI8I6cxK62vIxIIY0VWUZVW8XwA2048vs2IY02 0E87I2jVAFwI0_JrWl82xGYIkIc2x26xkF7I0E14v26ryj6s0DM28lY4IEw2IIxxk0rwA2 F7IY1VAKz4vEj48ve4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_Ar0_tr1l84ACjcxK6xIIjx v20xvEc7CjxVAFwI0_Gr1j6F4UJwA2z4x0Y4vEx4A2jsIE14v26rxl6s0DM28EF7xvwVC2 z280aVCY1x0267AKxVW0oVCq3wAS0I0E0xvYzxvE52x082IY62kv0487Mc02F40EFcxC0V AKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWUJVWUGwAv7VC2z280aVAFwI0_Jr0_Gr1l Ox8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcxkI7VAKI48JM4x0x7Aq67IIx4CEVc8vx2IErc IFxwCF04k20xvY0x0EwIxGrwCFx2IqxVCFs4IE7xkEbVWUJVW8JwC20s026c02F40E14v2 6r1j6r18MI8I3I0E7480Y4vE14v26r106r1rMI8E67AF67kF1VAFwI0_Jw0_GFylIxkGc2 Ij64vIr41lIxAIcVC0I7IYx2IY67AKxVWUJVWUCwCI42IY6xIIjxv20xvEc7CjxVAFwI0_ Gr0_Cr1lIxAIcVCF04k26cxKx2IYs7xG6r1j6r1xMIIF0xvEx4A2jsIE14v26r1j6r4UMI IF0xvEx4A2jsIEc7CjxVAFwI0_Gr0_Gr1UYxBIdaVFxhVjvjDU0xZFpf9x0pRPEf5UUUUU = X-CM-SenderInfo: 5vklyvpphqwq5kxd4v5lfo033gof0z/ X-CFilter-Loop: Reflected X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00,DATE_IN_FUTURE_06_12, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772753981321208862 X-GMAIL-MSGID: 1772753981321208862 From: Kemeng Shi Merge the end_pfn boundary check for single page block forward and multiple page blocks forward to avoid do twice boundary check for multiple page blocks forward. Signed-off-by: Kemeng Shi Reviewed-by: Baolin Wang --- mm/compaction.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/mm/compaction.c b/mm/compaction.c index 4a784872565a..65791a74c5e8 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -740,8 +740,6 @@ isolate_freepages_range(struct compact_control *cc, /* Protect pfn from changing by isolate_freepages_block */ unsigned long isolate_start_pfn = pfn; - block_end_pfn = min(block_end_pfn, end_pfn); - /* * pfn could pass the block_end_pfn if isolated freepage * is more than pageblock order. In this case, we adjust @@ -750,9 +748,10 @@ isolate_freepages_range(struct compact_control *cc, if (pfn >= block_end_pfn) { block_start_pfn = pageblock_start_pfn(pfn); block_end_pfn = pageblock_end_pfn(pfn); - block_end_pfn = min(block_end_pfn, end_pfn); } + block_end_pfn = min(block_end_pfn, end_pfn); + if (!pageblock_pfn_to_page(block_start_pfn, block_end_pfn, cc->zone)) break; From patchwork Sat Jul 29 17:43:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kemeng Shi X-Patchwork-Id: 128090 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp945832vqg; Sat, 29 Jul 2023 03:05:01 -0700 (PDT) X-Google-Smtp-Source: APBJJlExVy+qdlViGmInEJO15pHJRkJ5A17fYU1ONkpI+RRGjXgdWZyP8GnNr99lvxfgEoOdGwil X-Received: by 2002:a05:6a20:96d2:b0:138:1c5b:274f with SMTP id hq18-20020a056a2096d200b001381c5b274fmr3522485pzc.45.1690625100860; Sat, 29 Jul 2023 03:05:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690625100; cv=none; d=google.com; s=arc-20160816; b=OUC7FvmLq8ItNroK4rqbtwnyWpZjI9qQ2dsgSdMptickzi9uCqtu8inU9DFWYCqQCE DRk07cVj/ZMApxXTrK56plWhwjGR6qxilMRWhpwRdIC0ulK952JTBvOzfmtPpHOk7SWL Oq9DnVa+Fn4ufp0ZmIghhaCdMMwHLAq0WrfmIVEM1fX/IHlbjaVjOCGwcm1eGRXUgW4r UAJ2tDO3sAhn3WjTrVZXrSfztzoumeAxaUz73HhtSbTS5KdJ86kfj1x0zuY4Gwn94WDz lsVyTiXh4vdcm/beIi5N1pfbKq7m8rwurudj9Bm0ugxFGixmrrJZtDLW97khv7n0I1Ck xv+w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=9meaUYhEJFtb8FcCU7b6xQzuXYdX0srDSO6ev7eCARE=; fh=62l4Cw9qWcpYrK61xUehDvMiFjGrQXJUwd3X3H+4A88=; b=js0IZaGNSisN4SInWqeCk8F+4hUpSGwKnm4HC4IF3WhEeciWj9bsm8jewSaLHkq/kB a5KwvkOlpa5xSTMy5zCWYWfx/1nzSkTyck4173TBPjHEGeE63nhAHqHHtmU130qXFrDo apsEVkvJvFRepRaMkwKl4RZpCDTBcK2d+XU4diBsUAOFVsjtOmkiGViIZxZyBFJg27zR 08U4GcHJcBh2M/KwN+zYbmomVVses5lUbV+V79Qj4gCbj66dXmP30FLZA68JI3EOPRlX StHTYdt0nTRjX9b7jyN9looAIIWjAKb7LSnxIHvWso6GC9GplRwNbkqtejdOB9CZPH4d lUgw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id z13-20020a056a001d8d00b00686f001cb5esi4421086pfw.319.2023.07.29.03.04.40; Sat, 29 Jul 2023 03:05:00 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231859AbjG2Jnv (ORCPT + 99 others); Sat, 29 Jul 2023 05:43:51 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50424 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231684AbjG2Jnk (ORCPT ); Sat, 29 Jul 2023 05:43:40 -0400 Received: from dggsgout12.his.huawei.com (dggsgout12.his.huawei.com [45.249.212.56]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6CCB535A3 for ; Sat, 29 Jul 2023 02:43:39 -0700 (PDT) Received: from mail02.huawei.com (unknown [172.30.67.153]) by dggsgout12.his.huawei.com (SkyGuard) with ESMTP id 4RCfhP2P7cz4f3nwX for ; Sat, 29 Jul 2023 17:43:33 +0800 (CST) Received: from huaweicloud.com (unknown [10.175.124.27]) by APP1 (Coremail) with SMTP id cCh0CgDHMy5D38RkAQcpOQ--.25635S6; Sat, 29 Jul 2023 17:43:35 +0800 (CST) From: Kemeng Shi To: linux-mm@kvack.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org Cc: baolin.wang@linux.alibaba.com, mgorman@techsingularity.net, willy@infradead.org, david@redhat.com, shikemeng@huaweicloud.com Subject: [PATCH 4/5] mm/compaction: remove unnecessary cursor page in isolate_freepages_block Date: Sun, 30 Jul 2023 01:43:53 +0800 Message-Id: <20230729174354.2239980-5-shikemeng@huaweicloud.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20230729174354.2239980-1-shikemeng@huaweicloud.com> References: <20230729174354.2239980-1-shikemeng@huaweicloud.com> MIME-Version: 1.0 X-CM-TRANSID: cCh0CgDHMy5D38RkAQcpOQ--.25635S6 X-Coremail-Antispam: 1UD129KBjvJXoW7WrWxAFW3WF17CFWkXF1UKFg_yoW8Ww47pF yjkw1IyF40g345uFy7t3ykCr17Grs8KanrCr4fJrWrZFyftFyxZry2yr1UCrW8XrZrurZa 93yDKF97Kan5JaUanT9S1TB71UUUUUUqnTZGkaVYY2UrUUUUjbIjqfuFe4nvWSU5nxnvy2 9KBjDU0xBIdaVrnRJUUUPY14x267AKxVWrJVCq3wAFc2x0x2IEx4CE42xK8VAvwI8IcIk0 rVWrJVCq3wAFIxvE14AKwVWUJVWUGwA2jI8I6cxK62vIxIIY0VWUZVW8XwA2048vs2IY02 0E87I2jVAFwI0_JF0E3s1l82xGYIkIc2x26xkF7I0E14v26ryj6s0DM28lY4IEw2IIxxk0 rwA2F7IY1VAKz4vEj48ve4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_Ar0_tr1l84ACjcxK6x IIjxv20xvEc7CjxVAFwI0_Gr1j6F4UJwA2z4x0Y4vEx4A2jsIE14v26rxl6s0DM28EF7xv wVC2z280aVCY1x0267AKxVW0oVCq3wAS0I0E0xvYzxvE52x082IY62kv0487Mc02F40EFc xC0VAKzVAqx4xG6I80ewAv7VC0I7IYx2IY67AKxVWUJVWUGwAv7VC2z280aVAFwI0_Jr0_ Gr1lOx8S6xCaFVCjc4AY6r1j6r4UM4x0Y48IcxkI7VAKI48JM4x0x7Aq67IIx4CEVc8vx2 IErcIFxwCF04k20xvY0x0EwIxGrwCFx2IqxVCFs4IE7xkEbVWUJVW8JwC20s026c02F40E 14v26r1j6r18MI8I3I0E7480Y4vE14v26r106r1rMI8E67AF67kF1VAFwI0_Jw0_GFylIx kGc2Ij64vIr41lIxAIcVC0I7IYx2IY67AKxVWUJVWUCwCI42IY6xIIjxv20xvEc7CjxVAF wI0_Gr0_Cr1lIxAIcVCF04k26cxKx2IYs7xG6r1j6r1xMIIF0xvEx4A2jsIE14v26r1j6r 4UMIIF0xvEx4A2jsIEc7CjxVAFwI0_Gr0_Gr1UYxBIdaVFxhVjvjDU0xZFpf9x0pRvJPtU UUUU= X-CM-SenderInfo: 5vklyvpphqwq5kxd4v5lfo033gof0z/ X-CFilter-Loop: Reflected X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00,DATE_IN_FUTURE_06_12, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772748906136039953 X-GMAIL-MSGID: 1772748906136039953 The cursor is only used for page forward currently. We can simply move page forward directly to remove unnecessary cursor. Signed-off-by: Kemeng Shi Reviewed-by: Baolin Wang --- mm/compaction.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/mm/compaction.c b/mm/compaction.c index 65791a74c5e8..cfb661f4ce23 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -589,7 +589,7 @@ static unsigned long isolate_freepages_block(struct compact_control *cc, bool strict) { int nr_scanned = 0, total_isolated = 0; - struct page *cursor; + struct page *page; unsigned long flags = 0; spinlock_t *locked = NULL; unsigned long blockpfn = *start_pfn; @@ -599,12 +599,11 @@ static unsigned long isolate_freepages_block(struct compact_control *cc, if (strict) stride = 1; - cursor = pfn_to_page(blockpfn); + page = pfn_to_page(blockpfn); /* Isolate free pages. */ - for (; blockpfn < end_pfn; blockpfn += stride, cursor += stride) { + for (; blockpfn < end_pfn; blockpfn += stride, page += stride) { int isolated; - struct page *page = cursor; /* * Periodically drop the lock (if held) regardless of its @@ -628,7 +627,7 @@ static unsigned long isolate_freepages_block(struct compact_control *cc, if (likely(order <= MAX_ORDER)) { blockpfn += (1UL << order) - 1; - cursor += (1UL << order) - 1; + page += (1UL << order) - 1; nr_scanned += (1UL << order) - 1; } goto isolate_fail; @@ -665,7 +664,7 @@ static unsigned long isolate_freepages_block(struct compact_control *cc, } /* Advance to the end of split page */ blockpfn += isolated - 1; - cursor += isolated - 1; + page += isolated - 1; continue; isolate_fail: From patchwork Sat Jul 29 17:43:54 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kemeng Shi X-Patchwork-Id: 128096 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:918b:0:b0:3e4:2afc:c1 with SMTP id s11csp952985vqg; Sat, 29 Jul 2023 03:25:04 -0700 (PDT) X-Google-Smtp-Source: APBJJlE3EwXAvDHV+pKSvyu4AO+GyoS7TDsYxurZw4/kHJn4WS6C78tvuEwNy4qt7PJhh6j8HYcb X-Received: by 2002:a17:907:7895:b0:99b:ef56:935 with SMTP id ku21-20020a170907789500b0099bef560935mr1987625ejc.65.1690626304558; Sat, 29 Jul 2023 03:25:04 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1690626304; cv=none; d=google.com; s=arc-20160816; b=exK9IYwnK7JJt/YPJwYWgm4wlgJJsTIcEFzmgZdcL10r3SzDyQzJqqTdxwSXdC4zRy xVgvPBcr/usYuaeJqybiS/eUKN+ydLCRG7Xn2wXuMu1S3geJbsk69Aq5Pn7M5Xyu1zCU lGA0r+XYzI+T5GyR0l/tcmKdEgWxT0pZvNzZwjS4YcUd0SchpqrOfkiONiDOfTgk8zIW 6J1f6eaHCiuEBgHmSfgwIQTDabmE0G5sq93cFAdhRtnm6PHdmuilwd/mPb21SD28LmX0 EF7iJNt14kRZ9aVBfMpec5TNwLePncoqN6KseTJ/vSJBtcAiWGvsZTDXAq3O2lS0gz84 +hTw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=bkhKZF0Hkri+kaq1HSuOKHk5DQCqZxLUUZ1GDcCrDQQ=; fh=62l4Cw9qWcpYrK61xUehDvMiFjGrQXJUwd3X3H+4A88=; b=QZhAwPMDKFg9O2hCJoGRZB8Pmho/G7hIiVIppq1mSHS4YY8+FuwtwXhDwuJpjOSoEe U3JN12SWTXxMuN+BdI1ssUF80VHrn0bPcNFMPZSFwOryBvdaIELBrqPo5JxQs5JxAZ+1 mJlSmX3JTHS3JODXXJh2ifMeOTY2h8TizSAIu+in2s2gun9MwM+3Ju0DnY1MzRXmAXbK i0J+AIat0DYx71mIoZdOnMhl3lD1e4p3j/vckarMUHyCBe9PHWnXxbVLk5AlxAR5kcwg G9+JFaP2dm7hDJAOPQadedunnYxJ7haxN+5sJ7R9wbJ+WUq/9FDIXvYNpyjiIzFZIdwo S3lA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id i9-20020a170906a28900b0098d52a2bc16si4407633ejz.438.2023.07.29.03.24.40; Sat, 29 Jul 2023 03:25:04 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231791AbjG2Jns (ORCPT + 99 others); Sat, 29 Jul 2023 05:43:48 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50426 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231686AbjG2Jnk (ORCPT ); Sat, 29 Jul 2023 05:43:40 -0400 Received: from dggsgout11.his.huawei.com (dggsgout11.his.huawei.com [45.249.212.51]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8A22C35A8 for ; Sat, 29 Jul 2023 02:43:39 -0700 (PDT) Received: from mail02.huawei.com (unknown [172.30.67.153]) by dggsgout11.his.huawei.com (SkyGuard) with ESMTP id 4RCfhR2rb2z4f3pJP for ; Sat, 29 Jul 2023 17:43:35 +0800 (CST) Received: from huaweicloud.com (unknown [10.175.124.27]) by APP1 (Coremail) with SMTP id cCh0CgDHMy5D38RkAQcpOQ--.25635S7; Sat, 29 Jul 2023 17:43:36 +0800 (CST) From: Kemeng Shi To: linux-mm@kvack.org, linux-kernel@vger.kernel.org, akpm@linux-foundation.org Cc: baolin.wang@linux.alibaba.com, mgorman@techsingularity.net, willy@infradead.org, david@redhat.com, shikemeng@huaweicloud.com Subject: [PATCH 5/5] mm/compaction: remove unnecessary "else continue" at end of loop in isolate_freepages_block Date: Sun, 30 Jul 2023 01:43:54 +0800 Message-Id: <20230729174354.2239980-6-shikemeng@huaweicloud.com> X-Mailer: git-send-email 2.30.0 In-Reply-To: <20230729174354.2239980-1-shikemeng@huaweicloud.com> References: <20230729174354.2239980-1-shikemeng@huaweicloud.com> MIME-Version: 1.0 X-CM-TRANSID: cCh0CgDHMy5D38RkAQcpOQ--.25635S7 X-Coremail-Antispam: 1UD129KBjDUn29KB7ZKAUJUUUUU529EdanIXcx71UUUUU7v73 VFW2AGmfu7bjvjm3AaLaJ3UjIYCTnIWjp_UUUOb7AC8VAFwI0_Wr0E3s1l1xkIjI8I6I8E 6xAIw20EY4v20xvaj40_Wr0E3s1l1IIY67AEw4v_Jr0_Jr4l87I20VAvwVAaII0Ic2I_JF v_Gryl82xGYIkIc2x26280x7IE14v26r126s0DM28IrcIa0xkI8VCY1x0267AKxVW5JVCq 3wA2ocxC64kIII0Yj41l84x0c7CEw4AK67xGY2AK021l84ACjcxK6xIIjxv20xvE14v26F 1j6w1UM28EF7xvwVC0I7IYx2IY6xkF7I0E14v26r4UJVWxJr1l84ACjcxK6I8E87Iv67AK xVW0oVCq3wA2z4x0Y4vEx4A2jsIEc7CjxVAFwI0_GcCE3s1le2I262IYc4CY6c8Ij28IcV AaY2xG8wAqx4xG64xvF2IEw4CE5I8CrVC2j2WlYx0E2Ix0cI8IcVAFwI0_Jr0_Jr4lYx0E x4A2jsIE14v26r1j6r4UMcvjeVCFs4IE7xkEbVWUJVW8JwACjcxG0xvY0x0EwIxGrwACjI 8F5VA0II8E6IAqYI8I648v4I1l42xK82IYc2Ij64vIr41l4I8I3I0E4IkC6x0Yz7v_Jr0_ Gr1lx2IqxVAqx4xG67AKxVWUJVWUGwC20s026x8GjcxK67AKxVWUGVWUWwC2zVAF1VAY17 CE14v26r1q6r43MIIYrxkI7VAKI48JMIIF0xvE2Ix0cI8IcVAFwI0_JFI_Gr1lIxAIcVC0 I7IYx2IY6xkF7I0E14v26r4j6F4UMIIF0xvE42xK8VAvwI8IcIk0rVWUJVWUCwCI42IY6I 8E87Iv67AKxVWUJVW8JwCI42IY6I8E87Iv6xkF7I0E14v26r4j6r4UJbIYCTnIWIevJa73 UjIFyTuYvjTRKfOwUUUUU X-CM-SenderInfo: 5vklyvpphqwq5kxd4v5lfo033gof0z/ X-CFilter-Loop: Reflected X-Spam-Status: No, score=0.0 required=5.0 tests=BAYES_00,DATE_IN_FUTURE_06_12, SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1772750168248168412 X-GMAIL-MSGID: 1772750168248168412 There is no behavior change to remove "else continue" code at end of scan loop. Just remove it to make code cleaner. Signed-off-by: Kemeng Shi Reviewed-by: Baolin Wang --- mm/compaction.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/mm/compaction.c b/mm/compaction.c index cfb661f4ce23..d38297018077 100644 --- a/mm/compaction.c +++ b/mm/compaction.c @@ -670,9 +670,6 @@ static unsigned long isolate_freepages_block(struct compact_control *cc, isolate_fail: if (strict) break; - else - continue; - } compact_unlock_irqrestore(&locked, flags);