Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

Hi Mark, On 12/15/2015 03:18 AM, Mark Fasheh wrote: > On Mon, Dec 14, 2015 at 02:03:17PM +0800, Junxiao Bi wrote: >>> Second, this issue can be reproduced in old Linux kernels (e.g. 3.16.7-24)? >>> there should not be any regression issue? >> Maybe just hard to reproduce, ocfs2 supports

Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

On 12/14/2015 04:44 PM, Eric Ren wrote: > Hi Junxiao, > > On Mon, Dec 14, 2015 at 09:57:38AM +0800, Junxiao Bi wrote: >> The following locking order can cause a deadlock. >> Process A on Node X: Process B on Node Y: >> lock_XYZ(PR) >>

Re: [Ocfs2-devel] [PATCH 0/8] ocfs2: fix ocfs2 direct io code patch to support sparse file and data ordering semantics

Hi Ryan, On 2015/12/14 13:31, Ryan Ding wrote: > Hi Joseph, > > On 12/10/2015 06:36 PM, Joseph Qi wrote: >> Hi Ryan, >> >> On 2015/12/10 16:48, Ryan Ding wrote: >>> Hi Joseph, >>> >>> Thanks for your comments, please see my reply: >>> >>> On 12/10/2015 03:54 PM, Joseph Qi wrote: Hi Ryan,

Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

Hi, On Mon, Dec 14, 2015 at 05:02:26PM +0800, Junxiao Bi wrote: > On 12/14/2015 04:44 PM, Eric Ren wrote: > > Hi Junxiao, > > > > On Mon, Dec 14, 2015 at 09:57:38AM +0800, Junxiao Bi wrote: > >> The following locking order can cause a deadlock. > >> Process A on Node X: Process

Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

> 在 2015年12月14日，下午4:57，Eric Ren 写道： > > Hi, > > On Mon, Dec 14, 2015 at 02:03:17PM +0800, Junxiao Bi wrote: >> On 12/14/2015 01:39 PM, Gang He wrote: >>> Hello Junxiao, >>> >>> From the initial description, the second lock_XYZ(PR) should be blocked, >>> since DLM have a fair

Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

On Mon, Dec 14, 2015 at 02:03:17PM +0800, Junxiao Bi wrote: > > Second, this issue can be reproduced in old Linux kernels (e.g. 3.16.7-24)? > > there should not be any regression issue? > Maybe just hard to reproduce, ocfs2 supports recursive locking. In what sense? The DLM might but the FS

Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

Hi Junxiao, On Mon, Dec 14, 2015 at 09:57:38AM +0800, Junxiao Bi wrote: > The following locking order can cause a deadlock. > Process A on Node X: Process B on Node Y: > lock_XYZ(PR) > lock_XYZ(EX) > lock_XYZ(PR) >>> blocked forever by

Re: [Ocfs2-devel] [PATCH] ocfs2: dlm: fix recursive locking deadlock

Hi, On Mon, Dec 14, 2015 at 02:03:17PM +0800, Junxiao Bi wrote: > On 12/14/2015 01:39 PM, Gang He wrote: > > Hello Junxiao, > > > > From the initial description, the second lock_XYZ(PR) should be blocked, > > since DLM have a fair queue mechanism, otherwise, it looks to bring a > > write