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 507FAC678D5 for ; Wed, 8 Mar 2023 18:17:23 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id D7C086B0072; Wed, 8 Mar 2023 13:17:22 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id D2B9B280001; Wed, 8 Mar 2023 13:17:22 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id BF25A6B0075; Wed, 8 Mar 2023 13:17:22 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id B15576B0072 for ; Wed, 8 Mar 2023 13:17:22 -0500 (EST) Received: from smtpin26.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 6242D40F5F for ; Wed, 8 Mar 2023 18:17:22 +0000 (UTC) X-FDA: 80546538324.26.8945E4F Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf25.hostedemail.com (Postfix) with ESMTP id ADA00A0025 for ; Wed, 8 Mar 2023 18:17:20 +0000 (UTC) Authentication-Results: imf25.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=fm9Tp+jD; spf=pass (imf25.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1678299440; a=rsa-sha256; cv=none; b=0t0kinnNZcsFrcMspS6nL2Qhuugt1FiY3kC5dPOuGhlmpsocxGvm6TmcqBXv0Mc3Jra+So tHLCHKYt9wwrxznNOp5lK4fbZr455VQwGsvEflm06rAyT0ejNI0B3aMdZou40/0BKT9waB rSDJSqtSpRoMuAQludGW8YCbKnJebwU= ARC-Authentication-Results: i=1; imf25.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=fm9Tp+jD; spf=pass (imf25.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org; dmarc=pass (policy=none) header.from=kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1678299440; 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-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=9zx2rYyBb8vKx4wAWPR7ki/ln8GdXCLKZ+SebZW8z0k=; b=OHUnz14qEggB21VXaWe8JcrfgciaabbvmQ4dIFMkRpzT0RfUTYUWbllTnQmJw9voj8u9oU KAGmsQCBBM9X1zGwrlEyuokZP75wVnVOOTttOW8UdJAjAAr9TXf+/rcdwRjbiX/RLcXsH/ +fPycdcHqqBxmKepWblDRclLqsWxrcI= Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id A6670618E3; Wed, 8 Mar 2023 18:17:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 9319BC433EF; Wed, 8 Mar 2023 18:17:18 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1678299439; bh=WSCK4EjboqTykWYFTwfbIgSgksqAobdYHV2zBrtxayQ=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=fm9Tp+jDqiNJLqPdubQL+sEqoF7WZwjzbyVuKq+9MWn9ysbdCt5ZNPU6JbsKRN9Zg JkNLuMwYtPX2w/9tvhKaBqMIj07vlY3iYlZlBEYZJPY9ZMIejpGFpG1QswMMXHZAP5 YNk/VSlUjOYLNxMQknxZksyrIkd43L/P54M3dWwVGmuGZuqoyVE94rUkuy0/CjfH7C 9xk+6GsVbcc/U3HEIPRscOwUkG2+2HIyUaaP8ISTMxMUgD+3f/BkZjqMgaeq03nomF Sm74VuBfnURUVXRNZbd1apZCYyTbaV/1QwjQfG33+M+J5fNwGfJNRZ0xXRDIhX6jBF /ibogoPIXwq4A== From: SeongJae Park To: Kefeng Wang Cc: SeongJae Park , Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, damon@lists.linux.dev Subject: Re: [PATCH v3 0/3] mm/damon/paddr: minor code improvement Date: Wed, 8 Mar 2023 18:17:17 +0000 Message-Id: <20230308181717.46186-1-sj@kernel.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230308083311.120951-1-wangkefeng.wang@huawei.com> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspam-User: X-Rspamd-Queue-Id: ADA00A0025 X-Rspamd-Server: rspam01 X-Stat-Signature: 19fxyk8iyxy7oew4kdg9qy1t8e4hn93u X-HE-Tag: 1678299440-627306 X-HE-Meta: U2FsdGVkX19j8W7wUdbbS2uUz05YbYg+Z3ZwZx0IqclnFrz4oUXR8CzzN9Lw8YrEDutPB2OlgKerr3bBtr0yt+40ZCX8oSwrJjIbMxxO6lSYtJ6+q60VJTf+f8fnlM1oxEt4qFz0xRjJwcYSnMO3mehhVf0ZhlLxACwA7CgF6lHsq82ubYOGnLxiLL4rtb4H+KSwpGdv8VZVg5K28isncVZvq3I5hqH1BfJdI10tzQF7GegiCsH+LshflimR6S8rV6uM+H+WzqupP8nBtoZ8BNCtV8C10PWQ0SwPaUA6Yk6IUZ8fTLxmzPYpXp/7mN3DVhoXYN79Rk17EG9ESQpxGhFjFVNOl0nUoQJy0TV2O8BnH9AsMbUtenNWnGqD2eAYdTHGUNrSXkfsAQNFXdLGAqQ2ZNWToaBC7K4K9S3GHyAtlk23kI3l9Gx7K19Y0RaRnwAAysGv+AOUJOWryY/ziwr+sBzZr0PByg3TGvVqJ1NzLUdS+ZAKFN97lGWh3RH92OK4Xva7OEp4ru0c2vEYUZ8iMbvGLzhCQLOvVil+HJXuPLFYCf5tDlvW1RbiTtO4Q5XzA9XOQUZ17iPGA+vhztodzFmsXrkc8tPGL26mwROibO1U0eKEgozehEna4B07hixbPLTFekjHJ/c2IG0B5ndamDv/qBAYQOL9oIoOzMrNNSa7sT5hp1e7dYm7uSrcnFCMDMmFFuyoXBbzz96GhrYNBgjOery+vgyQ8tB+6o2WxLHIdRqCFo9BDhnft09hwLxRvyqwcVRpGukkwDVMBQ9WMvn1O4shsMKp/X2HZwptQdjFe7X5eUjsY2t+zSBXvpJ0caHnoitup5aFEYY/lCJXm+NUmRUsNwk4tYF0ulNC0IE8v/ioFGnkmOmusBqovNgdRsPBapl4+tuSEWgBn+ToUqmTc3XCYVQt6p40SxxnPKLacuj8WqG+jlLkk0C/kK4hIDfLUZ9KRtQGgu7 4TO1p7kF 0MBgceJr21LAy7wNYXnOlCyp7lC4lKI9CcrPNNtmf4Mw7GXlvKBv7JW3Yobvi6FbM4+jCWsrH2CnnWljndtBQ+Qz1ghOJDO9jMsY3K9b7TfAR6sBCYA6n2NEBkq9jqOHPQHO6A20fCySQVnEAJj2J8OZbWcH+wwWAwMIeJ48OuBhbyPG6P63tjds5Ma5KcDGF5DE4nRxG1eBLLGCRWkrhy5dfmtTZoBHFbwXiUUGnGp5vPbIaKQ+qq7Gi5YmZ1nDxd0SFb9/rYgCTYcFr8IC9mI9WMEVcGq3a5UsQkQav8zjZAKSDIG20Vca8RibOgv1AQuSGqCXK6DuYXBgDzXbsqASm8Vq18APw/R1ekhRGJ+KTI2GFEYsKGjURWA== 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: Hi Kefeng, On Wed, 8 Mar 2023 16:33:08 +0800 Kefeng Wang wrote: > Unify folio_put() to make code more clear, and also fix minor issue in > damon_pa_young(). > > v3: > - as SJ split the bugfix from v2, and it has merged[1], only do code > improvement > - fix a missing folio_sz update in damon_pa_young() > - add RB from SJ > - rebase on next-20230308 > > [1] https://lkml.kernel.org/r/20230304193949.296391-1-sj@kernel.org > > v2: > - use goto lable in damon_pa_pageout() and refact too more functions > > Kefeng Wang (3): > mm/damon/paddr: minor refactor of damon_pa_pageout() > mm/damon/paddr: minor refactor of > damon_pa_mark_accessed_or_deactivate() > mm/damon/paddr: fix missing folio_sz update in damon_pa_young() > > mm/damon/paddr.c | 26 ++++++++++---------------- > 1 file changed, 10 insertions(+), 16 deletions(-) For the series, Reviewed-by: SeongJae Park Thank you for your efforts on this, Kefeng! Thanks, SJ > > -- > 2.35.3 >