Re: 2.6.23-rc4-mm1

From: Valdis . Kletnieks
Date: Wed Sep 05 2007 - 11:58:21 EST


On Wed, 05 Sep 2007 08:12:07 PDT, Andrew Morton said:

> Seems to me that there's a missing pte_unmap_lock() in insert_page().
>
> Also, a hunk in do_anonymous_page() is indented one tabstop too far, which
> makes me suspect that patch(1) might have put it in the wrong place.
> Balbir, can you please check that?
>
> diff -puN mm/memory.c~memory-controller-memory-accounting-v7-fix mm/memory.c
> --- a/mm/memory.c~memory-controller-memory-accounting-v7-fix
> +++ a/mm/memory.c
> @@ -1135,7 +1135,7 @@ static int insert_page(struct mm_struct

Confirming that this patch fixes things.

Tested-by: Valdis Kletnieks <valdis.kletnieks@xxxxxx>

Attachment: pgp00000.pgp
Description: PGP signature