Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-11 Thread alex chen
On 2017/12/8 18:04, Changwei Ge wrote: > On 2017/12/8 14:21, alex chen wrote: >> >> >> On 2017/12/8 13:36, Ben Hutchings wrote: >>> On Fri, 2017-12-08 at 12:03 +0800, alex chen wrote: On 2017/12/8 10:26, Ben Hutchings wrote: > On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote:

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-08 Thread Changwei Ge
On 2017/12/8 14:21, alex chen wrote: > > > On 2017/12/8 13:36, Ben Hutchings wrote: >> On Fri, 2017-12-08 at 12:03 +0800, alex chen wrote: >>> >>> On 2017/12/8 10:26, Ben Hutchings wrote: On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote: > > On 2017/12/8 2:25, Ben Hutchings wrote:

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-07 Thread alex chen
On 2017/12/8 13:36, Ben Hutchings wrote: > On Fri, 2017-12-08 at 12:03 +0800, alex chen wrote: >> >> On 2017/12/8 10:26, Ben Hutchings wrote: >>> On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote: On 2017/12/8 2:25, Ben Hutchings wrote: > On Wed, 2017-12-06 at 09:02 +0800, alex che

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-07 Thread Ben Hutchings
On Fri, 2017-12-08 at 12:03 +0800, alex chen wrote: > > On 2017/12/8 10:26, Ben Hutchings wrote: > > On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote: > > > > > > On 2017/12/8 2:25, Ben Hutchings wrote: > > > > On Wed, 2017-12-06 at 09:02 +0800, alex chen wrote: > > > > > Hi Ben, > > > > > > >

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-07 Thread alex chen
On 2017/12/8 10:26, Ben Hutchings wrote: > On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote: >> >> On 2017/12/8 2:25, Ben Hutchings wrote: >>> On Wed, 2017-12-06 at 09:02 +0800, alex chen wrote: Hi Ben, Thanks for your reply. On 2017/12/5 23:49, Ben Hutchings wrote: >>>

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-07 Thread Ben Hutchings
On Fri, 2017-12-08 at 08:39 +0800, alex chen wrote: > > On 2017/12/8 2:25, Ben Hutchings wrote: > > On Wed, 2017-12-06 at 09:02 +0800, alex chen wrote: > > > Hi Ben, > > > > > > Thanks for your reply. > > > > > > On 2017/12/5 23:49, Ben Hutchings wrote: > > > > On Wed, 2017-11-22 at 11:12 +0100,

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-07 Thread alex chen
On 2017/12/8 2:25, Ben Hutchings wrote: > On Wed, 2017-12-06 at 09:02 +0800, alex chen wrote: >> Hi Ben, >> >> Thanks for your reply. >> >> On 2017/12/5 23:49, Ben Hutchings wrote: >>> On Wed, 2017-11-22 at 11:12 +0100, Greg Kroah-Hartman wrote: 4.4-stable review patch. If anyone has any ob

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-07 Thread Ben Hutchings
On Wed, 2017-12-06 at 09:02 +0800, alex chen wrote: > Hi Ben, > > Thanks for your reply. > > On 2017/12/5 23:49, Ben Hutchings wrote: > > On Wed, 2017-11-22 at 11:12 +0100, Greg Kroah-Hartman wrote: > > > 4.4-stable review patch.  If anyone has any objections, please let me > > > know. > > > >

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-06 Thread Greg Kroah-Hartman
On Wed, Dec 06, 2017 at 09:02:42AM +0800, alex chen wrote: > Hi Ben, > > Thanks for your reply. > > On 2017/12/5 23:49, Ben Hutchings wrote: > > On Wed, 2017-11-22 at 11:12 +0100, Greg Kroah-Hartman wrote: > >> 4.4-stable review patch. If anyone has any objections, please let me know. > >> > >>

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-05 Thread alex chen
Hi Ben, Thanks for your reply. On 2017/12/5 23:49, Ben Hutchings wrote: > On Wed, 2017-11-22 at 11:12 +0100, Greg Kroah-Hartman wrote: >> 4.4-stable review patch. If anyone has any objections, please let me know. >> >> -- >> >> From: alex chen >> >> commit 28f5a8a7c033cbf3e32277

Re: [PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-12-05 Thread Ben Hutchings
On Wed, 2017-11-22 at 11:12 +0100, Greg Kroah-Hartman wrote: > 4.4-stable review patch.  If anyone has any objections, please let me know. > > -- > > From: alex chen > > commit 28f5a8a7c033cbf3e32277f4cc9c6afd74f05300 upstream. > > we should wait dio requests to finish before i

[PATCH 4.4 13/16] ocfs2: should wait dio before inode lock in ocfs2_setattr()

2017-11-22 Thread Greg Kroah-Hartman
4.4-stable review patch. If anyone has any objections, please let me know. -- From: alex chen commit 28f5a8a7c033cbf3e32277f4cc9c6afd74f05300 upstream. we should wait dio requests to finish before inode lock in ocfs2_setattr(), otherwise the following deadlock will happen: pr