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=-9.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,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 7B0D4C47423 for ; Fri, 2 Oct 2020 11:43:17 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id CF478206FA for ; Fri, 2 Oct 2020 11:43:16 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b="gVXtVyle" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org CF478206FA Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 1FA856B005D; Fri, 2 Oct 2020 07:43:16 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 183DE6B0062; Fri, 2 Oct 2020 07:43:16 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 04A836B0068; Fri, 2 Oct 2020 07:43:15 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0131.hostedemail.com [216.40.44.131]) by kanga.kvack.org (Postfix) with ESMTP id C86156B005D for ; Fri, 2 Oct 2020 07:43:15 -0400 (EDT) Received: from smtpin18.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id 6681F180AD801 for ; Fri, 2 Oct 2020 11:43:15 +0000 (UTC) X-FDA: 77326799550.18.vest57_060f23b271a3 Received: from filter.hostedemail.com (10.5.16.251.rfc1918.com [10.5.16.251]) by smtpin18.hostedemail.com (Postfix) with ESMTP id 46952100EC663 for ; Fri, 2 Oct 2020 11:43:15 +0000 (UTC) X-HE-Tag: vest57_060f23b271a3 X-Filterd-Recvd-Size: 5678 Received: from mail-qk1-f196.google.com (mail-qk1-f196.google.com [209.85.222.196]) by imf10.hostedemail.com (Postfix) with ESMTP for ; Fri, 2 Oct 2020 11:43:14 +0000 (UTC) Received: by mail-qk1-f196.google.com with SMTP id g72so942314qke.8 for ; Fri, 02 Oct 2020 04:43:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=C4mW4jmQiUVM5ODOj2Y+K5sHn+/g6biBAH0gCyhwiEg=; b=gVXtVylePjM3/u0XGDB6MreiXopfmG2teSFQ22xJMa8IuhxllbxEs0hpxJto022pVB YJ2n/L4JxcA0ZpEDQHCNigywuT8CFvPg5raX6lFic7q0WGqRKaOiMhaSjoywXgpzPdME WsZH79mTQIwmVWRfa/RV0HnB8rek/VZ8caBooRhco5sNcoDR4gNTYC3sjso2ipA1AP2k /CBjQ/9M635HP+VAQfhf8CAW8KEMtC2WMYpcm8KBnW+JgXK0VUgqEBZOpy37wvKtIbnq sUK5LFTLUSng2W/xXkrsk22dZIX2xwLiBkAFimkV/WDPx6E6cm9c7eWFvCfPCQ/BZr57 phRg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=C4mW4jmQiUVM5ODOj2Y+K5sHn+/g6biBAH0gCyhwiEg=; b=n59ryAHY5wfkv1jOy0jksgYWjtccR41fYRLN9c3gGcwV5CT2D7Pjl+5ZjUXW3ObRfT CTy2skq6zDzEeIHIHnIJVQo/gocaY6k6CU0bC/5yWqk6FVyHztyBzciipUmShTD9r0MJ 2QjobW9VgT4e2pm5AOg3JesPhuWG726LysKM699VD1RsKwqG+A3c3TsRSHDzSUTa4yQR dZVQxVLcwbsYZXY1bXYqBWKzBFdQ22PHw7sDvm1b/Z4Xw3zDWirYS7bln/tu8gzK30BQ 5clqZwWnqk3NVcufEF6UXGXx5cCpn/m2XeUrCzdB1pGspn2Fjasiy8yNrn0TEQ+IokRC WtMA== X-Gm-Message-State: AOAM532XsZ2wXsAlnXXqPZB4UckPUGjYrdrNGQIv2apS2XexGh7OVXzG KD/A4WunvKDp6C68yLpJL06YWA== X-Google-Smtp-Source: ABdhPJwj4XkAb9DCtBHvHQ8z80pFaHpYrtmCYbNEVt/MLnHuOBoPk0ZIs7I6F5cDt9HCcvKU78qZ9Q== X-Received: by 2002:a37:7c4:: with SMTP id 187mr1587359qkh.112.1601638994028; Fri, 02 Oct 2020 04:43:14 -0700 (PDT) Received: from ziepe.ca (hlfxns017vw-156-34-48-30.dhcp-dynamic.fibreop.ns.bellaliant.net. [156.34.48.30]) by smtp.gmail.com with ESMTPSA id v30sm815678qtj.52.2020.10.02.04.43.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 02 Oct 2020 04:43:13 -0700 (PDT) Received: from jgg by mlx with local (Exim 4.94) (envelope-from ) id 1kOJSy-005Z2Z-Lq; Fri, 02 Oct 2020 08:43:12 -0300 Date: Fri, 2 Oct 2020 08:43:12 -0300 From: Jason Gunthorpe To: Peter Xu Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, Andrew Morton , "Kirill A . Shutemov" Subject: Re: [PATCH] mm: Remove src/dst mm parameter in copy_page_range() Message-ID: <20201002114312.GI9916@ziepe.ca> References: <20200930204950.6668-1-peterx@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200930204950.6668-1-peterx@redhat.com> 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 Wed, Sep 30, 2020 at 04:49:50PM -0400, Peter Xu wrote: > Both of the mm pointers are not needed after commit 7a4830c380f3 ("mm/fork: > Pass new vma pointer into copy_page_range()"). > > Reported-by: Kirill A. Shutemov > Signed-off-by: Peter Xu > --- > include/linux/mm.h | 3 +-- > kernel/fork.c | 2 +- > mm/memory.c | 43 ++++++++++++++++++++++--------------------- > 3 files changed, 24 insertions(+), 24 deletions(-) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index 16b799a0522c..8a0ec8dce5f6 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -1645,8 +1645,7 @@ struct mmu_notifier_range; > > void free_pgd_range(struct mmu_gather *tlb, unsigned long addr, > unsigned long end, unsigned long floor, unsigned long ceiling); > -int copy_page_range(struct mm_struct *dst, struct mm_struct *src, > - struct vm_area_struct *vma, struct vm_area_struct *new); > +int copy_page_range(struct vm_area_struct *vma, struct vm_area_struct *new); > int follow_pte_pmd(struct mm_struct *mm, unsigned long address, > struct mmu_notifier_range *range, > pte_t **ptepp, pmd_t **pmdpp, spinlock_t **ptlp); > diff --git a/kernel/fork.c b/kernel/fork.c > index da8d360fb032..5f42d4afe0ae 100644 > --- a/kernel/fork.c > +++ b/kernel/fork.c > @@ -589,7 +589,7 @@ static __latent_entropy int dup_mmap(struct mm_struct *mm, > > mm->map_count++; > if (!(tmp->vm_flags & VM_WIPEONFORK)) > - retval = copy_page_range(mm, oldmm, mpnt, tmp); > + retval = copy_page_range(mpnt, tmp); > > if (tmp->vm_ops && tmp->vm_ops->open) > tmp->vm_ops->open(tmp); > diff --git a/mm/memory.c b/mm/memory.c > index fcfc4ca36eba..251bb5082f4e 100644 > --- a/mm/memory.c > +++ b/mm/memory.c > @@ -957,11 +957,12 @@ page_copy_prealloc(struct mm_struct *src_mm, struct vm_area_struct *vma, > return new_page; > } > > -static int copy_pte_range(struct mm_struct *dst_mm, struct mm_struct *src_mm, > - pmd_t *dst_pmd, pmd_t *src_pmd, struct vm_area_struct *vma, > - struct vm_area_struct *new, > +static int copy_pte_range(pmd_t *dst_pmd, pmd_t *src_pmd, > + struct vm_area_struct *vma, struct vm_area_struct *new, > unsigned long addr, unsigned long end) I link this, my only minor quibble is the mixing of dst/src and new language, and then reversing the order in each place. Would read better to be consistent: copy_pte_range(dst_vma, dst_pmd, src_vma, src_pmd, addr, end) Regards, Jason