Fix missing last node

This patch fixes a typo which results in duplication of the first node (and the last node was missing then).

---
 makedumpfile.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

@@ -2278,7 +2278,7 @@ get_mm_discontigmem(struct DumpInfo *inf
 			if ((node = next_online_node(node + 1)) < 0) {
 				ERRMSG("Can't get next online node.\n");
 				return FALSE;
-			} else if (!(pgdat = next_online_pgdat(info, node))) {
+			} else if (!(pgdat = next_online_pgdat(info, node + 1))) {
 				ERRMSG("Can't determine pgdat list (node %d).\n",
 				    node);
 				return FALSE;

_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec

Hi Bernhard,

Thank you for your patches and your cooperation.

I think the cause of this problem is the other typo. Does the attached patch fix it ?

Please apply the attached patch with the following makedumpfile:

makedumpfile-1.1.3
+ 2007/05/11 by Ken'ichi, 01-vaddr_to_offset_ia64.patch
+ 2007/05/11 by Ken'ichi, 02-ia64-discontigmem.patch
+ 2007/05/16 by Bernhard, [Re: Implement memory regions on IA64] patch

Thanks
Ken'ichi Ohmichi

diff -puN makedumpfile.org/makedumpfile.c makedumpfile/makedumpfile.c
@@ -2121,7 +2121,7 @@ pgdat3:
 	if (node == 0)
 		return pgdat;
 
-	for (i = 1; i < node; i++) {
+	for (i = 1; i <= node; i++) {
 		if (!readmem(info, pgdat + OFFSET(pglist_data.pgdat_next),
 		    &pgdat, sizeof pgdat))
 			goto pgdat4;

_

_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec

Yes, thanks!

Bernhard

_______________________________________________
kexec mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/kexec