From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx127.postini.com [74.125.245.127]) by kanga.kvack.org (Postfix) with SMTP id 3A37D6B0005 for ; Tue, 22 Jan 2013 13:02:15 -0500 (EST) Received: by mail-da0-f53.google.com with SMTP id x6so3338505dac.26 for ; Tue, 22 Jan 2013 10:02:14 -0800 (PST) Message-ID: <50FED422.4020508@vflare.org> Date: Tue, 22 Jan 2013 10:02:10 -0800 From: Nitin Gupta MIME-Version: 1.0 Subject: Re: [PATCH v4 1/4] zram: Fix deadlock bug in partial write References: <1358813253-20913-1-git-send-email-minchan@kernel.org> In-Reply-To: <1358813253-20913-1-git-send-email-minchan@kernel.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Minchan Kim Cc: Greg Kroah-Hartman , Seth Jennings , Dan Magenheimer , Konrad Rzeszutek Wilk , Jerome Marchand , Pekka Enberg , linux-mm@kvack.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org On 01/21/2013 04:07 PM, Minchan Kim wrote: > Now zram allocates new page with GFP_KERNEL in zram I/O path > if IO is partial. Unfortunately, It may cuase deadlock with > reclaim path so this patch solves the problem. > > Cc: Nitin Gupta > Cc: Jerome Marchand > Cc: stable@vger.kernel.org > Signed-off-by: Minchan Kim > --- > For the entire series: Acked-by: Nitin Gupta > We could use GFP_IO instead of GFP_ATOMIC in zram_bvec_read with > some modification related to buffer allocation in case of partial IO. > But it needs more churn and prevent merge this patch into stable > if we should send this to stable so I'd like to keep it as simple > as possbile. GFP_IO usage could be separate patch after we merge it. > Thanks. > > drivers/staging/zram/zram_drv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/zram/zram_drv.c b/drivers/staging/zram/zram_drv.c > index 61fb8f1..b285b3a 100644 > --- a/drivers/staging/zram/zram_drv.c > +++ b/drivers/staging/zram/zram_drv.c > @@ -220,7 +220,7 @@ static int zram_bvec_read(struct zram *zram, struct bio_vec *bvec, > user_mem = kmap_atomic(page); > if (is_partial_io(bvec)) > /* Use a temporary buffer to decompress the page */ > - uncmem = kmalloc(PAGE_SIZE, GFP_KERNEL); > + uncmem = kmalloc(PAGE_SIZE, GFP_ATOMIC); > else > uncmem = user_mem; > > @@ -268,7 +268,7 @@ static int zram_bvec_write(struct zram *zram, struct bio_vec *bvec, u32 index, > * This is a partial IO. We need to read the full page > * before to write the changes. > */ > - uncmem = kmalloc(PAGE_SIZE, GFP_KERNEL); > + uncmem = kmalloc(PAGE_SIZE, GFP_NOIO); > if (!uncmem) { > pr_info("Error allocating temp memory!\n"); > ret = -ENOMEM; > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org