On Sun, Feb 7, 2016 at 2:49 PM, Greg KH <[email protected]> wrote:
> On Fri, Jan 29, 2016 at 10:07:32PM -0800, John Stultz wrote:
>> From: San Mehat <[email protected]>
>>
>> As it turns out, the CONFIG_PROFILING interfaces leak a
>> task struct if the notifier chain returns NOTIFY_OK.. doh.
>>
>> This patch reworks lowmemkiller to use the new generic task
>> free notifier chain.
>>
>> Cc: Android Kernel Team <[email protected]>
>> Cc: Greg KH <[email protected]>
>> Signed-off-by: San Mehat <[email protected]>
>> [jstultz: Commit subject tweak]
>> Signed-off-by: John Stultz <[email protected]>
>> ---
>> drivers/staging/android/lowmemorykiller.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/drivers/staging/android/lowmemorykiller.c
>> b/drivers/staging/android/lowmemorykiller.c
>> index 8b5a4a8..806643d 100644
>> --- a/drivers/staging/android/lowmemorykiller.c
>> +++ b/drivers/staging/android/lowmemorykiller.c
>> @@ -40,7 +40,6 @@
>> #include <linux/sched.h>
>> #include <linux/swap.h>
>> #include <linux/rcupdate.h>
>> -#include <linux/profile.h>
>> #include <linux/notifier.h>
>>
>> static u32 lowmem_debug_level = 1;
>> --
>> 1.9.1
>
> The changelog comment seems a bit "odd" when looking at the patch here.
> Is the patch really correct? The changelog?
Yea. Sorry for my oversight here. Had to do some patch archaeology and
found that the content of that patch basically dropped out when
83dbbdbb38 ("android, lowmemorykiller: remove task handoff notifier")
landed upstream in 3.4. Since then its been rebased along being
basically content-less.
So please ignore it. I'll try to get it dropped from the AOSP tree.
thanks
-john
