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=-3.6 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no 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 30696C433C1 for ; Tue, 23 Mar 2021 02:11:45 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 1EABD619AB for ; Tue, 23 Mar 2021 02:11:43 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1EABD619AB Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 658F96B00FB; Mon, 22 Mar 2021 22:11:43 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 608186B010B; Mon, 22 Mar 2021 22:11:43 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 45ACB6B011E; Mon, 22 Mar 2021 22:11:43 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0060.hostedemail.com [216.40.44.60]) by kanga.kvack.org (Postfix) with ESMTP id 20DBA6B00FB for ; Mon, 22 Mar 2021 22:11:43 -0400 (EDT) Received: from smtpin35.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with ESMTP id D1EDE8249980 for ; Tue, 23 Mar 2021 02:11:42 +0000 (UTC) X-FDA: 77949512844.35.541B41B Received: from casper.infradead.org (casper.infradead.org [90.155.50.34]) by imf30.hostedemail.com (Postfix) with ESMTP id DC60DE0001B4 for ; Tue, 23 Mar 2021 02:11:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=0jV8L/eta9FUAW6CaxrcK1SMnq0K3JU8UUtVFBh1gwA=; b=YwTPJ8jt+kwsZR3kT+ATPd0haX KaSdDuDBAgCK1eYS79Qpn8PGTIONmZQEbbQKB2ntdrD0pThDWhDT4xrrXO+KG4l0sFX7wJjQO6ccx WGcW2ltnUYgJl+Um/A4QSmDPS8ca1WsF14sxlmwdEdnZrCG2KgOiFDRiuQAFP+VKS+37nTfqg1Q67 z9/RqD2ECS2rYuMjSa6TSPtcergEqN113zNeJM6oNDJsuSmNlpUa6cZETlbX7MVlp8J3a/bI98f70 P1Puoeu98IA3tu3UHnQwzCVIWmWddRWpqy8Ba0jTJiJChjH/I7+6T1u5ePWvF8RlVtHJEhbTpPNZS s6w1nurA==; Received: from willy by casper.infradead.org with local (Exim 4.94 #2 (Red Hat Linux)) id 1lOWW1-009OoH-84; Tue, 23 Mar 2021 02:11:31 +0000 Date: Tue, 23 Mar 2021 02:11:29 +0000 From: Matthew Wilcox To: Peter Xu Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, "Kirill A . Shutemov" , Jerome Glisse , Mike Kravetz , Andrew Morton , Axel Rasmussen , Hugh Dickins , Nadav Amit , Andrea Arcangeli , Mike Rapoport Subject: Re: [PATCH 07/23] mm: Introduce zap_details.zap_flags Message-ID: <20210323021129.GZ1719932@casper.infradead.org> References: <20210323004912.35132-1-peterx@redhat.com> <20210323004912.35132-8-peterx@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210323004912.35132-8-peterx@redhat.com> X-Stat-Signature: qnt4sgnnwrhu9bc6k1jhbzounidncerk X-Rspamd-Server: rspam05 X-Rspamd-Queue-Id: DC60DE0001B4 Received-SPF: none (infradead.org>: No applicable sender policy available) receiver=imf30; identity=mailfrom; envelope-from=""; helo=casper.infradead.org; client-ip=90.155.50.34 X-HE-DKIM-Result: pass/pass X-HE-Tag: 1616465500-490400 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 Mon, Mar 22, 2021 at 08:48:56PM -0400, Peter Xu wrote: > +/* Whether to check page->mapping when zapping */ > +#define ZAP_FLAG_CHECK_MAPPING BIT(0) > + > /* > * Parameter block passed down to zap_pte_range in exceptional cases. > */ > struct zap_details { > - struct address_space *check_mapping; /* Check page->mapping if set */ > + struct address_space *zap_mapping; /* Check page->mapping if set */ Now the comment is wrong. It used to mean "If this is NULL, zap pages with any mapping", but now it's always set, and the decision about whether to check the mapping is in the flag. Honestly, I'd remove the comments from both these members. They don't add anything to understandability now. > + unsigned long zap_flags; /* Special flags for zapping */ > };