[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handlin
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handling of SEEK_DATA failure |
Date: |
Thu, 13 Nov 2014 07:52:10 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 11/13/2014 06:03 AM, Kevin Wolf wrote:
> Am 13.11.2014 um 11:17 hat Markus Armbruster geschrieben:
>> When SEEK_HOLE tells us we're in a hole, we try SEEK_DATA to find its
>> end. When that fails, we pretend the hole extends to the end of file.
>> Wrong.
>
> Wrong only in some cases, see below.
>
>> Except when SEEK_END fails, we screw up and claim it extends
>> to offset -1. More wrong.
>> +++ b/block/raw-posix.c
>> @@ -1494,8 +1494,9 @@ static int try_seek_hole(BlockDriverState *bs, off_t
>> start, off_t *data,
>> } else {
>> /* On a hole. We need another syscall to find its end. */
>> *data = lseek(s->fd, start, SEEK_DATA);
>> - if (*data == -1) {
>> - *data = lseek(s->fd, 0, SEEK_END);
>> + if (*data < 0) {
>> + /* no idea where the hole ends, give up (unlikely to happen) */
>
> Not quite unlikely. If the file ends with a sparse area, we'll get
> -1/ENXIO here.
>
> lseek() with SEEK_DATA starting in a hole when there is no data until
> EOF is actually the part that isn't documented in the man page, but
> ENXIO is what I'm seeing here on RHEL 7.
Here's the (proposed) POSIX wording:
http://austingroupbugs.net/view.php?id=415
And ENXIO is indeed the expected error for SEEK_DATA on a trailing hole,
so maybe we should special case it.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 0/4] raw-posix: Get rid of FIEMAP, and more, Markus Armbruster, 2014/11/13
- [Qemu-devel] [PATCH v2 2/4] raw-posix: SEEK_HOLE suffices, get rid of FIEMAP, Markus Armbruster, 2014/11/13
- [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handling of SEEK_DATA failure, Markus Armbruster, 2014/11/13
- Re: [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handling of SEEK_DATA failure, Eric Blake, 2014/11/13
- Re: [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handling of SEEK_DATA failure, Eric Blake, 2014/11/13
- Re: [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handling of SEEK_DATA failure, Markus Armbruster, 2014/11/14
- Re: [Qemu-devel] [PATCH v2 3/4] raw-posix: Fix try_seek_hole()'s handling of SEEK_DATA failure, Eric Blake, 2014/11/14
[Qemu-devel] [PATCH v2 4/4] raw-posix: Clean up around raw_co_get_block_status(), Markus Armbruster, 2014/11/13