Prev: [PATCH] staging:cx25821: fixes coding styles issues in cx25821-gpio.c This patch is to the cx25821-gpio.c file that fixes up warning and error found by the checkpatch.pl tool Signed-off-by: sai <tsg321@gmail.com>
Next: Enhance perf to collect KVM guest os statistics from host side
From: KOSAKI Motohiro on 16 Mar 2010 01:50
> ------------------------------------------------------
> Subject: tmpfs: fix oops on remounts with mpol=default
> From: Ravikiran G Thirumalai <kiran(a)scalex86.org>
>
> Fix an 'oops' when a tmpfs mount point is remounted with the 'default'
> mempolicy.
>
> Upon remounting a tmpfs mount point with 'mpol=default' option, the
> remount code crashed with a null pointer dereference. The initial problem
> report was on 2.6.27, but the problem exists in mainline 2.6.34-rc as
> well. On examining the code, we see that mpol_new returns NULL if default
> mempolicy was requested. This 'NULL' mempolicy is accessed to store the
> node mask resulting in oops.
>
> The following patch fixes the oops by avoiding dereferencing NULL if the
> new mempolicy is NULL. The patch also sets 'err' to 0 if MPOL_DEFAULT is
> passed (err is initialized to 1 initially at mpol_parse_str())

Hi Ravikiran,

I'm glad to your contribution. Unfortunately I've found various related
issue in mpol_parse_str() while reviewing your patch.

So, I'll post updated patches.

- kosaki


>
> Signed-off-by: Ravikiran Thirumalai <kiran(a)scalex86.org>
> Cc: KOSAKI Motohiro <kosaki.motohiro(a)jp.fujitsu.com>
> Cc: Christoph Lameter <cl(a)linux-foundation.org>
> Cc: Mel Gorman <mel(a)csn.ul.ie>
> Cc: Lee Schermerhorn <lee.schermerhorn(a)hp.com>
> Cc: Hugh Dickins <hugh.dickins(a)tiscali.co.uk>
> Cc: <stable(a)kernel.org>
> Signed-off-by: Andrew Morton <akpm(a)linux-foundation.org>
> ---
>
> mm/mempolicy.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
>
> diff -puN mm/mempolicy.c~tmpfs-fix-oops-on-remounts-with-mpol=default mm/mempolicy.c
> --- a/mm/mempolicy.c~tmpfs-fix-oops-on-remounts-with-mpol=default
> +++ a/mm/mempolicy.c
> @@ -2213,10 +2213,14 @@ int mpol_parse_str(char *str, struct mem
> goto out;
> mode = MPOL_PREFERRED;
> break;
> -
> + case MPOL_DEFAULT:
> + /*
> + * mpol_new() enforces empty nodemask, ignores flags.
> + */
> + err = 0;
> + break;
> /*
> * case MPOL_BIND: mpol_new() enforces non-empty nodemask.
> - * case MPOL_DEFAULT: mpol_new() enforces empty nodemask, ignores flags.
> */
> }
>
> @@ -2250,7 +2254,7 @@ int mpol_parse_str(char *str, struct mem
> if (ret) {
> err = 1;
> mpol_put(new);
> - } else if (no_context) {
> + } else if (no_context && new) {
> /* save for contextualization */
> new->w.user_nodemask = nodes;
> }
> _
>
> Patches currently in -mm which might be from kiran(a)scalex86.org are
>
> tmpfs-fix-oops-on-remounts-with-mpol=default.patch
> slab-leaks3-default-y.patch
>



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: [PATCH] staging:cx25821: fixes coding styles issues in cx25821-gpio.c This patch is to the cx25821-gpio.c file that fixes up warning and error found by the checkpatch.pl tool Signed-off-by: sai <tsg321@gmail.com>
Next: Enhance perf to collect KVM guest os statistics from host side