Thread (10 messages) 10 messages, 4 authors, 2019-10-08

Re: [PATCH V8 2/2] arm64/mm: Enable memory hot remove

From: Matthew Wilcox <willy@infradead.org>
Date: 2019-09-23 11:17:52
Also in: linux-mm, lkml

On Mon, Sep 23, 2019 at 11:13:45AM +0530, Anshuman Khandual wrote:
+#ifdef CONFIG_MEMORY_HOTPLUG
+static void free_hotplug_page_range(struct page *page, size_t size)
+{
+	WARN_ON(!page || PageReserved(page));
WARN_ON(!page) isn't terribly useful.  You're going to crash on the very
next line when you call page_address() anyway.  If this line were

	if (WARN_ON(!page || PageReserved(page)))
		return;

it would make sense, or if it were just

	WARN_ON(PageReserved(page))

it would also make sense.
+	free_pages((unsigned long)page_address(page), get_order(size));
+}
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help