| Message ID | 20250219220146.133650-1-sj@kernel.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | mm/damon: introduce DAMOS filter type for unmapped pages | expand |
Hi SeongJae, On 2/20/2025 7:01 AM, SeongJae Park wrote: > User decides whether their memory will be mapped or unmapped. It > implies that the two types of memory can have different characteristics > and management requirements. Provide the DAMON-observaibility > DAMOS-operation capability for the different types by introducing a new > DAMOS filter type for unmapped pages. I asked it before at https://github.com/damonitor/damo/issues/13 about monitoring unused paddr area but I see this patch series is related to applying DAMOS action. Regarding that, do you think we can skip those unused memory area using this filter before applying DAMOS action? I'm not sure if the current DAMOS tries pageout/migrate action for those unused area because they are detected as cold area although those will be imediately skiped inside action scheme. Thanks, Honggyu > > Changes from RFC > (https://lore.kernel.org/20241127205624.86986-1-sj@kernel.org) > - Rebase on latest mm-unstable > - Wordsmith commit message > - Add documentation > > SeongJae Park (2): > mm/damon: implement a new DAMOS filter type for unmapped pages > Docs/mm/damon/design: document unmapped DAMOS filter type > > Documentation/mm/damon/design.rst | 2 ++ > include/linux/damon.h | 2 ++ > mm/damon/paddr.c | 3 +++ > mm/damon/sysfs-schemes.c | 1 + > 4 files changed, 8 insertions(+) > > > base-commit: a2130e89cbd08ddb6f023b0b10eb87ebbc67add1
Hello Honggyu, On Thu, 20 Feb 2025 16:45:56 +0900 Honggyu Kim <honggyu.kim@sk.com> wrote: > Hi SeongJae, > > On 2/20/2025 7:01 AM, SeongJae Park wrote: > > User decides whether their memory will be mapped or unmapped. It > > implies that the two types of memory can have different characteristics > > and management requirements. Provide the DAMON-observaibility > > DAMOS-operation capability for the different types by introducing a new > > DAMOS filter type for unmapped pages. > > I asked it before at https://github.com/damonitor/damo/issues/13 about > monitoring unused paddr area but I see this patch series is related to > applying DAMOS action. My understanding of "unused" memory that you mentioned is memory that not allocated for user data. This memory cannot get accessed by user, so DAMON treats this memory as always not accessed. "unmapped" memory I'm saying about here is memory that not mapped to userspace. For example, unmapped pages in pge cache. This memory can be accessed, for eaxmple through page cache using read()/write() like system calls. So I think this is not really related with the GitHub discussion. Please let me know if I'm missing something. > > Regarding that, do you think we can skip those unused memory area using > this filter before applying DAMOS action? So, no. > > I'm not sure if the current DAMOS tries pageout/migrate action for those > unused area because they are detected as cold area although those will > be imediately skiped inside action scheme. This is the current behavior. DAMOS will try to do whatever action to whatever region if requested, see the action is not applicable to the page, and move on to the next page. Please note that filter-based page level skipping is not that different from the page level action applicability checking. If you know where "unused" memory located, and you want to make DAMON/S entirely ignore it, you could use DAMON target address range, or address range type DAMOS filter. I'm not sure if this is what you're looking for, though. My understanding of your concern about "unused" memory on the GitHub discussion was more about adaptive reions adjustment efficiency. I recently posted[1] some ideas to improve the mechanism. Any feedback about the idea will be welcomed and help prioritizing it. [1] https://lore.kernel.org/20250213222303.244724-1-sj@kernel.org Thanks, SJ [...]