Re: [Ocfs2-devel] [PATCH] ocfs2: add trimfs dlm lock

2017-12-07 Thread alex chen
Hi Gang, On 2017/12/8 8:35, Gang He wrote: > Hello Andrew and All, > Andrew Morton 12/08/17 7:34 AM >>> > On Thu, 7 Dec 2017 21:21:58 +0800 Gang He wrote: > >> As you know, ocfs2 has support trim the underlying disk via >> fstrim command. But

Re: [Ocfs2-devel] [PATCH] ocfs2: add trimfs dlm lock

2017-12-07 Thread Gang He
Hello Andrew and All, >>> Andrew Morton 12/08/17 7:34 AM >>> On Thu, 7 Dec 2017 21:21:58 +0800 Gang He wrote: > As you know, ocfs2 has support trim the underlying disk via > fstrim command. But there is a problem, ocfs2 is a shared storage > cluster

Re: [Ocfs2-devel] [PATCH] ocfs2: add trimfs dlm lock

2017-12-07 Thread Andrew Morton
On Thu, 7 Dec 2017 21:21:58 +0800 Gang He wrote: > As you know, ocfs2 has support trim the underlying disk via > fstrim command. But there is a problem, ocfs2 is a shared storage > cluster file system, if the user configures a scheduled fstrim > job on each file system node, this

[Ocfs2-devel] [PATCH] ocfs2: add trimfs dlm lock

2017-12-07 Thread Gang He
As you know, ocfs2 has support trim the underlying disk via fstrim command. But there is a problem, ocfs2 is a shared storage cluster file system, if the user configures a scheduled fstrim job on each file system node, this will trigger multiple nodes trim a shared disk simultaneously, it is very

Re: [Ocfs2-devel] [PATCH] ocfs2: fix a potential 'ABBA' deadlock caused by 'l_lock' and 'dentry_attach_lock'

2017-12-07 Thread Changwei Ge
Hi Jun, On 2017/12/7 19:30, piaojun wrote: > CPUA CPUB > > ocfs2_dentry_convert_worker > get 'l_lock' This lock belongs to ocfs2_dentry_lock::ocfs2_lock_res::l_lock > > get 'dentry_attach_lock' > >

Re: [Ocfs2-devel] [PATCH] ocfs2: fix a potential 'ABBA' deadlock caused by 'l_lock' and 'dentry_attach_lock'

2017-12-07 Thread piaojun
Hi Changwei, On 2017/12/7 19:59, Changwei Ge wrote: > Hi Jun, > > On 2017/12/7 19:30, piaojun wrote: >> CPUA CPUB >> >> ocfs2_dentry_convert_worker >> get 'l_lock' > > This lock belongs to ocfs2_dentry_lock::ocfs2_lock_res::l_lock > >> >>

[Ocfs2-devel] [PATCH] ocfs2: fix a potential 'ABBA' deadlock caused by 'l_lock' and 'dentry_attach_lock'

2017-12-07 Thread piaojun
CPUA CPUB ocfs2_dentry_convert_worker get 'l_lock' get 'dentry_attach_lock' interruptted by dio_end_io: dio_end_io