* [PATCH] mm: compaction: use helper isolation_suitable
@ 2022-03-22 11:07 Miaohe Lin
2022-03-26 9:43 ` Wei Yang
0 siblings, 1 reply; 2+ messages in thread
From: Miaohe Lin @ 2022-03-22 11:07 UTC (permalink / raw)
To: akpm; +Cc: linux-mm, linux-kernel, linmiaohe
Use helper isolation_suitable to check whether page is suitable to isolate
to simplify the code. Minor readability improvement.
Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
---
mm/compaction.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/mm/compaction.c b/mm/compaction.c
index c3e37aa9ff9e..86f34053c39d 100644
--- a/mm/compaction.c
+++ b/mm/compaction.c
@@ -899,7 +899,7 @@ isolate_migratepages_block(struct compact_control *cc, unsigned long low_pfn,
* not falsely conclude that the block should be skipped.
*/
if (!valid_page && IS_ALIGNED(low_pfn, pageblock_nr_pages)) {
- if (!cc->ignore_skip_hint && get_pageblock_skip(page)) {
+ if (!isolation_suitable(cc, page)) {
low_pfn = end_pfn;
page = NULL;
goto isolate_abort;
--
2.23.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] mm: compaction: use helper isolation_suitable
2022-03-22 11:07 [PATCH] mm: compaction: use helper isolation_suitable Miaohe Lin
@ 2022-03-26 9:43 ` Wei Yang
0 siblings, 0 replies; 2+ messages in thread
From: Wei Yang @ 2022-03-26 9:43 UTC (permalink / raw)
To: Miaohe Lin; +Cc: akpm, linux-mm, linux-kernel
On Tue, Mar 22, 2022 at 07:07:50PM +0800, Miaohe Lin wrote:
>Use helper isolation_suitable to check whether page is suitable to isolate
>to simplify the code. Minor readability improvement.
>
>Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
Reviewed-by: Wei Yang <richard.weiyang@gmail.com>
>---
> mm/compaction.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/mm/compaction.c b/mm/compaction.c
>index c3e37aa9ff9e..86f34053c39d 100644
>--- a/mm/compaction.c
>+++ b/mm/compaction.c
>@@ -899,7 +899,7 @@ isolate_migratepages_block(struct compact_control *cc, unsigned long low_pfn,
> * not falsely conclude that the block should be skipped.
> */
> if (!valid_page && IS_ALIGNED(low_pfn, pageblock_nr_pages)) {
>- if (!cc->ignore_skip_hint && get_pageblock_skip(page)) {
>+ if (!isolation_suitable(cc, page)) {
> low_pfn = end_pfn;
> page = NULL;
> goto isolate_abort;
>--
>2.23.0
--
Wei Yang
Help you, Help me
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-03-26 9:43 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-22 11:07 [PATCH] mm: compaction: use helper isolation_suitable Miaohe Lin
2022-03-26 9:43 ` Wei Yang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox