Re: [PATCH 2/2] zram: allocate metadata when disksize is set up

2012-11-27 Thread Minchan Kim
On Tue, Nov 27, 2012 at 04:45:54PM +0100, Jerome Marchand wrote: > On 11/27/2012 06:13 AM, Nitin Gupta wrote: > > On 11/22/2012 06:42 PM, Minchan Kim wrote: > >> Lockdep complains about recursive deadlock of zram->init_lock. > >> Because zram_init_device could be called in reclaim context and > >>

Re: [PATCH 2/2] zram: allocate metadata when disksize is set up

2012-11-27 Thread Minchan Kim
On Mon, Nov 26, 2012 at 09:13:24PM -0800, Nitin Gupta wrote: > On 11/22/2012 06:42 PM, Minchan Kim wrote: > >Lockdep complains about recursive deadlock of zram->init_lock. > >Because zram_init_device could be called in reclaim context and > >it requires a page with GFP_KERNEL. > > > >We can fix it

Re: [PATCH 2/2] zram: allocate metadata when disksize is set up

2012-11-27 Thread Jerome Marchand
On 11/27/2012 06:13 AM, Nitin Gupta wrote: > On 11/22/2012 06:42 PM, Minchan Kim wrote: >> Lockdep complains about recursive deadlock of zram->init_lock. >> Because zram_init_device could be called in reclaim context and >> it requires a page with GFP_KERNEL. >> >> We can fix it via replacing GFP_K

Re: [PATCH 2/2] zram: allocate metadata when disksize is set up

2012-11-26 Thread Nitin Gupta
On 11/22/2012 06:42 PM, Minchan Kim wrote: Lockdep complains about recursive deadlock of zram->init_lock. Because zram_init_device could be called in reclaim context and it requires a page with GFP_KERNEL. We can fix it via replacing GFP_KERNEL with GFP_NOIO. But more big problem is vzalloc in z

[PATCH 2/2] zram: allocate metadata when disksize is set up

2012-11-22 Thread Minchan Kim
Lockdep complains about recursive deadlock of zram->init_lock. Because zram_init_device could be called in reclaim context and it requires a page with GFP_KERNEL. We can fix it via replacing GFP_KERNEL with GFP_NOIO. But more big problem is vzalloc in zram_init_device which calls GFP_KERNEL. We ca