xfs
[Top] [All Lists]

Re: AIO read returns negative number for bytes read

To: Avi Kivity <avi@xxxxxxxxxxxx>
Subject: Re: AIO read returns negative number for bytes read
From: Jeff Moyer <jmoyer@xxxxxxxxxx>
Date: Mon, 16 Nov 2015 14:27:43 -0500
Cc: xfs@xxxxxxxxxxx, linux-aio@xxxxxxxxx, Steven Whitehouse <swhiteho@xxxxxxxxxx>, Jan Kara <jack@xxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <564883BD.8070607@xxxxxxxxxxxx> (Avi Kivity's message of "Sun, 15 Nov 2015 15:08:13 +0200")
References: <564883BD.8070607@xxxxxxxxxxxx>
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)
Hi Avi,

Avi Kivity <avi@xxxxxxxxxxxx> writes:

> Due to a bug in my program, I initiated a read beyond
> eof. Specifically, the file size is 13002 bytes and the read offset is
> 13312 (0x3400).
>
> I would expect such a read to return 0 bytes read, but io_getevents
> returns -310, which is suspiciously equal to (13002 - 13312).
>
> I attach a reproducer.
>
> 4.2.5-201.fc22.x86_64
>
> Are my expectations incorrect, or is this a bug in aio or xfs?

Your expectations are correct.  The bug was introduced by commit
9fe55eea7e4b4 (Fix race when checking i_size on direct i/o read).  I've
CC'd the patch author and linux-fsdevel.  I'm not sure what the right
fix is, given that the size checks were removed from the vfs to fix some
race condition.  Unfortunately, the commit message doesn't really do a
good job of explaining the race.  In order to save others time, here is
a good explanation of the problem that commit is meant to fix, along
with a reproducer:
  http://marc.info/?l=linux-fsdevel&m=138641356614458&w=2

Thanks for the great bug report, and sorry I have no solution to
proffer.

Cheers,
Jeff

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