Received: by 2002:a25:4158:0:0:0:0:0 with SMTP id o85csp2843105yba; Fri, 10 May 2019 21:11:10 -0700 (PDT) X-Google-Smtp-Source: APXvYqx8AoF3KkdeZ/pHEMs/TxaUQoXb9Q+yad2Sjk7FgyGb2HmUBv9DWWLkKS5s5pSQ2hBl7Ar2 X-Received: by 2002:a63:1558:: with SMTP id 24mr18350620pgv.126.1557547870192; Fri, 10 May 2019 21:11:10 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1557547870; cv=none; d=google.com; s=arc-20160816; b=e5bsejlWbzRcGb2mWsZLYsTxSpDA/vAv3PQft3N7Lo36aQ0MlM9gqNSaCk6IKQkPeI zkRTAlaOR3DdVNx/t8Hy2F9ul16ev9UGIlQxYcCTGbfRmLGMFBLdQ26Ruyn/XJHRmiwm sKYeQOyfus7CylpZ18S5xTOPgz+bGv/6qI+e1Qw+CU86y0V43FFpIaOYhX/nAzNd6VXq wkx8IlBXRPJfzM5P+mwk04drReY6QXuYsoIMusFk9eLE0jopgUKUSmrh48dl8mB3KjD3 sq+RD/NIUJj7TP0bYYXFXJtwkoABpXbJefYJjYC3WYU2jhDTwBzzoepL8fiUNWSnxyxT AoIw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=gZ9xWFVonfSWOeedDrbddVsWW/90+f1Lep6wrJrzXd0=; b=gVowgWqvY3rp5N/TUGm56QoWZ5aER48oo4PEklfqOXt8qu6qTksSZNMMscdEHPYOPI HJZjTubG9S3xWiXfBhs/D8EIeXiVq8vN4/cM0cxq7s7Eg0FrtK92rzuYhM9tTd6KZEZo 63QVnaI+j23TZSAxI6RxmggqNt2FcSQlGbbDiclNE28pg+qQ7C5O/aKvbfJ2maW3aAzb mxe5oUpkGmuvzE33OnNW4BXBHPW89IPYRoLj5DGL9ResGxyofiFvn/HsV/LmKa1Hq/Rg uU7osRVdf0Q/tA1zIy0wZzEtnqQDSAVVwor+lNmlZsB7alnD+chfY0rYc2jiAat0pQAq P6Bg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=WrAicx2d; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 2si9758122ple.275.2019.05.10.21.10.53; Fri, 10 May 2019 21:11:10 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=WrAicx2d; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726142AbfEKEKF (ORCPT + 99 others); Sat, 11 May 2019 00:10:05 -0400 Received: from mail-it1-f195.google.com ([209.85.166.195]:39133 "EHLO mail-it1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725957AbfEKEKF (ORCPT ); Sat, 11 May 2019 00:10:05 -0400 Received: by mail-it1-f195.google.com with SMTP id 9so5234975itf.4 for ; Fri, 10 May 2019 21:10:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=gZ9xWFVonfSWOeedDrbddVsWW/90+f1Lep6wrJrzXd0=; b=WrAicx2dt3Kys4ZYx+qN0emWJsBCRP2UnXTZ1rKAJnnJneTbFYRGkUE4WTQqRD3/Aw PKc574CF0CRTgg7H/cUEyduqP1yFUo57zU6bA3SpbmjMe+ieCD5G9v+xU8tfuLXFdwmw TQQRqx4LLb1x+G3kPRU7oxP2+x0abm10LsagTGk8xckv//ReuD4GzYydxOrofns9Ho77 OBgz/aU8aYNlnN1436QtZcwQ3BRZybsebzcTSf/SPVsZkxWVSJM71gq1Eggqu511xgAj DbYr1DUEy/GJFuNLzDp3hTzExQy3WN0V1FD4iQg3SorQVmsyXqRp8RRMP1NBxWrwzSsJ u//A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=gZ9xWFVonfSWOeedDrbddVsWW/90+f1Lep6wrJrzXd0=; b=VrGlGcduLtFwre0JcAhm0+quZZiW4UYbZPPb7u0hlAoXV39ETrA0THb9q5NfKk97+/ gjoR6uqmIzWJntl1O5unZ5Yk6bIOWhoWV2x2wsMmdDX6OfQ0pLgO1WDRFlqxggnZvZqX 7D8K3pzEM/h7Itc/qWjv5CnePaRDMTQ9rNSIz9w2+7MQcIkEuRr2K7F8G+A8DT6zMKx1 kmDdGXgnFatpSAkhS5DOeK3CGrPj2auMy+rCkLrrlVQS5RHc6i2py8yDSTOf3BywTT1W dDCp4GaZswu2jWv8qWiQUpgckT1Vj8KE/CfU6Kymvdp8Fw6fUVUWmNhOfVpuM+RJEW+C Ffuw== X-Gm-Message-State: APjAAAUdFNDxFwR/IWIptUc0r9FWWYz/exvWj5eHpxnzOMcReb8SV4ov mHsFC1KWU/1KRTv0O+x64G7f8Jf7Elvt0rOJiMQ= X-Received: by 2002:a24:9f86:: with SMTP id c128mr9519107ite.154.1557547804563; Fri, 10 May 2019 21:10:04 -0700 (PDT) MIME-Version: 1.0 References: <1557447392-61607-1-git-send-email-yang.shi@linux.alibaba.com> <87y33fjbvr.fsf@yhuang-dev.intel.com> <20190510163612.GA23417@bombadil.infradead.org> In-Reply-To: <20190510163612.GA23417@bombadil.infradead.org> From: Yafang Shao Date: Sat, 11 May 2019 12:09:57 +0800 Message-ID: Subject: Re: [PATCH] mm: vmscan: correct nr_reclaimed for THP To: Matthew Wilcox Cc: "Huang, Ying" , Yang Shi , Johannes Weiner , Michal Hocko , mgorman@techsingularity.net, kirill.shutemov@linux.intel.com, Hugh Dickins , Andrew Morton , Linux MM , LKML Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, May 11, 2019 at 12:36 AM Matthew Wilcox wrote: > > On Fri, May 10, 2019 at 10:12:40AM +0800, Huang, Ying wrote: > > > + nr_reclaimed += (1 << compound_order(page)); > > > > How about to change this to > > > > > > nr_reclaimed += hpage_nr_pages(page); > > Please don't. That embeds the knowledge that we can only swap out either > normal pages or THP sized pages. Agreed. compound_order() is more general than hpage_nr_pages(). It seems to me that hpage_nr_pages() is a little abuse in lots of places. Thanks Yafang