linux-stable.git/include/linux/gfp.h, branch linux-2.6.37.y

PM / Hibernate: Fix memory corruption related to swap

2010-12-06T22:52:08+00:00

There is a problem that swap pages allocated before the creation of
a hibernation image can be released and used for storing the contents
of different memory pages while the image is being saved.  Since the
kernel stored in the image doesn't know of that, it causes memory
corruption to occur after resume from hibernation, especially on
systems with relatively small RAM that need to swap often.

This issue can be addressed by keeping the GFP_IOFS bits clear
in gfp_allowed_mask during the entire hibernation, including the
saving of the image, until the system is finally turned off or
the hibernation is aborted.  Unfortunately, for this purpose
it's necessary to rework the way in which the hibernate and
suspend code manipulates gfp_allowed_mask.

This change is based on an earlier patch from Hugh Dickins.

Signed-off-by: Rafael J. Wysocki 
Reported-by: Ondrej Zary 
Acked-by: Hugh Dickins 
Reviewed-by: KAMEZAWA Hiroyuki 
Cc: stable@kernel.org

mm: fix sparse warnings on GFP_ZONE_TABLE/BAD

2010-10-26T23:52:10+00:00

Introduce ___GFP_* masks in order for gfp_t to not be mixed with plain
integers which causes a lot of warnings like the following:

 warning: restricted gfp_t degrades to integer

Signed-off-by: Namhyung Kim 
Cc: Al Viro 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

include/linux/gfp.h: fix coding style

2010-05-25T15:07:01+00:00

Add parenthesis in a define.  This doesn't change functionality.

checkpatch errors:
1) white space fixes
2) add spaces after comas

Signed-off-by: matt mooney 
Cc: Dan Carpenter 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

include/linux/gfp.h: spelling fixes

2010-05-25T15:07:00+00:00

Fix minor spelling errors in a few comments; no code changes.

Signed-off-by: matt mooney 
Cc: Dan Carpenter 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm: add comment about deprecation of __GFP_NOFAIL

2010-03-06T19:26:27+00:00

__GFP_NOFAIL was deprecated in dab48dab, so add a comment that no new
users should be added.

Reviewed-by: KAMEZAWA Hiroyuki 
Signed-off-by: David Rientjes 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm/pm: force GFP_NOIO during suspend/hibernation and resume

2010-03-06T19:26:26+00:00

There are quite a few GFP_KERNEL memory allocations made during
suspend/hibernation and resume that may cause the system to hang, because
the I/O operations they depend on cannot be completed due to the
underlying devices being suspended.

Avoid this problem by clearing the __GFP_IO and __GFP_FS bits in
gfp_allowed_mask before suspend/hibernation and restoring the original
values of these bits in gfp_allowed_mask durig the subsequent resume.

[akpm@linux-foundation.org: fix CONFIG_PM=n linkage]
Signed-off-by: Rafael J. Wysocki 
Reported-by: Maxim Levitsky 
Cc: Sebastian Ott 
Cc: Benjamin Herrenschmidt 
Cc: KOSAKI Motohiro 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

mm: remove free_hot_page()

2010-03-06T19:26:25+00:00

free_hot_page() is just a wrapper around free_hot_cold_page() with
parameter 'cold = 0'.  After adding a clear comment for
free_hot_cold_page(), it is reasonable to remove a level of call.

[akpm@linux-foundation.org: fix build]
Signed-off-by: Li Hong 
Cc: Mel Gorman 
Cc: Rik van Riel 
Cc: Ingo Molnar 
Cc: Larry Woodman 
Cc: Peter Zijlstra 
Cc: Li Ming Chun 
Cc: KOSAKI Motohiro 
Cc: Americo Wang 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

BUILD_BUG_ON(): fix it and a couple of bogus uses of it

2009-09-23T14:39:29+00:00

gcc permitting variable length arrays makes the current construct used for
BUILD_BUG_ON() useless, as that doesn't produce any diagnostic if the
controlling expression isn't really constant.  Instead, this patch makes
it so that a bit field gets used here.  Consequently, those uses where the
condition isn't really constant now also need fixing.

Note that in the gfp.h, kmemcheck.h, and virtio_config.h cases
MAYBE_BUILD_BUG_ON() really just serves documentation purposes - even if
the expression is compile time constant (__builtin_constant_p() yields
true), the array is still deemed of variable length by gcc, and hence the
whole expression doesn't have the intended effect.

[akpm@linux-foundation.org: make arch/sparc/include/asm/vio.h compile]
[akpm@linux-foundation.org: more nonsensical assertions in tpm.c..]
Signed-off-by: Jan Beulich 
Cc: Andi Kleen 
Cc: Rusty Russell 
Cc: Catalin Marinas 
Cc: "David S. Miller" 
Cc: Rajiv Andrade 
Cc: Mimi Zohar 
Cc: James Morris 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

oom: move oom_killer_enable()/oom_killer_disable to where they belong

2009-09-22T14:17:38+00:00

Signed-off-by: Alexey Dobriyan 
Acked-by: David Rientjes 
Reviewed-by: KOSAKI Motohiro 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

page-allocator: remove dead function free_cold_page()

2009-09-22T14:17:34+00:00

The function free_cold_page() has no callers so delete it.

Signed-off-by: Mel Gorman 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds