linux-stable.git/drivers/staging/android, branch v3.3.6

android, lowmemorykiller: remove task handoff notifier

2012-04-13T16:13:55+00:00

commit 83dbbdbb38666e20a75fad2294cf1df77c52f121 upstream.

The task handoff notifier leaks task_struct since it never gets freed
after the callback returns NOTIFY_OK, which means it is responsible for
doing so.

It turns out the lowmemorykiller actually doesn't need this notifier at
all.  It's used to prevent unnecessary killing by waiting for a thread
to exit as a result of lowmem_shrink(), however, it's possible to do
this in the same way the kernel oom killer works by setting TIF_MEMDIE
and avoid killing if we're still waiting for it to exit.

The kernel oom killer will already automatically set TIF_MEMDIE for
threads that are attempting to allocate memory that have a fatal signal.
The thread selected by lowmem_shrink() will have such a signal after the
lowmemorykiller sends it a SIGKILL, so this won't result in an
unnecessary use of memory reserves for the thread to exit.

This has the added benefit that we don't have to rely on
CONFIG_PROFILING to prevent needlessly killing tasks.

Reported-by: Werner Landgraf 
Signed-off-by: David Rientjes 
Acked-by: Colin Cross 
Signed-off-by: Linus Torvalds 
Signed-off-by: Greg Kroah-Hartman

staging: android/lowmemorykiller: Don't unregister notifier from atomic context

2012-04-02T17:31:31+00:00

commit 1eda5166c7640092f512138be6899d050c3d62ed upstream.

The lowmemorykiller registers an atomic notifier for notfication of when
the task is freed.  From this atomic notifier callback, it removes the
atomic notifier via task_free_unregister().  This is incorrect because
atomic_notifier_chain_unregister() calls syncronize_rcu(), which can
sleep, which shouldn't be done from an atomic notifier.

Fix this by registering the notifier during init, and only unregister it
if the lowmemorykiller is unloaded.

Rebased to -next by Paul E. McKenney.
Rebased to -next again by Anton Vorontsov.

Signed-off-by: Rabin Vincent 
Signed-off-by: Christian Bejram 
Signed-off-by: Paul E. McKenney 
Reported-by: John Stultz 
Signed-off-by: Anton Vorontsov 
Signed-off-by: Greg Kroah-Hartman

staging: android/ram_console: Don't build on arches w/o ioremap

2012-02-08T22:14:15+00:00

This patch fixes UML build:

    CC      drivers/staging/android/ram_console.o
  drivers/staging/android/ram_console.c: In function
  'ram_console_driver_probe':
  drivers/staging/android/ram_console.c:358:2: error: implicit declaration
  of function 'ioremap' [-Werror=implicit-function-declaration]
  cc1: some warnings being treated as errors
  make[3]: *** [drivers/staging/android/ram_console.o] Error 1

Signed-off-by: Anton Vorontsov 
Signed-off-by: Greg Kroah-Hartman

staging: android: lowmemorykiller: Don't wait more than one second for a process to die

2012-02-08T22:14:14+00:00

If a process forked and the child process was killed by the
lowmemorykiller, the lowmemory killer would be disabled until
the parent process reaped the child or it died itself.

Signed-off-by: Arve Hjønnevåg 
Signed-off-by: Anton Vorontsov 
Signed-off-by: Greg Kroah-Hartman

Staging: android: binder: Fix crashes when sharing a binder file between processes

2012-02-08T22:14:13+00:00

Opening the binder driver and sharing the file returned with
other processes (e.g. by calling fork) can crash the kernel.
Prevent these crashes with the following changes:
- Add a mutex to protect against two processes mmapping the
  same binder_proc.
- After locking mmap_sem, check that the vma we want to access
  (still) points to the same mm_struct.
- Use proc->tsk instead of current to get the files struct since
  this is where we get the rlimit from.

Signed-off-by: Arve Hjønnevåg 
Cc: stable 
Signed-off-by: Greg Kroah-Hartman

Staging: android: Remove pmem driver

2012-02-08T22:14:13+00:00

Addroid pmem driver is no longer used in any of the Android products.
This patch removes pmem driver from Android staging area

Reference: https://lkml.org/lkml/2012/1/23/183

Signed-off-by: Shuah Khan 
Signed-off-by: Greg Kroah-Hartman

Staging: android: binder: Don't call dump_stack in binder_vma_open

2012-02-08T22:14:13+00:00

If user-space partially unmaps the driver, binder_vma_open
would dump the kernel stack. This is not a kernel bug however
and will be treated as if the whole area was unmapped once
binder_vma_close gets called.

Signed-off-by: Arve Hjønnevåg 
Cc: stable 
Signed-off-by: Greg Kroah-Hartman

Staging: android: fixed 80 characters warnings in lowmemorykiller.c

2011-12-22T21:33:57+00:00

This patch fixes some 80 chatacters limit warnings in the lowmemorykiller.c file

Signed-off-by: Marco Navarra 
Signed-off-by: Greg Kroah-Hartman

Staging: android: fixed a space warning in binder.h

2011-12-22T21:33:57+00:00

This patch fixes a simple tab-space warning in binder.h found by checkpatch tool

Signed-off-by: Marco Navarra 
Signed-off-by: Greg Kroah-Hartman

ashmem: Whitespace cleanups

2011-12-21T21:40:25+00:00

Fixes checkpatch warnings with the ashmem.c file

CC: Brian Swetland 
CC: Colin Cross 
CC: Arve Hjønnevåg 
CC: Dima Zavin 
CC: Robert Love 
Signed-off-by: John Stultz 
Signed-off-by: Greg Kroah-Hartman