Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-10-31 Thread Al Viro
On Fri, Oct 31, 2014 at 01:57:22AM -0700, Christoph Hellwig wrote: > Could someone please pick up this regression fix? Done. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.o

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-10-31 Thread Christoph Hellwig
Could someone please pick up this regression fix? On Mon, Sep 29, 2014 at 10:21:10AM -0400, David Jeffery wrote: > Changes to the basic direct I/O code have broken the raw driver when reading > to the end of a raw device. Instead of returning a short read for a read that > extends partially beyon

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-10-11 Thread Christoph Hellwig
On Fri, Oct 10, 2014 at 01:17:28PM -0400, Jeff Moyer wrote: > Christoph, do you have any issues with the patch as it stands? Whose > tree do you think this should go through? akpm's? It looks good. I think Al's tree should be fine. -- To unsubscribe from this list: send the line "unsubscribe l

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-10-10 Thread Jeff Moyer
David Jeffery writes: > On 09/30/2014 11:28 AM, Christoph Hellwig wrote: >> On Mon, Sep 29, 2014 at 06:08:11PM -0400, David Jeffery wrote: >>> On 09/29/2014 03:05 PM, Christoph Hellwig wrote: Seems like this should be changed in the generic code, or is there some reason why it would ret

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-09-30 Thread David Jeffery
On 09/30/2014 11:28 AM, Christoph Hellwig wrote: > On Mon, Sep 29, 2014 at 06:08:11PM -0400, David Jeffery wrote: >> On 09/29/2014 03:05 PM, Christoph Hellwig wrote: >>> Seems like this should be changed in the generic code, or is there some >>> reason why it would return EIO only for devices, but

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-09-30 Thread Christoph Hellwig
On Mon, Sep 29, 2014 at 06:08:11PM -0400, David Jeffery wrote: > On 09/29/2014 03:05 PM, Christoph Hellwig wrote: > > Seems like this should be changed in the generic code, or is there some > > reason why it would return EIO only for devices, but not for regular > > files in this case? > > > > Re

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-09-29 Thread David Jeffery
On 09/29/2014 03:05 PM, Christoph Hellwig wrote: > Seems like this should be changed in the generic code, or is there some > reason why it would return EIO only for devices, but not for regular > files in this case? > Regular files shouldn't be returning EIO and don't in my tests. The file system

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-09-29 Thread Christoph Hellwig
On Mon, Sep 29, 2014 at 10:21:10AM -0400, David Jeffery wrote: > Changes to the basic direct I/O code have broken the raw driver when reading > to the end of a raw device. Instead of returning a short read for a read that > extends partially beyond the device's end or 0 when at the end of the devi

Re: [PATCH] Return short read or 0 at end of a raw device, not EIO

2014-09-29 Thread Jeff Moyer
David Jeffery writes: > Changes to the basic direct I/O code have broken the raw driver when reading > to the end of a raw device. Instead of returning a short read for a read that > extends partially beyond the device's end or 0 when at the end of the device, > these reads now return EIO. > > T