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 AD2A9CD11C2 for ; Fri, 5 Apr 2024 19:28:08 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4A9826B0096; Fri, 5 Apr 2024 15:28:08 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 432FC6B0098; Fri, 5 Apr 2024 15:28:08 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 2FAA66B0099; Fri, 5 Apr 2024 15:28:08 -0400 (EDT) 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 10A7C6B0096 for ; Fri, 5 Apr 2024 15:28:08 -0400 (EDT) Received: from smtpin15.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 07CD741298 for ; Fri, 5 Apr 2024 19:28:07 +0000 (UTC) X-FDA: 81976463814.15.B489CCA Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by imf22.hostedemail.com (Postfix) with ESMTP id 4EA35C000B for ; Fri, 5 Apr 2024 19:28:05 +0000 (UTC) Authentication-Results: imf22.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=tCDnoxOU; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf22.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1712345285; 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=PuQOWt+eLRJlzPuVHynCiU8xMzZV6C3KtU48/WpN5NI=; b=ECkKONnfey7a1LT208pHreuuIfYeWq6hl/5nON6FZOR6o4E/dda0Qe/EXEVZKIyZlOpl9a fOfqnWygsF7zJYpPeqGVQu3ayLR17nnL+c4rDuZkaD3t2gJB+5pkJHPjtHUoWGUim18WPQ jP/s8ZWYayXxLl0qu33ESx3dAbhiRL8= ARC-Authentication-Results: i=1; imf22.hostedemail.com; dkim=pass header.d=kernel.org header.s=k20201202 header.b=tCDnoxOU; dmarc=pass (policy=none) header.from=kernel.org; spf=pass (imf22.hostedemail.com: domain of sj@kernel.org designates 139.178.84.217 as permitted sender) smtp.mailfrom=sj@kernel.org ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1712345285; a=rsa-sha256; cv=none; b=i6wjsTSD6w8BEcdhkXySeq7l9paGJPl4fmUkXj5+o0hhSOzzC95/CFTTeK/wnIIKgagH88 GvaV31J46OOoY5Hhiv+J9wcOnPW3CzfiLVUDqhkeFAR5wFefEeRKJtOvLgFaZAl2TlLrfC wJsAlUB4lNG6Xz2AUfS+7y8qtt1ldrY= Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 84AE26168A; Fri, 5 Apr 2024 19:28:04 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 825B1C433F1; Fri, 5 Apr 2024 19:28:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1712345284; bh=s4X5xF38G0xdBCRwrgje/a8N9SSzOSAJM1btVV1Oid0=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=tCDnoxOUCYeV7Pblmh2okpq4jgNajyC9ezTEfuysj8xIgQKvwqhTYee3ZS3FxYtGk DBOFJx2XagEzwPdeTYjk6C+UP8e6UrD7gyf/wU/1yucN4R9ldy2eot1NzCotWPvacE V6lYbcEcSYhEfUfSfzDcT28OfD/yEHL4Qj0GfsXuwOEex5sy1QxFCijWCMNYrX1MPH 0ctHqJrxJBtK1OjhAdshieBRj4Lc0632VWS5fzLo/FlklR2lnGZITVAA0IfDmdhnvs U6+woybOxPLwlLrGCqCgeDVTbN/mL2EVGoqUxyZfKWfFMz41nCtxnFzCmqn7wiPanQ tZaTkhPL2Uz8g== From: SeongJae Park To: Honggyu Kim Cc: SeongJae Park , damon@lists.linux.dev, linux-mm@kvack.org, akpm@linux-foundation.org, apopple@nvidia.com, baolin.wang@linux.alibaba.com, dave.jiang@intel.com, hyeongtak.ji@sk.com, kernel_team@skhynix.com, linmiaohe@huawei.com, linux-kernel@vger.kernel.org, linux-trace-kernel@vger.kernel.org, mathieu.desnoyers@efficios.com, mhiramat@kernel.org, rakie.kim@sk.com, rostedt@goodmis.org, surenb@google.com, yangx.jy@fujitsu.com, ying.huang@intel.com, ziy@nvidia.com, 42.hyeyoo@gmail.com, art.jeongseob@gmail.com Subject: Re: [RFC PATCH v3 0/7] DAMON based tiered memory management for CXL memory Date: Fri, 5 Apr 2024 12:28:00 -0700 Message-Id: <20240405192800.67163-1-sj@kernel.org> X-Mailer: git-send-email 2.39.2 In-Reply-To: <20240405060858.2818-1-honggyu.kim@sk.com> References: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Rspam-User: X-Stat-Signature: ckxk8pa43gr577r6kawxmcjonc96n5jt X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 4EA35C000B X-HE-Tag: 1712345285-631887 X-HE-Meta: U2FsdGVkX18o5M8OOZ/XS5G1z9paDJtHgNWbaL0k5pDq7NOexln1yg5CX5be4LkQZ7eMyejFwSE40QIe/TI+HpJH9KQCQ0T7NJYyoIf55xECNyjvf0iBYGV28qTg+ObQHXR5TDgkbCnZfZzdCsACEAHM1DoF0EWikmZMis92wF88Y7BrSzvIY715kzfSupNKLg0bqNsjpbXU+l4mClJLiAkWC07nP2w7cUl07hWvEZOWacEEY/3rJJL8Vr6lHn5BSE3jyn5u+/ZSoB75VVmumB1OiVtKpMG/7TwqjVU7RD0wTT+ecm3khXfMuq0vo2F1KJAliWS0BJccI9aZae/7dsXrVCpBC6CtE/bSkFqcrKwq/J/NUbSd1XGv/7vhppsT/EGktlGjWHEK4QHU62gh2r7v0T1hDR3nZfSivNz9aWrU+cEo31vrsBNtyFfRnbF9mPeKvxMktbAQb7GfNWV3y6lNZeconXwql5zDg7M70v3nExeC55QPlhLZw9u07ngeIkGMBz+EvQqcwz9OcJeZ44mSgr6DT8cKU0ImBLb5herwPUhg+Rk23gJKhrrCyeH1x6oqi3UYaOqHzujGnd8j9sktR7AveR7ckIX539zSFy6Yq14L5rekhjdQlgHYBzdQoFik9Uo513c6Apm7q8NtnY3WIQ5RkQ8YkO8gnKBnoTZ/bVYiiJKFoi7/NNQcbdztJSNJYz4kpQZGUlI9v6FQF6OnCsuL2dVakNpgR+lIGGxnpbbrScD0eDCX54QZKRclHUcDNW5km5F6vGY9nP2dcVJdoGXfAEGNEvqzlisMvaQ0a2GXrRk5GyNtJn6B7k7Tt6G2uPaA4PaSMqFbvrVMbt7Ls6j6yzSXxLNVgn8ikJYoRIL4dk/RrYmoffHLqFq8rAElotDNl6YzH1OmkVv2BZTZjUvv8Y/R7cBiOsmS9HRO9k/aYT5U1L2c/1LsvBN2yYe+9cY3sTVO4etFzh5 3g01xFCJ gQoK2p7LbOKMQSmjkVMmh0g8ymKJ+OJ96BoNTxjTRmcc9Q5Oe9ufmBNIu31DtlHaaVu87J22Zc8IszOg+Z8Uw3nj+Lwa1W6AdYqCyw4CfMRUGXUziPoQUJl2NfuPPbENdtRjCjBSfVQZrCx+NrRU4uYe3q7sgLhbrZO20yNDYcscUW6Bd4XRy4umT0UXlG3xqAo+x4Ehi0SOZcCMklHCRQPUwJyHOGoZgGZ/TdhyXsF5cJlg1zmJ7kDTjJ34g/BYpTBSVSZrNKWd4FbgKbJrQObbQO8lMCDI5qlnuur8Qw5rP/JSjHB+/KjE6tw== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000051, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Hello Honggyu, On Fri, 5 Apr 2024 15:08:49 +0900 Honggyu Kim wrote: > There was an RFC IDEA "DAMOS-based Tiered-Memory Management" previously > posted at [1]. > > It says there is no implementation of the demote/promote DAMOS action > are made. This RFC is about its implementation for physical address > space. > > > Changes from RFC v2: > 1. Rename DAMOS_{PROMOTE,DEMOTE} actions to DAMOS_MIGRATE_{HOT,COLD}. > 2. Create 'target_nid' to set the migration target node instead of > depending on node distance based information. > 3. Instead of having page level access check in this patch series, > delegate the job to a new DAMOS filter type YOUNG[2]. > 4. Introduce vmstat counters "damon_migrate_{hot,cold}". > 5. Rebase from v6.7 to v6.8. Thank you for patiently keeping discussion and making this great version! I left comments on each patch, but found no special concerns. Per-page access recheck for MIGRATE_HOT and vmstat change are taking my eyes, though. I doubt if those really needed. It would be nice if you could answer to the comments. Once my comments on this version are addressed, I would have no reason to object at dropping the RFC tag from this patchset. Nonetheless, I show some warnings and errors from checkpatch.pl. I don't really care about those for RFC patches, so no problem at all. But if you agree to my opinion about RFC tag dropping, and therefore if you will send next version without RFC tag, please make sure you also run checkpatch.pl before posting. Thanks, SJ [...]