[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAA25o9QUtut3+nEs0H8H5qa2H7tQokq+_UoOrAaVGhmYvMTz0Q@mail.gmail.com>
Date: Mon, 8 Jun 2020 15:23:22 -0700
From: Luigi Semenzato <semenzato@...gle.com>
To: Andrea Righi <andrea.righi@...onical.com>
Cc: Pavel Machek <pavel@....cz>,
linux-kernel <linux-kernel@...r.kernel.org>,
Linux Memory Management List <linux-mm@...ck.org>,
Linux PM <linux-pm@...r.kernel.org>,
Andrew Morton <akpm@...ux-foundation.org>,
Len Brown <len.brown@...el.com>,
"Rafael J . Wysocki" <rjw@...ysocki.net>
Subject: Re: [RFC PATCH 2/2] PM: hibernate: introduce opportunistic memory reclaim
Hi Andrea,
1. This mechanism is quite general. It is possible that, although
hibernation may be an important use, there will be other uses for it.
I suggest leaving the hibernation example and performance analysis,
but not mentioning PM or hibernation in the patch subject.
2. It may be useful to have run_show() return the number of pages
reclaimed in the last attempt. (I had suggested something similar in
https://lore.kernel.org/linux-mm/CAA25o9SxajRaa+ZyhvTYdaKdXokcrNYXgEUimax4sUJGCmRYLA@mail.gmail.com/).
3. It is not clear how much mm_reclaim/release is going to help. If
the preloading of the swapped-out pages uses some kind of LIFO order,
and can batch multiple pages, then it might help. Otherwise demand
paging is likely to be more effective. If the preloading does indeed
help, it may be useful to explain why in the commit message.
Thanks!
Powered by blists - more mailing lists