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 8ED54C369D3 for ; Mon, 21 Apr 2025 09:11:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 335A66B0006; Mon, 21 Apr 2025 05:11:34 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 2E5406B0007; Mon, 21 Apr 2025 05:11:34 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 1D5216B0008; Mon, 21 Apr 2025 05:11:34 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id F28456B0006 for ; Mon, 21 Apr 2025 05:11:33 -0400 (EDT) Received: from smtpin03.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id 705F0BA457 for ; Mon, 21 Apr 2025 09:11:34 +0000 (UTC) X-FDA: 83357482908.03.35DEE24 Received: from out-185.mta0.migadu.com (out-185.mta0.migadu.com [91.218.175.185]) by imf05.hostedemail.com (Postfix) with ESMTP id 62F03100006 for ; Mon, 21 Apr 2025 09:11:32 +0000 (UTC) Authentication-Results: imf05.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=Pm9wg5oI; spf=pass (imf05.hostedemail.com: domain of ye.liu@linux.dev designates 91.218.175.185 as permitted sender) smtp.mailfrom=ye.liu@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1745226692; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=h/En9Stu4OkS759SXeoMP2rUGBMH0kSmoju2wjtY+Rw=; b=Tb4lBQcXaMt1FXzSsbVeN5vtV/G7ig4c71YoJA+PKxO36BRQsmI2oPXoFiGJNEGvDWcuVQ OMr0G/3giULdk1a8Jb0mrXE36ZZSwmQjrke+vCdl7BS6JZ5iQVeAX0y1PBuAF3Djhf+ZCW clVXURlJU2bnD4Wk0fP7sJByHiKImR4= ARC-Authentication-Results: i=1; imf05.hostedemail.com; dkim=pass header.d=linux.dev header.s=key1 header.b=Pm9wg5oI; spf=pass (imf05.hostedemail.com: domain of ye.liu@linux.dev designates 91.218.175.185 as permitted sender) smtp.mailfrom=ye.liu@linux.dev; dmarc=pass (policy=none) header.from=linux.dev ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1745226692; a=rsa-sha256; cv=none; b=m3LK+9ZAoAArJ4rf73+nJXv3H0LmeJwV62Twj3xbmaZS9+mt91AmYBES2X+nOpazjIZl8V HeEZn7FZFHolc7g6T5BQ79jtLnkiuEXMcGylFNDx2NaPtpDjRiYV5nWYYHj2uoH8/xKRkS aFC3Q2hNN8eMvd4k2JXNV1rC7zfJqQw= Message-ID: <1a52c747-3642-4f05-a662-c010240e9e4b@linux.dev> DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1745226689; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=h/En9Stu4OkS759SXeoMP2rUGBMH0kSmoju2wjtY+Rw=; b=Pm9wg5oIzV0PNml0dLOFVIRyLwwYT6th+yG+HaXmWxO9kM2ko5r4yWklEIk2Vo/h3wqPQY U/5HTaJIyHFdPh49OHmWMQw02LRpDFVgK2OaKbRXXI1bJOs3Nq0a+gCttrYl3moUbwpuLt VNK8VXRjbqfYjug1SiFwoQy3346Y3mA= Date: Mon, 21 Apr 2025 17:11:22 +0800 MIME-Version: 1.0 Subject: Re: [PATCH v3] mm/rmap: rename page__anon_vma to anon_vma for consistency To: Lorenzo Stoakes Cc: akpm@linux-foundation.org, nao.horiguchi@gmail.com, linmiaohe@huawei.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, Liam.Howlett@oracle.com, david@redhat.com, harry.yoo@oracle.com, riel@surriel.com, vbabka@suse.cz, liuye@kylinos.cn References: <20250421015823.32009-1-ye.liu@linux.dev> <9b0bd289-40a8-4e4d-89e0-7eae922bd41a@lucifer.local> Content-Language: en-US X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Ye Liu In-Reply-To: <9b0bd289-40a8-4e4d-89e0-7eae922bd41a@lucifer.local> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-Migadu-Flow: FLOW_OUT X-Rspam-User: X-Rspamd-Server: rspam09 X-Rspamd-Queue-Id: 62F03100006 X-Stat-Signature: ghjukjcdn7qmrujdyip4uxgcqgegdpw9 X-HE-Tag: 1745226692-203356 X-HE-Meta: U2FsdGVkX1/vqfq3lKt/OtaHKgetB4JdgfEwNoPo0MF9XFsqET4pMVKk3gOhYG3sOXGO9v/QYwI5htNdUUd0h6iJBE4OycuGlO+A7mbBU13hN9e3ZTaYAHznuMIRcgZ1kQjkwtVlDySdp+3ewOZFgBIGZBJB+1vpiuvNOyhusozz2eD45a81DlQ2dSbYlQGRWDZEcv1kooHVlbGyNgLrpKRcMxrjGxh1aSu13pR/JHVLVaJ63mmt5WNx8Q+9zA+FQ9V5ZHihgcb+h16Sttd4Vl96A/htJeTxW36W0K1XTzykXrt1ezzzlsAJ5QwO4mSYK4gRw8Sj5Sa4j/edLQiHUFOKDXdKW5xItPAsOCgRNHSS6onA4oz2iXT0aBlaOjKxqee8gXdzpKPcEhvjCCwB3Ec1T04ZgUzFh8esEoJUNa/AoaXCsKWZX6QDpcB7uNdAKLzILrSyBMOmF0SwuSYSOpjO86PJ3mgYpFYnCQvgHoFXY5WjUjHZuDU5q6duhHDcEuDO3gNwOoO1IhMQLCc7Y0Z9ycPkXZ8aB9v8Sw9ykbcBuzLfYb05/SPGQJNErGH04iH4Qxzd/LJ/rZuA7OKhdmNTvCpREJWGxYBZVyKP7M6UiKkE1WLHzVmN+75sF074N0HKJByWkuQrZ0Mv6UAqTQFZ0RBZF+F9om0SN95zzDiM4YLMSNfJTXsPMiRkyQEdK7U3xd/Mtt2KexaQUB4ljzWFFtq9uGwUw3V72VCGYEEYKREt80JaZcI3A66xY/nSJHyZ9+K2CeKNuNKdgNDA1guw3Pq4cP8Hk4jzJdR8ONUjmOhkAHjNqmmbKsbGYWaHo1KzCDFj3s2EGh2n5y/dbtQN3pXdXkL+I/xS0pKu8KUsI0bmeWZk/BPqmDXvvlvygGGcEXqtAa/11ULRji/pmacCAwqRKl/PjZSJQqEnn9WbHIzksVSr4YImdZpIeEzkeObJjLHmLOkcD56/+Ou E1RoAEZO SbpNwVIiep32ubUPUjN92pXifW+5g6Yx4x6+VsCIj9t29vgKpWlu9PwcYkzcsnAZu6BxikzVtQkUmfGo2n1uM8TIWYrXJGQNyl7JfiW/fEWfy1/RKNhzOaF8OGyBd+cHk16DmzbVoCbgWMb9Xw3sV9+qBy7ivsnQa3HdP0eI3i//INNjb6UuErs9+lJ5dWtRkgdBCdSZGoysAY5aBfLXTEh+qAwPz0uBoEh0CPJ0YLtohbt4gv1SjgzygJ36HKgdIRkWoZan8RwKaWXexIcWnXLvp7NdlW/sl5K30ZAFWKzi9RdjaoDzjaHSULTn1dEJpM8SZEzpAYQkYJdmp4lEk7PMScaJcUwixPBEO 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: List-Subscribe: List-Unsubscribe: 在 2025/4/21 14:03, Lorenzo Stoakes 写道: > On Mon, Apr 21, 2025 at 09:58:23AM +0800, Ye Liu wrote: >> From: Ye Liu >> >> Renamed local variable page__anon_vma in page_address_in_vma() to >> anon_vma. The previous naming convention of using double underscores >> (__) is unnecessary and inconsistent with typical kernel style, which uses >> single underscores to denote local variables. Also updated comments to >> reflect the new variable name. >> >> Functionality unchanged. >> >> Signed-off-by: Ye Liu > Hi Ye, > > I see you're still getting used to the process :) this patch isn't correct, > you're sending a single patch at v3 against a series with 2 patches in it > [0], but you have sent it entirely separately - this is not correct. > > If you want to revise a series, you have a couple choices - you can write a > 'fix patch' -in reply to- the patch you are altering, which provides the > delta against that patch. > > This is the preferred way in mm, unless you have made such a big change > that this is infeasible or if multiple files change at once or if so much > has changed it would be a bit silly to do this. > > To do this, you use a client like mutt/neomutt/whatever you use (but one > that can do the linux-y In-Reply-To stuff correctly) to reply directly to > the mail, and then say something like 'hey here's a fix patch please apply, > and add the output of the git format-patch that contains the delta against > the file at the end after a single line with '----8<----' on it. > > See [1] for an example. > > Here, however, you've confused matters a bit by sending this as a v3 when > it's not really a v3 properly, so I suggest in _this case_ you just resend > the whole thing as a v3 with this correction in place. > > You should also propagate tags, I gave a reviewed-by tag here, so make sure > to include this and others given when you resend the v3 series. > > Thanks! > > [0]: https://lore.kernel.org/all/20250418095600.721989-1-ye.liu@linux.dev/ > [1]: https://lore.kernel.org/linux-mm/13426c71-d069-4407-9340-b227ff8b8736@lucifer.local/ > >> Changes in v3: >> - Rename variable from page_anon_vma to anon_vma. > Also _please_ copy/paste all changes for all versions each time, and add > lore links for each prior version. > > You can figure out the lore link as https://lore.kernel.org/all/ from mail headers>/ - I always do this manually :) you can see I already > figured it out for the v2 in [0] above. Thank you very much for your detailed feedback and guidance! I sincerely appreciate you taking the time to explain the correct workflow and the options for revising the patch series. I’ve taken your advice and resent the entire series as v3 with the following updates: - Incorporated the variable rename fix directly into the series. - Propagated all relevant tags (including your Reviewed-by). - Added complete version history with lore links for v1 and v2. Your patience and clear instructions are incredibly helpful as I learn the community’s processes. If there’s anything else that needs adjustment, please let me know. Thanks, Ye Liu                                                                   >> --- >> mm/rmap.c | 8 ++++---- >> 1 file changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/mm/rmap.c b/mm/rmap.c >> index 67bb273dfb80..447e5b57e44f 100644 >> --- a/mm/rmap.c >> +++ b/mm/rmap.c >> @@ -789,13 +789,13 @@ unsigned long page_address_in_vma(const struct folio *folio, >> const struct page *page, const struct vm_area_struct *vma) >> { >> if (folio_test_anon(folio)) { >> - struct anon_vma *page__anon_vma = folio_anon_vma(folio); >> + struct anon_vma *anon_vma = folio_anon_vma(folio); >> /* >> * Note: swapoff's unuse_vma() is more efficient with this >> * check, and needs it to match anon_vma when KSM is active. >> */ >> - if (!vma->anon_vma || !page__anon_vma || >> - vma->anon_vma->root != page__anon_vma->root) >> + if (!vma->anon_vma || !anon_vma || >> + vma->anon_vma->root != anon_vma->root) >> return -EFAULT; >> } else if (!vma->vm_file) { >> return -EFAULT; >> @@ -803,7 +803,7 @@ unsigned long page_address_in_vma(const struct folio *folio, >> return -EFAULT; >> } >> >> - /* KSM folios don't reach here because of the !page__anon_vma check */ >> + /* KSM folios don't reach here because of the !anon_vma check */ >> return vma_address(vma, page_pgoff(folio, page), 1); >> } >> >> -- >> 2.25.1 >>