Re: false positive lockdep splat with loop device

2017-10-10 Thread Ilya Dryomov
On Tue, Oct 10, 2017 at 11:16 AM, Ilya Dryomov wrote: > On Thu, Oct 5, 2017 at 6:33 PM, Christoph Hellwig wrote: >> Does the patch below fix the warning for you? >> >> -- >> From 28aae7104425433d39e6142adcd5b88dc5b0ad5f Mon Sep 17 00:00:00 2001 >> From: Christoph Hellwig >> Date: Thu, 5 Oct 2017

Re: false positive lockdep splat with loop device

2017-10-10 Thread Ilya Dryomov
On Thu, Oct 5, 2017 at 6:33 PM, Christoph Hellwig wrote: > Does the patch below fix the warning for you? > > -- > From 28aae7104425433d39e6142adcd5b88dc5b0ad5f Mon Sep 17 00:00:00 2001 > From: Christoph Hellwig > Date: Thu, 5 Oct 2017 18:31:02 +0200 > Subject: block: use DECLARE_COMPLETION_ONSTAC

Re: false positive lockdep splat with loop device

2017-10-05 Thread Christoph Hellwig
Does the patch below fix the warning for you? -- >From 28aae7104425433d39e6142adcd5b88dc5b0ad5f Mon Sep 17 00:00:00 2001 From: Christoph Hellwig Date: Thu, 5 Oct 2017 18:31:02 +0200 Subject: block: use DECLARE_COMPLETION_ONSTACK in submit_bio_wait This way we get our lockdep annotations right in

Re: false positive lockdep splat with loop device

2017-09-26 Thread Amir Goldstein
On Tue, Sep 26, 2017 at 7:24 AM, Dave Chinner wrote: > On Thu, Sep 21, 2017 at 09:43:41AM +0300, Amir Goldstein wrote: >> On Thu, Sep 21, 2017 at 1:22 AM, Dave Chinner wrote: >> > [cc lkml, PeterZ and Byungchul] >> ... >> > The thing is, this IO completion has nothing to do with the lower >> > fi

Re: false positive lockdep splat with loop device

2017-09-25 Thread Dave Chinner
On Thu, Sep 21, 2017 at 09:43:41AM +0300, Amir Goldstein wrote: > On Thu, Sep 21, 2017 at 1:22 AM, Dave Chinner wrote: > > [cc lkml, PeterZ and Byungchul] > ... > > The thing is, this IO completion has nothing to do with the lower > > filesystem - it's the IO completion for the filesystem on the l

Re: false positive lockdep splat with loop device

2017-09-20 Thread Amir Goldstein
On Thu, Sep 21, 2017 at 1:22 AM, Dave Chinner wrote: > [cc lkml, PeterZ and Byungchul] ... > The thing is, this IO completion has nothing to do with the lower > filesystem - it's the IO completion for the filesystem on the loop > device (the upper filesystem) and is not in any way related to the >