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=-8.4 required=3.0 tests=DKIMWL_WL_MED,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,USER_IN_DEF_DKIM_WL autolearn=no 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 59FFFC3F2CD for ; Mon, 23 Mar 2020 17:29:38 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2044B20722 for ; Mon, 23 Mar 2020 17:29:38 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="FxgC6y+z" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727461AbgCWR3h (ORCPT ); Mon, 23 Mar 2020 13:29:37 -0400 Received: from mail-oi1-f195.google.com ([209.85.167.195]:38557 "EHLO mail-oi1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727112AbgCWR3h (ORCPT ); Mon, 23 Mar 2020 13:29:37 -0400 Received: by mail-oi1-f195.google.com with SMTP id w2so4567158oic.5 for ; Mon, 23 Mar 2020 10:29:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=coauMnrl6/+PptbG82eRore8posxzJJD9TGT8ilHcxo=; b=FxgC6y+z2lfFKzKhX7N23ChynlrKNVvuzpjrWiLCulaYwMp10qLX9MbPhOFlnq8jsW sY3v4tsVnUxdMrjfPutOxtKBw4dbfXW5BX9PL2JxLFcGsvK8/WKfuMJtt6rN1xCGutDh WU0JVZLs+U+oxJsHHFGfuLL8MQZPfbsnM+0bV5pzLxjKRKu5+I6uP2qLOvZBCz+7cQvf 7+Gp9ZEUDeHD3Cul07BSxKygxabkZi0DRkc5HwQB25pLtGtSWWUlKkj4qQV1plq1XkcF Z0OagJGhnq5m8J/JxzRz+LYI+o1+nnOjJtSUXjDLipAO5Ho6xaCSRdA+LZ7vZiUt4bHR AZhA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=coauMnrl6/+PptbG82eRore8posxzJJD9TGT8ilHcxo=; b=I3/h0Pm9VpAXH076eS3dwKdE/TPb7CzUin2Vn6BJUjcXHqdL2aw/CThRfQrif5ATci MYbDVnFfZ5SKAwZHb6glhKJZ3fx2ZFeMeqDU1yNlsvQ2/3h/PbYL+sldJRMRq4vURWXf wDzlfM0xXBsAdokd4tHMMhm0k/F52GM53vXZs+EVV1kuPrpEmdfGkhMnpbWuvf8UBXiJ f3OtbhvEWHwgqhPFnplv0Ct1wkb+Vx/K63ZS46qA734GcRJ6lAw9wwSrggn74YERkj0W +nCJrfWeIyaBL16U2ldarGGsY/6e3gHMAL7DMPNIpj8VN1RJ6vjETMAyk4Lb8RrEQX0O PIXg== X-Gm-Message-State: ANhLgQ1gNYytI/td5JFd3tpKzVnlhspZ6e/DddBjWTOxptg2tNG/SrWX sNRfyHUTSfim1lZ3T2sDQHF5A7YRJFYZlzz4HOKNsQ== X-Google-Smtp-Source: ADFU+vvnpd1CGPY5YrzOYcSAmKn5g0ovK39HJgfqd1R3bB1CCa9F8gGIz0jAEgWdK2oFXBLLC17nE5VYLM3curbyt3k= X-Received: by 2002:aca:ed54:: with SMTP id l81mr315133oih.69.1584984576071; Mon, 23 Mar 2020 10:29:36 -0700 (PDT) MIME-Version: 1.0 References: <20200319090301.1038-1-sjpark@amazon.com> In-Reply-To: <20200319090301.1038-1-sjpark@amazon.com> From: Shakeel Butt Date: Mon, 23 Mar 2020 10:29:24 -0700 Message-ID: Subject: Re: Re: Re: Re: [PATCH v6 00/14] Introduce Data Access MONitor (DAMON) To: SeongJae Park Cc: Andrew Morton , SeongJae Park , Andrea Arcangeli , Yang Shi , acme@kernel.org, alexander.shishkin@linux.intel.com, amit@kernel.org, brendan.d.gregg@gmail.com, Brendan Higgins , Qian Cai , Colin Ian King , Jonathan Corbet , dwmw@amazon.com, jolsa@redhat.com, "Kirill A. Shutemov" , mark.rutland@arm.com, Mel Gorman , Minchan Kim , Ingo Molnar , namhyung@kernel.org, peterz@infradead.org, Randy Dunlap , David Rientjes , Steven Rostedt , shuah@kernel.org, sj38.park@gmail.com, Vlastimil Babka , Vladimir Davydov , Linux MM , linux-doc@vger.kernel.org, LKML Content-Type: text/plain; charset="UTF-8" Sender: linux-doc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org On Thu, Mar 19, 2020 at 2:04 AM SeongJae Park wrote: > > On Wed, 18 Mar 2020 12:52:48 -0700 Shakeel Butt wrote: > > > On Thu, Mar 12, 2020 at 3:44 AM SeongJae Park wrote: > > > > > > On Thu, 12 Mar 2020 11:07:59 +0100 SeongJae Park wrote: > > > > > > > On Tue, 10 Mar 2020 10:21:34 -0700 Shakeel Butt wrote: > > > > > > > > > On Mon, Feb 24, 2020 at 4:31 AM SeongJae Park wrote: > > > > > > > > > > > > From: SeongJae Park > > > > > > > > > > > > Introduction > > > > > > ============ > > > > > > > > > [...] > > > > > > > > > > I do want to question the actual motivation of the design followed by this work. > > > > > > > > > > With the already present Page Idle Tracking feature in the kernel, I > > > > > can envision that the region sampling and adaptive region adjustments > > > > > can be done in the user space. Due to sampling, the additional > > > > > overhead will be very small and configurable. > > > > > > > > > > Additionally the proposed mechanism has inherent assumption of the > > > > > presence of spatial locality (for virtual memory) in the monitored > > > > > processes which is very workload dependent. > > > > > > > > > > Given that the the same mechanism can be implemented in the user space > > > > > within tolerable overhead and is workload dependent, why it should be > > > > > done in the kernel? What exactly is the advantage of implementing this > > > > > in kernel? > > > > > > > > First of all, DAMON is not for only user space processes, but also for kernel > > > > space core mechanisms. Many of the core mechanisms will be able to use DAMON > > > > for access pattern based optimizations, with light overhead and reasonable > > > > accuracy. > > > > Which kernel space core mechanisms? I can see memory reclaim, do you > > envision some other component as well. > > In addition to reclmation, I am thinking THP promotion/demotion decision, page > migration among NUMA nodes on tier-memory configuration, and on-demand virtual > machine live migration mechanisms could benefit from DAMON, for now. I also > believe more use-cases could be found. > I am still struggling to see how these use-cases require in-kernel DAMON. For THP promotion/demotaion, madvise(MADV_[NO]HUGEPAGE) can be used or we can introduce a new MADV_HUGIFY to synchronously convert small pages to hugepages. Page migration on tier-memory is similar to proactive reclaim and we already have migrate_pages/move_pages syscalls. Basically why userspace DAMON can not perform these operations? > > > > Let's discuss how this can interact with memory reclaim and we can see > > if there is any benefit to do this in kernel. > > For reclaim, I believe we could try the proactive reclamation again using DAMON > (Yes, I'm a fan of the idea of proactive reclamation). I already implemented > and evaluated a simple form of DAMON-based proactive reclamation for the proof > of the concept (not for production). In best case (parsec3/freqmine), it > reduces 22.42% of system memory usage and 88.86% of residential sets while > incurring only 3.07% runtime overhead. Please refer to 'Appendix E' of the v7 > patchset[1] of DAMON. It also describes the implementation and the evaluation > of a data access monitoring-based THP promotion/demotion policy. > > The experimental implementation cannot be directly applied to kernel > reclamation mechanism, because it requires users to specify the target > applications. Nonetheless, I think we can also easily adopt it inside the > kernel by modifying kswapd to periodically select processes having huge RSS as > targets, or by creating proactive reclamation type cgroups which selects every > processes in the cgroup as targets. Again I feel like these should be done in user space as these are more policies instead of mechanisms. However if it can be shown that doing that in userspace is very expensive as compared to in-kernel solution then we can think about it. > > Of course, we can extend DAMON to support physical memory address space instead > of virtual memory of specific processes. Actually, this is in our TODO list. > With the extension, applying DAMON to core memory management mechanisms will be > even easier. See below on physical memory monitoring. > > Nonetheless, this is only example but not concrete plan. I didn't make the > concrete plan yet, but believe that of DAMON will open the gates. > > [1] https://lore.kernel.org/linux-mm/20200318112722.30143-1-sjpark@amazon.com/ > > > > > > > > > > > Implementing DAMON in user space is of course possible, but it will be > > > > inefficient. Using it from kernel space would make no sense, and it would > > > > incur unnecessarily frequent kernel-user context switches, which is very > > > > expensive nowadays. > > > > > > Forgot mentioning about the spatial locality. Yes, it is workload dependant, > > > but still pervasive in many case. Also, many core mechanisms in kernel such as > > > read-ahead or LRU are already using some similar assumptions. > > > > > > > Not sure about the LRU but yes read-ahead in several places does > > assume spatial locality. However most of those are configurable and > > the userspace can enable/disable the read-ahead based on the workload. > > Sorry for my ambiguous description. LRU uses temporal locality, which is > somewhat similar to spatial locality, in terms of workload dependency. > > > > > > > > > If it is so problematic, you could set the maximum number of regions to the > > > number of pages in the system so that each region monitors each page. > > > > > > > How will this work in the process context? Number of regions equal to > > the number of mapped pages? > > Suppose that a process has 1024 pages of working set and each of the pages has > totally different access frequency. If the maximum number of regions is 1024, > the adaptive regions adjustment mechanism of DAMON will create each region for > each page and monitor the access to each page. So, the output will be same to > straightforward periodic page-granularity access checking methods, which does > not depend on the spatial locality. Nevertheless, the monitoring overhead will > be also similar to that. > > However, if any adjacent pages have similar access frequencies, DAMON will > group those pages into one region. This will reduce the total number of PTE > Accessed bit checks and thus decrease the overhead. In other words, DAMON do > its best effort to minimize the overhead while preserving quality. > > Also suppose that the maximum number of region is smaller than 1024 in this > case. Pages having different access frequency will be grouped in same region > and thus the output quality will be decreased. However, the overhead will > be half, as DAMON does one access check per each region. This means that you > can easily trade the monitoring quality with overhead by adjusting the maximum > number of regions. > So, users can select to not merge the regions to keep the monitoring quality high, right? > > > > Basically I am trying to envision the comparison of physical memory > > based monitoring (using idle page tracking) vs pid+VA based > > monitoring. > > I believe the core mechanisms of DAMON could be easily extended to the physical > memory. Indeed, it is in our TODO list, and I believe it would make use of > DAMON in kernel core mechanisms much easier. > How will the sampling and regions representation/resizing work in physical memory? > > > > Anyways I am not against your proposal. I am trying to see how to make > > it more general to be applicable to more use-cases and one such > > use-case which I am interested in is monitoring all the user pages on > > the system for proactive reclaim purpose. > > Your questions gave me many insight and shed lights to the way DAMON should go. > Really appreciate. If you have any more questions or need my help, please let > me know. > > Shakeel