[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <47108ae5-fc73-432f-9942-c2fe5fbf4bb5@kernel.org>
Date: Mon, 19 Jan 2026 23:44:47 +0100
From: "David Hildenbrand (Red Hat)" <david@...nel.org>
To: Lorenzo Stoakes <lorenzo.stoakes@...cle.com>
Cc: linux-kernel@...r.kernel.org, linux-mm@...ck.org,
linuxppc-dev@...ts.ozlabs.org,
Broadcom internal kernel review list
<bcm-kernel-feedback-list@...adcom.com>, linux-doc@...r.kernel.org,
virtualization@...ts.linux.dev, Andrew Morton <akpm@...ux-foundation.org>,
Oscar Salvador <osalvador@...e.de>, "Liam R. Howlett"
<Liam.Howlett@...cle.com>, Vlastimil Babka <vbabka@...e.cz>,
Mike Rapoport <rppt@...nel.org>, Suren Baghdasaryan <surenb@...gle.com>,
Michal Hocko <mhocko@...e.com>, Jonathan Corbet <corbet@....net>,
Madhavan Srinivasan <maddy@...ux.ibm.com>,
Michael Ellerman <mpe@...erman.id.au>, Nicholas Piggin <npiggin@...il.com>,
Christophe Leroy <christophe.leroy@...roup.eu>, Arnd Bergmann
<arnd@...db.de>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Jerrin Shaji George <jerrin.shaji-george@...adcom.com>,
"Michael S. Tsirkin" <mst@...hat.com>, Jason Wang <jasowang@...hat.com>,
Xuan Zhuo <xuanzhuo@...ux.alibaba.com>, Eugenio Pérez
<eperezma@...hat.com>, Zi Yan <ziy@...dia.com>
Subject: Re: [PATCH v2 03/23] powerpc/pseries/cmm: remove
cmm_balloon_compaction_init()
>> @@ -573,11 +567,12 @@ static int cmm_init(void)
>> return -EOPNOTSUPP;
>>
>> balloon_devinfo_init(&b_dev_info);
>> - cmm_balloon_compaction_init();
>> + if (IS_ENABLED(CONFIG_BALLOON_COMPACTION))
>> + b_dev_info.migratepage = cmm_migratepage;
>>
>> rc = register_oom_notifier(&cmm_oom_nb);
>> if (rc < 0)
>> - goto out_balloon_compaction;
>> + return rc;
>>
>> if ((rc = register_reboot_notifier(&cmm_reboot_nb)))
>> goto out_oom_notifier;
>> @@ -606,7 +601,6 @@ static int cmm_init(void)
>> unregister_reboot_notifier(&cmm_reboot_nb);
>> out_oom_notifier:
>> unregister_oom_notifier(&cmm_oom_nb);
>> -out_balloon_compaction:
>
> So silly, I assume before there was more that happened here?
Right, I commented that above in the patch description.
Thanks!
--
Cheers
David
Powered by blists - more mailing lists