Patch [3/4] x86_64 sparsmem add - acpi added pages are not reserved?

I got everything in place to do a working hot-add with 2.6.17-mm1 and I ran into another trouble spot.

I added the memory just fine(new device is sysfs) but when I went to on- line them in sysfs I tripped over the little section of code this patch comments out. I was get device not ready messages on my console and the comment printed in my kernel log.

With hacked acpi drivers outside of -mm I don't run into the problem
so I think something is a little off in -mm.

The code is expecting the added but not on-lined code to be marked
reserved. This isn't happening for my ACPI hot-add on x86_64. I am not sure who in this call path needs to reserve the pages or if the check for reserve is a valid with this new hot-add code.

Any ideas?

diff -urN linux-2.6.17-mm1-orig/drivers/base/memory.c linux-2.6.17-mm1/drivers/base/memory.c
@@ -163,9 +163,9 @@
 	/*
 	 * The probe routines leave the pages reserved, just
 	 * as the bootmem code does.  Make sure they're still
-	 * that way.
+	 * that way.  UNLESS you do real hot add? 
 	 */
-	if (action == MEM_ONLINE) {
+/*	if (action == MEM_ONLINE) {
 		for (i = 0; i < PAGES_PER_SECTION; i++) {
 			if (PageReserved(first_page+i))
 				continue;

@@ -176,6 +176,7 @@
 			return -EBUSY;
 		}
 	}
+*/
 
 	switch (action) {
 		case MEM_ONLINE:

/* if (action == MEM_ONLINE) {
for (i = 0; i < PAGES_PER_SECTION; i++) {
if (PageReserved(first_page+i))
continue;
@@ -176,6 +176,7 @@
return -EBUSY;
}
}
+*/
Pages are marked as Reserved before onlined. Then, all pages in the area should be reserved.(and sparsemem allocates SECTION_SIZE aligned memmap.) (see __add_zone() -> memmap_init_zone().)
newly initialized memmap are marked as reserved.
Plz confirm your "currently unused memmap" is properly marked as reserved.

It's important to find "why" before doing this kind of workaround.

Hmm... at first, could you show your /proc/iomem before and after hot-add event ?

Thanks,
-Kame

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"[email protected]"> [email protected] </a>