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 ADE2BC433F5 for ; Wed, 2 Mar 2022 08:41:02 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0F0DD8D0002; Wed, 2 Mar 2022 03:41:02 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 07A8C8D0001; Wed, 2 Mar 2022 03:41:02 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E5CE58D0002; Wed, 2 Mar 2022 03:41:01 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0156.hostedemail.com [216.40.44.156]) by kanga.kvack.org (Postfix) with ESMTP id D48FD8D0001 for ; Wed, 2 Mar 2022 03:41:01 -0500 (EST) Received: from smtpin23.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 8EAE68249980 for ; Wed, 2 Mar 2022 08:41:01 +0000 (UTC) X-FDA: 79198801122.23.B18596C Received: from mail-yw1-f181.google.com (mail-yw1-f181.google.com [209.85.128.181]) by imf04.hostedemail.com (Postfix) with ESMTP id F179C40005 for ; Wed, 2 Mar 2022 08:41:00 +0000 (UTC) Received: by mail-yw1-f181.google.com with SMTP id 00721157ae682-2dbc48104beso8800277b3.5 for ; Wed, 02 Mar 2022 00:41:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=JwMioHCl9Ph8eWOImmniTuBw63TpPfziFubOm/sEXr8=; b=gEaPbwXdXoBPUhBU1qYUb+mWBtCc1FFu9FTQhyaHcYakYRSYnscz5mDrIPWjDqQlBX qcS28j00iaRWfwkxxuzxbhB2C3RZg5BGJsCEjj+Wcn8tyCPhENJ2M1eMXEvzmdduAXNC iS1rrj3qFEcIRpuTMiaqr46KkdiOoVHJcL1VI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=JwMioHCl9Ph8eWOImmniTuBw63TpPfziFubOm/sEXr8=; b=SDJMORfbD+e9HI/YbIR/doOtHTwyRhkqQwSbjzTfZtHb8uah5A0GkdbF/ICg4NiuEy n+9NupM49slb5M01YIBhYXO9N3llxVPS0I94zKyrmCrrkbJdqYOut3uJriGdpC+5g7J1 Bnr9rc/CFgPEaYWX7q00+i1EkF0f4Zb7x/GFq2TTfjC9HzLvOCvS7luV1Bk4yuiU1iFe KByES/9NNhuV5ui0hFjJAB/xHnDMS/8xQd4LfKqTVzT08lgLfc2mvDJmNMwBZujhWARA e8N9RA5V45w6hl7gQJ+QS7NWl4N4cB8ZrzAxyehogH6jEAGv6N1J9yiZKe4EtlEY1xk2 ClSw== X-Gm-Message-State: AOAM533Xf/ZH0m4OrGYpC5ai3g3ATjUs1QgOykbuVsM2vD1EEQ9H6dC0 4mvCHfSZfNG/t9SyWqgq4+VZfWZLYW83mPEy2Vk9yg== X-Google-Smtp-Source: ABdhPJw91B6/5J4Hf19SaEnixGP84dBaoO6UieG1QDiBm8ydxmNwVI3nGXbW986eTX4giyzEyaAa8QwPX746DuUocPg= X-Received: by 2002:a81:846:0:b0:2db:f920:5c62 with SMTP id 67-20020a810846000000b002dbf9205c62mr5093359ywi.489.1646210460323; Wed, 02 Mar 2022 00:41:00 -0800 (PST) MIME-Version: 1.0 References: <20220219092533.12596-1-linmiaohe@huawei.com> <20220219092533.12596-10-linmiaohe@huawei.com> In-Reply-To: <20220219092533.12596-10-linmiaohe@huawei.com> From: Vitaly Wool Date: Wed, 2 Mar 2022 09:40:49 +0100 Message-ID: Subject: Re: [PATCH 9/9] mm/z3fold: remove unneeded return value of z3fold_compact_page() To: Miaohe Lin Cc: Andrew Morton , Linux-MM , LKML Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: F179C40005 X-Stat-Signature: 5j1zcm63tugpm783qmpdwdhy3ntcg488 X-Rspam-User: Authentication-Results: imf04.hostedemail.com; dkim=pass header.d=konsulko.com header.s=google header.b=gEaPbwXd; spf=pass (imf04.hostedemail.com: domain of vitaly.wool@konsulko.com designates 209.85.128.181 as permitted sender) smtp.mailfrom=vitaly.wool@konsulko.com; dmarc=pass (policy=none) header.from=konsulko.com X-Rspamd-Server: rspam07 X-HE-Tag: 1646210460-15693 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 Sat, Feb 19, 2022 at 10:26 AM Miaohe Lin wrote: > > Remove the unneeded return value of z3fold_compact_page() as it's > never checked. It was a sort of hook for gathering extended compaction statistics in future. Do you really gain a lot by removing this? ~Vitaly > Signed-off-by: Miaohe Lin > --- > mm/z3fold.c | 14 +++++--------- > 1 file changed, 5 insertions(+), 9 deletions(-) > > diff --git a/mm/z3fold.c b/mm/z3fold.c > index 83b5a3514427..db41b4227ec7 100644 > --- a/mm/z3fold.c > +++ b/mm/z3fold.c > @@ -746,18 +746,18 @@ static struct z3fold_header *compact_single_buddy(struct z3fold_header *zhdr) > > #define BIG_CHUNK_GAP 3 > /* Has to be called with lock held */ > -static int z3fold_compact_page(struct z3fold_header *zhdr) > +static void z3fold_compact_page(struct z3fold_header *zhdr) > { > struct page *page = virt_to_page(zhdr); > > if (test_bit(MIDDLE_CHUNK_MAPPED, &page->private)) > - return 0; /* can't move middle chunk, it's used */ > + return; /* can't move middle chunk, it's used */ > > if (unlikely(PageIsolated(page))) > - return 0; > + return; > > if (zhdr->middle_chunks == 0) > - return 0; /* nothing to compact */ > + return; /* nothing to compact */ > > if (zhdr->first_chunks == 0 && zhdr->last_chunks == 0) { > /* move to the beginning */ > @@ -766,7 +766,7 @@ static int z3fold_compact_page(struct z3fold_header *zhdr) > zhdr->middle_chunks = 0; > zhdr->start_middle = 0; > zhdr->first_num++; > - return 1; > + return; > } > > /* > @@ -778,7 +778,6 @@ static int z3fold_compact_page(struct z3fold_header *zhdr) > BIG_CHUNK_GAP) { > mchunk_memmove(zhdr, zhdr->first_chunks + ZHDR_CHUNKS); > zhdr->start_middle = zhdr->first_chunks + ZHDR_CHUNKS; > - return 1; > } else if (zhdr->last_chunks != 0 && zhdr->first_chunks == 0 && > TOTAL_CHUNKS - (zhdr->last_chunks + zhdr->start_middle > + zhdr->middle_chunks) >= > @@ -787,10 +786,7 @@ static int z3fold_compact_page(struct z3fold_header *zhdr) > zhdr->middle_chunks; > mchunk_memmove(zhdr, new_start); > zhdr->start_middle = new_start; > - return 1; > } > - > - return 0; > } > > static void do_compact_page(struct z3fold_header *zhdr, bool locked) > -- > 2.23.0 >