xfs
[Top] [All Lists]

Re: xfs_file_splice_read: possible circular locking dependency detected

To: Dave Chinner <david@xxxxxxxxxxxxx>, Al Viro <viro@xxxxxxxxxxxxxxxxxx>
Subject: Re: xfs_file_splice_read: possible circular locking dependency detected
From: CAI Qian <caiqian@xxxxxxxxxx>
Date: Thu, 8 Sep 2016 11:29:11 -0400 (EDT)
Cc: linux-xfs <linux-xfs@xxxxxxxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20160903003919.GI30056@dastard>
References: <723420070.1340881.1472835555274.JavaMail.zimbra@xxxxxxxxxx> <1832555471.1341372.1472835736236.JavaMail.zimbra@xxxxxxxxxx> <20160903003919.GI30056@dastard>
Thread-index: aEiZXDuBUrt7HIGBM0f2twqdeYYVUg==
Thread-topic: xfs_file_splice_read: possible circular locking dependency detected

----- Original Message -----
> From: "Dave Chinner" <david@xxxxxxxxxxxxx>
> To: "CAI Qian" <caiqian@xxxxxxxxxx>
> Cc: "linux-xfs" <linux-xfs@xxxxxxxxxxxxxxx>, "Linus Torvalds" 
> <torvalds@xxxxxxxxxxxxxxxxxxxx>, "Al Viro"
> <viro@xxxxxxxxxxxxxxxxxx>, xfs@xxxxxxxxxxx
> Sent: Friday, September 2, 2016 8:39:19 PM
> Subject: Re: xfs_file_splice_read: possible circular locking dependency 
> detected
> 
> On Fri, Sep 02, 2016 at 01:02:16PM -0400, CAI Qian wrote:
> > Spice seems start to deadlock using the reproducer,
> > 
> > https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/syscalls/splice/splice01.c
> > 
> > This seems introduced recently after v4.8-rc3 or -rc4, so suspect this xfs
> > update was one to blame,
> > 
> > 7d1ce606a37922879cbe40a6122047827105a332
> 
> Nope, this goes back to the splice rework back around ~3.16, IIRC.
Right. FYI, revert the commit below fixes the regression,

8d02076 : ->splice_write() via ->write_iter()

   CAI Qian

<Prev in Thread] Current Thread [Next in Thread>