Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-30 Thread Minchan Kim
On Wed, Jan 30, 2013 at 05:21:12PM +0900, Minchan Kim wrote: > Hi Greg, > > On Tue, Jan 29, 2013 at 11:20:06PM -0500, Greg Kroah-Hartman wrote: > > On Mon, Jan 28, 2013 at 09:38:23AM +0900, Minchan Kim wrote: > > > Now zram allocates new page with GFP_KERNEL in zram I/O path > > > if IO is partial

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-30 Thread Minchan Kim
Hi Greg, On Tue, Jan 29, 2013 at 11:20:06PM -0500, Greg Kroah-Hartman wrote: > On Mon, Jan 28, 2013 at 09:38:23AM +0900, 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 patc

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-30 Thread Greg Kroah-Hartman
On Mon, Jan 28, 2013 at 09:38:23AM +0900, 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: sta...@vger.kernel.org > Cc: Jerome Marchand > Acked-b

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-29 Thread Minchan Kim
On Tue, Jan 29, 2013 at 09:11:08AM +0200, Pekka Enberg wrote: > On Tue, Jan 29, 2013 at 1:21 AM, Minchan Kim wrote: > > How about this? > > - >8 --- > > > > From 9f8756ae0b0f2819f93cb94dcd38da372843aa12 Mon Sep 17 00:00:00 2001 > > From: Minchan

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-28 Thread Pekka Enberg
On Tue, Jan 29, 2013 at 1:21 AM, Minchan Kim wrote: > How about this? > - >8 --- > > From 9f8756ae0b0f2819f93cb94dcd38da372843aa12 Mon Sep 17 00:00:00 2001 > From: Minchan Kim > Date: Mon, 21 Jan 2013 13:58:52 +0900 > Subject: [RESEND PATCH v5

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-28 Thread Minchan Kim
On Mon, Jan 28, 2013 at 09:16:35AM +0200, Pekka Enberg wrote: > On Mon, Jan 28, 2013 at 2:38 AM, 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 > > s/cuase/cause/g Thanks! > > > reclaim path so

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-28 Thread Jerome Marchand
On 01/28/2013 02:26 PM, Pekka Enberg wrote: > On Mon, Jan 28, 2013 at 1:24 PM, Jerome Marchand wrote: >> On 01/28/2013 08:16 AM, Pekka Enberg wrote: >>> On Mon, Jan 28, 2013 at 2:38 AM, Minchan Kim wrote: Now zram allocates new page with GFP_KERNEL in zram I/O path if IO is partial. Unf

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-28 Thread Pekka Enberg
On Mon, Jan 28, 2013 at 1:24 PM, Jerome Marchand wrote: > On 01/28/2013 08:16 AM, Pekka Enberg wrote: >> On Mon, Jan 28, 2013 at 2:38 AM, 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 >> >> s/cua

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-28 Thread Jerome Marchand
On 01/28/2013 08:16 AM, Pekka Enberg wrote: > On Mon, Jan 28, 2013 at 2:38 AM, 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 > > s/cuase/cause/g > >> reclaim path so this patch solves the problem.

Re: [RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-27 Thread Pekka Enberg
On Mon, Jan 28, 2013 at 2:38 AM, 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 s/cuase/cause/g > reclaim path so this patch solves the problem. It'd be nice to know about the problem in more detail

[RESEND PATCH v5 1/4] zram: Fix deadlock bug in partial write

2013-01-27 Thread Minchan Kim
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: sta...@vger.kernel.org Cc: Jerome Marchand Acked-by: Nitin Gupta Signed-off-by: Minchan Kim --- drivers/staging/zram/zram_