Re: possible deadlock in generic_file_write_iter (2)

2017-12-07 Thread Byungchul Park
On 12/8/2017 2:07 AM, Dmitry Vyukov wrote: On Wed, Dec 6, 2017 at 6:05 AM, Byungchul Park wrote: On 12/4/2017 5:33 PM, Jan Kara wrote: Hello, adding Peter and Byungchul to CC since the lockdep report just looks strange and cross-release seems to be involved. Guys,

Re: possible deadlock in generic_file_write_iter (2)

2017-12-07 Thread Byungchul Park
On 12/8/2017 2:07 AM, Dmitry Vyukov wrote: On Wed, Dec 6, 2017 at 6:05 AM, Byungchul Park wrote: On 12/4/2017 5:33 PM, Jan Kara wrote: Hello, adding Peter and Byungchul to CC since the lockdep report just looks strange and cross-release seems to be involved. Guys, how did #5 get into the

Re: possible deadlock in generic_file_write_iter (2)

2017-12-07 Thread Dmitry Vyukov
On Wed, Dec 6, 2017 at 6:05 AM, Byungchul Park wrote: >> > On 12/4/2017 5:33 PM, Jan Kara wrote: >> >> >> >> Hello, >> >> >> >> adding Peter and Byungchul to CC since the lockdep report just looks >> >> strange and cross-release seems to be involved. Guys, how did #5 get

Re: possible deadlock in generic_file_write_iter (2)

2017-12-07 Thread Dmitry Vyukov
On Wed, Dec 6, 2017 at 6:05 AM, Byungchul Park wrote: >> > On 12/4/2017 5:33 PM, Jan Kara wrote: >> >> >> >> Hello, >> >> >> >> adding Peter and Byungchul to CC since the lockdep report just looks >> >> strange and cross-release seems to be involved. Guys, how did #5 get into >> >> the lock chain

Re: possible deadlock in generic_file_write_iter (2)

2017-12-06 Thread Andrew Morton
On Wed, 6 Dec 2017 14:05:47 +0900 Byungchul Park wrote: > > What is cross-release? Is it something new? Should we always enable > > crossrelease_fullstack during testing? > > Hello Dmitry, > > Yes, it's new one making lockdep track wait_for_completion() as well. > >

Re: possible deadlock in generic_file_write_iter (2)

2017-12-06 Thread Andrew Morton
On Wed, 6 Dec 2017 14:05:47 +0900 Byungchul Park wrote: > > What is cross-release? Is it something new? Should we always enable > > crossrelease_fullstack during testing? > > Hello Dmitry, > > Yes, it's new one making lockdep track wait_for_completion() as well. > > And we should enable

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Byungchul Park
On Tue, Dec 05, 2017 at 10:41:50AM +0100, Jan Kara wrote: > > Hello Byungchul, > > On Tue 05-12-17 13:58:09, Byungchul Park wrote: > > On 12/4/2017 5:33 PM, Jan Kara wrote: > > >adding Peter and Byungchul to CC since the lockdep report just looks > > >strange and cross-release seems to be

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Byungchul Park
On Tue, Dec 05, 2017 at 10:41:50AM +0100, Jan Kara wrote: > > Hello Byungchul, > > On Tue 05-12-17 13:58:09, Byungchul Park wrote: > > On 12/4/2017 5:33 PM, Jan Kara wrote: > > >adding Peter and Byungchul to CC since the lockdep report just looks > > >strange and cross-release seems to be

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Byungchul Park
On Tue, Dec 05, 2017 at 10:19:07AM +0100, Dmitry Vyukov wrote: > On Tue, Dec 5, 2017 at 5:58 AM, Byungchul Park wrote: > > On 12/4/2017 5:33 PM, Jan Kara wrote: > >> > >> Hello, > >> > >> adding Peter and Byungchul to CC since the lockdep report just looks > >> strange and

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Byungchul Park
On Tue, Dec 05, 2017 at 10:19:07AM +0100, Dmitry Vyukov wrote: > On Tue, Dec 5, 2017 at 5:58 AM, Byungchul Park wrote: > > On 12/4/2017 5:33 PM, Jan Kara wrote: > >> > >> Hello, > >> > >> adding Peter and Byungchul to CC since the lockdep report just looks > >> strange and cross-release seems to

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Jan Kara
Hello Byungchul, On Tue 05-12-17 13:58:09, Byungchul Park wrote: > On 12/4/2017 5:33 PM, Jan Kara wrote: > >adding Peter and Byungchul to CC since the lockdep report just looks > >strange and cross-release seems to be involved. Guys, how did #5 get into > >the lock chain and what does

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Jan Kara
Hello Byungchul, On Tue 05-12-17 13:58:09, Byungchul Park wrote: > On 12/4/2017 5:33 PM, Jan Kara wrote: > >adding Peter and Byungchul to CC since the lockdep report just looks > >strange and cross-release seems to be involved. Guys, how did #5 get into > >the lock chain and what does

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Dmitry Vyukov
On Tue, Dec 5, 2017 at 5:58 AM, Byungchul Park wrote: > On 12/4/2017 5:33 PM, Jan Kara wrote: >> >> Hello, >> >> adding Peter and Byungchul to CC since the lockdep report just looks >> strange and cross-release seems to be involved. Guys, how did #5 get into >> the lock

Re: possible deadlock in generic_file_write_iter (2)

2017-12-05 Thread Dmitry Vyukov
On Tue, Dec 5, 2017 at 5:58 AM, Byungchul Park wrote: > On 12/4/2017 5:33 PM, Jan Kara wrote: >> >> Hello, >> >> adding Peter and Byungchul to CC since the lockdep report just looks >> strange and cross-release seems to be involved. Guys, how did #5 get into >> the lock chain and what does

Re: possible deadlock in generic_file_write_iter (2)

2017-12-04 Thread Byungchul Park
On 12/4/2017 5:33 PM, Jan Kara wrote: Hello, adding Peter and Byungchul to CC since the lockdep report just looks strange and cross-release seems to be involved. Guys, how did #5 get into the lock chain and what does put_ucounts() have to do with sb_writers there? Thanks! Hello Jan, In order

Re: possible deadlock in generic_file_write_iter (2)

2017-12-04 Thread Byungchul Park
On 12/4/2017 5:33 PM, Jan Kara wrote: Hello, adding Peter and Byungchul to CC since the lockdep report just looks strange and cross-release seems to be involved. Guys, how did #5 get into the lock chain and what does put_ucounts() have to do with sb_writers there? Thanks! Hello Jan, In order