From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 9 Dec 2015 12:49:02 +0100 From: Jan Kara To: Al Viro Cc: Ross Zwisler , Jan Kara , stable@vger.kernel.org, Steven Whitehouse , Avi Kivity , Jens Axboe , linux-fsdevel@vger.kernel.org, xfs@oss.sgi.com, torvalds@linux-foundation.org Subject: Re: v4.4-rc4 regression with direct-io Message-ID: <20151209114902.GD3137@quack.suse.cz> References: <20151208213902.GA4212@linux.intel.com> <20151208215109.GR20997@ZenIV.linux.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20151208215109.GR20997@ZenIV.linux.org.uk> Sender: stable-owner@vger.kernel.org List-ID: On Tue 08-12-15 21:51:09, Al Viro wrote: > On Tue, Dec 08, 2015 at 02:39:02PM -0700, Ross Zwisler wrote: > > Hey Jan, > > > > In my test setup this commit: > > > > 74cedf9b6c60 ("direct-io: Fix negative return from dio read beyond eof") > > > > causes a regression with XFS and the test generic/113. The test used to > > pass, but with the above mentioned commit it just hangs indefinitely. > > > > This commit went in between v4.4-rc3 and v4.4-rc4, and it looks like it was > > marked for stable? > > See the tip of vfs.git#for-linus for fix... Ah, thanks for fixing this up! I'm just wondering how the test program I was using could have passed because pread() I'm doing there must have returned ENOMEM... Argh, I see. I have tested the problem with ext4 which uses DIO_LOCKING mode and in that case 'retval' gets overwritten to 0 by filemap_write_and_wait() call :-|. Honza -- Jan Kara SUSE Labs, CR