Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-30 Thread Minchan Kim
Hi Zlatko, On Fri, Dec 28, 2012 at 02:29:11PM +0100, Zlatko Calusic wrote: > On 28.12.2012 03:49, Minchan Kim wrote: > >Hello Zlatko, > > > >On Fri, Dec 28, 2012 at 03:16:38AM +0100, Zlatko Calusic wrote: > >>From: Zlatko Calusic > >> > >>The unintended consequence of commit 4ae0a48b is that >

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-30 Thread Minchan Kim
Hi Zlatko, On Fri, Dec 28, 2012 at 02:29:11PM +0100, Zlatko Calusic wrote: On 28.12.2012 03:49, Minchan Kim wrote: Hello Zlatko, On Fri, Dec 28, 2012 at 03:16:38AM +0100, Zlatko Calusic wrote: From: Zlatko Calusic zlatko.calu...@iskon.hr The unintended consequence of commit 4ae0a48b is

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-29 Thread Sedat Dilek
Just FYI: This patch landed upstream [1]. Thanks for all involved people. - Sedat - [1] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=ecccd1248d6e6986130ffcc3b0d003cb46a485c0 On Fri, Dec 28, 2012 at 3:16 AM, Zlatko Calusic wrote: > From: Zlatko Calusic > > The

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-29 Thread Sedat Dilek
Just FYI: This patch landed upstream [1]. Thanks for all involved people. - Sedat - [1] http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=ecccd1248d6e6986130ffcc3b0d003cb46a485c0 On Fri, Dec 28, 2012 at 3:16 AM, Zlatko Calusic zlatko.calu...@iskon.hr wrote: From:

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-28 Thread Zlatko Calusic
On 28.12.2012 03:49, Minchan Kim wrote: Hello Zlatko, On Fri, Dec 28, 2012 at 03:16:38AM +0100, Zlatko Calusic wrote: From: Zlatko Calusic The unintended consequence of commit 4ae0a48b is that wait_iff_congested() can now be called with NULL struct zone* producing kernel oops like this:

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-28 Thread Zlatko Calusic
On 28.12.2012 03:49, Minchan Kim wrote: Hello Zlatko, On Fri, Dec 28, 2012 at 03:16:38AM +0100, Zlatko Calusic wrote: From: Zlatko Calusic zlatko.calu...@iskon.hr The unintended consequence of commit 4ae0a48b is that wait_iff_congested() can now be called with NULL struct zone* producing

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-27 Thread Minchan Kim
Hello Zlatko, On Fri, Dec 28, 2012 at 03:16:38AM +0100, Zlatko Calusic wrote: > From: Zlatko Calusic > > The unintended consequence of commit 4ae0a48b is that > wait_iff_congested() can now be called with NULL struct zone* > producing kernel oops like this: For good description, it would be

[PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-27 Thread Zlatko Calusic
From: Zlatko Calusic The unintended consequence of commit 4ae0a48b is that wait_iff_congested() can now be called with NULL struct zone* producing kernel oops like this: BUG: unable to handle kernel NULL pointer dereference IP: [] wait_iff_congested+0x59/0x140 This trivial patch fixes it.

[PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-27 Thread Zlatko Calusic
From: Zlatko Calusic zlatko.calu...@iskon.hr The unintended consequence of commit 4ae0a48b is that wait_iff_congested() can now be called with NULL struct zone* producing kernel oops like this: BUG: unable to handle kernel NULL pointer dereference IP: [811542d9]

Re: [PATCH] mm: fix null pointer dereference in wait_iff_congested()

2012-12-27 Thread Minchan Kim
Hello Zlatko, On Fri, Dec 28, 2012 at 03:16:38AM +0100, Zlatko Calusic wrote: From: Zlatko Calusic zlatko.calu...@iskon.hr The unintended consequence of commit 4ae0a48b is that wait_iff_congested() can now be called with NULL struct zone* producing kernel oops like this: For good