[PATCH] do_wp_page: BUG on invalid pfn

[Ed L Cashin]

Rusty Russell <rusty@xxxxxxxxxxxxxxx> writes:

> In message <87d6fixvpm.fsf@xxxxxxx> you write:
>> This patch just does what the comment says should be done.
>
> Hi Ed!
>
> 	Not trivial I'm afraid.  Send to Linus and lkml.


This patch just does what the comment says should be done.  I thought
it was a trivial patch, but Rusty Russell has informed me otherwise.
(Thanks, RR).


--- linux-2.6.0-test2/mm/memory.c.orig	Sun Jul 27 13:01:24 2003
+++ linux-2.6.0-test2/mm/memory.c	Wed Aug  6 18:30:55 2003
@@ -990,15 +990,10 @@
 	int ret;
 
 	if (unlikely(!pfn_valid(pfn))) {
-		/*
-		 * This should really halt the system so it can be debugged or
-		 * at least the kernel stops what it's doing before it corrupts
-		 * data, but for the moment just pretend this is OOM.
-		 */
-		pte_unmap(page_table);
 		printk(KERN_ERR "do_wp_page: bogus page at address %08lx\n",
 				address);
-		goto oom;
+		dump_stack();
+		BUG();
 	}
 	old_page = pfn_to_page(pfn);
 
@@ -1054,7 +1049,6 @@
 
 no_mem:
 	page_cache_release(old_page);
-oom:
 	ret = VM_FAULT_OOM;
 out:
 	spin_unlock(&mm->page_table_lock);

-- 
--Ed L Cashin            |   PGP public key:
  ecashin@xxxxxxx        |   http://noserose.net/e/pgp/

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/