Messages in this thread | | | Date | Mon, 6 Feb 2012 20:42:15 +0400 | From | Anton Vorontsov <> | Subject | Re: [PATCH 3/6] staging: android/lowmemorykiller: Don't grab tasklist_lock |
| |
On Mon, Feb 06, 2012 at 08:36:49AM -0800, Greg KH wrote: > On Mon, Feb 06, 2012 at 08:29:41PM +0400, Anton Vorontsov wrote: > > Grabbing tasklist_lock has its disadvantages, i.e. it blocks > > process creation and destruction. If there are lots of processes, > > blocking doesn't sound as a great idea. > > > > For LMK, it is sufficient to surround tasks list traverse with > > rcu_read_{,un}lock(). > > > > >From now on using force_sig() is not safe, as it can race with an > > already exiting task, so we use send_sig() now. As a downside, it > > won't kill PID namespace init processes, but that's not what we > > want anyway. > > > > Suggested-by: Oleg Nesterov <oleg@redhat.com> > > Signed-off-by: Anton Vorontsov <anton.vorontsov@linaro.org> > > Are these last 4 patches independant of the first 2 and can be taken > through the staging tree now?
Yep, without the first two there is just a bit of sparse warnings. Not a big deal.
Thanks,
-- Anton Vorontsov Email: cbouatmailru@gmail.com
| |