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 3E167C5479D for ; Thu, 5 Jan 2023 20:17:05 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id B2AF98E0005; Thu, 5 Jan 2023 15:17:04 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id AB3B58E0001; Thu, 5 Jan 2023 15:17:04 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 9850C8E0005; Thu, 5 Jan 2023 15:17:04 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 8295E8E0001 for ; Thu, 5 Jan 2023 15:17:04 -0500 (EST) Received: from smtpin05.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay01.hostedemail.com (Postfix) with ESMTP id 567361C30C7 for ; Thu, 5 Jan 2023 20:17:04 +0000 (UTC) X-FDA: 80321854368.05.36594FD Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by imf26.hostedemail.com (Postfix) with ESMTP id 9BB93140014 for ; Thu, 5 Jan 2023 20:17:01 +0000 (UTC) Authentication-Results: imf26.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=UZ41CXtx; spf=pass (imf26.hostedemail.com: domain of sj@kernel.org designates 145.40.68.75 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=1672949821; 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=Ro6ZV0KueD+WBFZnlJYwaGtOgX1sJsVct3jvuPBlVJ0=; b=z0zKF7q0E8a+gwk/DIddKMImZk+Erp9ktdr7vwgwiW+kSiPbyiEcaJr+25KraDXspQ8k5D kakARe3WP+CnUxcq2EySQwLew39N5BH0qLV/Y5cadLeV+qjCUrsIIwQDaYnFUtnmzGCElu mDsQDbflq5iXAOPW7PvH6hRIDXwBpzk= ARC-Authentication-Results: i=1; imf26.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=UZ41CXtx; spf=pass (imf26.hostedemail.com: domain of sj@kernel.org designates 145.40.68.75 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=1672949821; a=rsa-sha256; cv=none; b=lX2hF80eYihdf8+Jw2zAS/N0JiJy1yoorySzRVgv4A27X5uC0aih3ta7T8m8s5eUc8DmHw PtxlJb2eFzu/rwYQ/8QDFfh18++ppzNoCuRZYJyCgR7rdOZp3G4QgEuci8E5sIHie1AGwH BUSdzuT3XCiiQupCTPeYSGfjlDkIDr0= 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 ams.source.kernel.org (Postfix) with ESMTPS id B9BB3B81BBE; Thu, 5 Jan 2023 20:16:59 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C55F4C433F0; Thu, 5 Jan 2023 20:16:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1672949818; bh=r3UUh01lMIQpHkRFoc0xsCt5i0tnsap0EO6qM6kQfmo=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=UZ41CXtxvnwcp4WgInyMxrMcSIoEwQbVWAd8MAelnrZzGFRcvmgFQX/ZtqSRZ9NQf Y339Drwthb7PxFoFu6G7iK89Oj3nCKBywcyC4wUWvr58X1iZd/g3Y09r8d3U7jx07l CQPTKHTVIYZ8Bot2f+1Ucwhs5OnOBBzcm3vcmpaMLI8K4n/iXSwKk6691Mv1JYiIed y8OQqRGxc6Hy96BlbAGWtsz3P0O6WwDgJn+jP7iZYN+QlbSaNAeol1wuaDKTxwx2Zc Rb7l87axgTRk5TTyLktKRipIn+vvguBBQC4VqXw6SW6awlDVYVhmAJi4sN/FvEwbHr Bpz98GnoTxjTw== From: SeongJae Park To: Liam Howlett Cc: SeongJae Park , "maple-tree@lists.infradead.org" , "linux-mm@kvack.org" , "linux-kernel@vger.kernel.org" , Andrew Morton , "damon@lists.linux.dev" , kernel test robot Subject: Re: [PATCH v2 26/44] mm/damon: Stop using vma_mas_store() for maple tree store Date: Thu, 5 Jan 2023 20:16:56 +0000 Message-Id: <20230105201656.49522-1-sj@kernel.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20230105195213.acthhrocx5vz33uy@revolver> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspamd-Queue-Id: 9BB93140014 X-Stat-Signature: 8azodcr6p3dzzjg8gfzrf4acg3emet6f X-Rspam-User: X-Rspamd-Server: rspam08 X-HE-Tag: 1672949821-28681 X-HE-Meta: U2FsdGVkX18qymf2+VzLj0Lr2Ihld3Owc/bOC/6dPIGzhVdG46UzTXMpHjvPFaVhXOsIobs8y01ozKwkKeI/BxWU1BBytOgVNz37thLsOTD9vJUrDscu+LKkiTkyBLBRiOkMjRSivN3yJriMZm8tTnxF+lHopeDcO7EirXc+Yut+RQDw/9avvpDKA+8HfX5hgVIsTpeLQYQKcsljIuy1wrpqwh/o+XMu5FX5DxQ0JEtO1vF9O4EtL94vnGdQCwTm41LKYgawz/fIdVS59PdN2p1Q/UzfhT6du7cukc+/NaMzvlAz0IP2Qh9oFJeylgy1ckpOB7spvsMc1W//WjOak8QWHlNmNdgycABTUQFo06tFx+BaVa5J9S2OI67Cvfnyy6e5+8GMP1mAsMmmz7h5ZzudefOwrpg1k6GALtQ6hXu51fxElVoT4dRPj2fW5q+wF4TMLiHJGOIKY8H64Znbl6K9Sl3yTw1nn1MeMprWmJ/K3G+zdx1bPUI7Eq1cNgiZ/YVAXgxEIV8CmVklW9PRGofvju0ZCJS3hDWVWJd2a12XV+OJ+Jqj/nBBFxCAcj4nqtGf+HQh7QQ/CfHOUsutigorGU4u9XM4ISApzBetKB8kawiWm1SPx58HOxOLWuePpBYRbUn99cHJZG5ZD4fcn61ygJM4WiK0tYn7CnmLW/pkyRFyAJUtuMHrI3XXwOD5fymtPQsgKwxPoh/UFPFgHkoHZgiOFqMkYqm2pR0Ek7qit3jrtFyUeCKdRnob1DwAvjiO4e05vsXdgpbkPbS1DW9Z7hUcGcNZuHbHJ8OczmwchoxDqjOFg3NkzfzivhjE3T1bpxdoHOzyNBoM9ee9KyxGY9UhcMyK2F/fFauYK5Ym2QvKdNSJzuOvOZ8jK0UGmfmfBy61/vxDngDsh06o+Qg9Lqn7b4jHgNuBJx8FaZc9OUfYi5GsK2l/vuEHobxxIfYT1yiHulIlkF3dLdE J9hGSWh4 W+/DNHz8ScbnWRnqFPlOWVwDvXtXHU+tFqMObf2You22OSY+e1DhxPzvqgoRO6OKkYYOKf0s7cMH5uNS8fLjlwnSXSjZ5JTgTSI7TY/UsDw+ZJbNJSEvj4ytDKnjRTIWWRxsz34S6i9nQVNNX05DlXqy/B3tYXYdZLuj3 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 Liam, On Thu, 5 Jan 2023 19:52:21 +0000 Liam Howlett wrote: > * SeongJae Park [230105 14:33]: > > Hi Liam, > > > > On Thu, 5 Jan 2023 19:16:00 +0000 Liam Howlett wrote: > > > > > From: "Liam R. Howlett" > > > > > > Prepare for the removal of the vma_mas_store() function by open coding > > > the maple tree store in this test code. > > > > But seems this series is not really removing 'vma_mas_store()'. Wouldn't it > > better to do the preparation and removal together in a same patch series? > > It does from the all code but the nommu side. The definition is dropped > from the header and c file in "mmap: Convert __vma_adjust() to use vma > iterator" [1]. Thank you for nice explanation. > > > > > > Set the range of the maple > > > state and call the store function directly. > > > > > > Cc: SeongJae Park > > > Cc: damon@lists.linux.dev > > > Reported-by: kernel test robot > > > Signed-off-by: Liam R. Howlett > > > --- > > > mm/damon/vaddr-test.h | 6 ++++-- > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > diff --git a/mm/damon/vaddr-test.h b/mm/damon/vaddr-test.h > > > index bce37c487540..41532f7355d0 100644 > > > --- a/mm/damon/vaddr-test.h > > > +++ b/mm/damon/vaddr-test.h > > > @@ -24,8 +24,10 @@ static void __link_vmas(struct maple_tree *mt, struct vm_area_struct *vmas, > > > return; > > > > > > mas_lock(&mas); > > > - for (i = 0; i < nr_vmas; i++) > > > - vma_mas_store(&vmas[i], &mas); > > > + for (i = 0; i < nr_vmas; i++) { > > > + mas_set_range(&mas, vmas[i].vm_start, vmas[i].vm_end - 1); > > > + mas_store_gfp(&mas, &vmas[i], GFP_KERNEL); > > > + } > > > > On the latest mm-unstable, vma_mas_store() uses mas_store_prealloc() instead of > > mas_store_gfp(). Seems the difference would make no problem to this test code > > in most cases, but could I ask the reason for this change? > > mas_store_prealloc() expects the maple state to have the necessary > memory to store the value. Using this function is the right way of > storing the range. In fact, we would only need a single node since > these values will be append operations anyways. Again, thank you for nice explanation. > > > > > Also, should we check the return value of mas_store_gfp()? > > I can add this. The only reason we would return an error is on ENOMEM > which seems unlikely here. Again, it is a single node that will be > used. The size is 256B, but it's safer to add the check. You're right. I'd prefer having the check, but I'd not block this for the trivial nit. Reviewed-by: SeongJae Park Thanks, SJ > > [1] https://lore.kernel.org/linux-mm/20230105191517.3099082-28-Liam.Howlett@oracle.com/ > > > Thanks, > Liam >