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=-10.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham 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 50EF0C433E6 for ; Tue, 1 Sep 2020 18:06:48 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 128732078B for ; Tue, 1 Sep 2020 18:06:47 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="uFvkOFoR" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 128732078B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 862376B00CC; Tue, 1 Sep 2020 14:06:47 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 811556B00CD; Tue, 1 Sep 2020 14:06:47 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 6DA0290001A; Tue, 1 Sep 2020 14:06:47 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0150.hostedemail.com [216.40.44.150]) by kanga.kvack.org (Postfix) with ESMTP id 59D8A6B00CC for ; Tue, 1 Sep 2020 14:06:47 -0400 (EDT) Received: from smtpin26.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id 1D38182499B9 for ; Tue, 1 Sep 2020 18:06:47 +0000 (UTC) X-FDA: 77215273254.26.bait61_36152a22709a Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin26.hostedemail.com (Postfix) with ESMTP id E2FB21804B655 for ; Tue, 1 Sep 2020 18:06:46 +0000 (UTC) X-HE-Tag: bait61_36152a22709a X-Filterd-Recvd-Size: 3773 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by imf11.hostedemail.com (Postfix) with ESMTP for ; Tue, 1 Sep 2020 18:06:46 +0000 (UTC) Received: from mail-lj1-f180.google.com (mail-lj1-f180.google.com [209.85.208.180]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id EBEAB2078B for ; Tue, 1 Sep 2020 18:06:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598983605; bh=2Lfvmfq4ZZ/+WMJSiH8u8cZm0zraIa4vbzwb4dTfe0g=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=uFvkOFoR7ee4CiEr9FYKckMkQuvtj01lK9phDiqWKg63igqVY2m+VnySzKWdLQdRj KIj5gxNLWlU2NoVo21kbPij9slwCdGbc2Ql33UmGdp8SC6gEdYXZ44zsW7MabKWHAw sJzWV6FSKaAfftthbnIYUgx7rpgxUa8s0NYC6KtI= Received: by mail-lj1-f180.google.com with SMTP id v23so2686606ljd.1 for ; Tue, 01 Sep 2020 11:06:44 -0700 (PDT) X-Gm-Message-State: AOAM531gztkqXwQmz5PuWzNqkIn2izsegu0s/YFUmTYVFQodv3uJK1lB Q/7cImRYxyiOlgLFpwd7eJF/mjNNkbNJiwvM/pg= X-Google-Smtp-Source: ABdhPJyY9bc1IKqI/4PFWILW0olpWrITWYBVlcmWHnFJfpVQkBPzscfRB0xW1XKLj6lsc9Uqrpy1b+BNTMFEe14K2wM= X-Received: by 2002:a2e:81c2:: with SMTP id s2mr1257862ljg.10.1598983603083; Tue, 01 Sep 2020 11:06:43 -0700 (PDT) MIME-Version: 1.0 References: <159897769535.405783.17587409235571100774.stgit@warthog.procyon.org.uk> <159897770245.405783.16506873187032379873.stgit@warthog.procyon.org.uk> In-Reply-To: <159897770245.405783.16506873187032379873.stgit@warthog.procyon.org.uk> From: Song Liu Date: Tue, 1 Sep 2020 11:06:31 -0700 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [RFC PATCH 1/7] Fix khugepaged's request size in collapse_file() To: David Howells Cc: Matthew Wilcox , Linux-Fsdevel , Linux-MM , open list Content-Type: text/plain; charset="UTF-8" X-Rspamd-Queue-Id: E2FB21804B655 X-Spamd-Result: default: False [0.00 / 100.00] X-Rspamd-Server: rspam02 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 Tue, Sep 1, 2020 at 9:28 AM David Howells wrote: > > collapse_file() in khugepaged passes PAGE_SIZE as the number of pages to be > read ahead to page_cache_sync_readahead(). It seems this was expressed as a > number of bytes rather than a number of pages. > > Fix it to use the number of pages to the end of the window instead. > > Fixes: 99cb0dbd47a1 ("mm,thp: add read-only THP support for (non-shmem) FS") > Signed-off-by: David Howells > cc: Matthew Wilcox > cc: Song Liu Thanks for the fix! Acked-by: Song Liu > --- > > mm/khugepaged.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/khugepaged.c b/mm/khugepaged.c > index 6d199c353281..f2d243077b74 100644 > --- a/mm/khugepaged.c > +++ b/mm/khugepaged.c > @@ -1706,7 +1706,7 @@ static void collapse_file(struct mm_struct *mm, > xas_unlock_irq(&xas); > page_cache_sync_readahead(mapping, &file->f_ra, > file, index, > - PAGE_SIZE); > + end - index); > /* drain pagevecs to help isolate_lru_page() */ > lru_add_drain(); > page = find_lock_page(mapping, index); > > >