[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test |
Date: |
Wed, 25 Nov 2015 09:02:55 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
On 11/25/2015 05:59 AM, address@hidden wrote:
> From: Marc-André Lureau <address@hidden>
>
> This test exhibits a POSIX behaviour regarding switching between write
> and read. It's undefined result if the application doesn't ensure a
> flush between the two operations (with glibc, the flush can be implicit
> when the buffer size is relatively small). The previous commit fixes
> this test.
>
> Related to:
> https://bugzilla.redhat.com/show_bug.cgi?id=1210246
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> tests/test-qga.c | 95
> ++++++++++++++++++++++++++++++++++++++++++++++++++++++--
> 1 file changed, 93 insertions(+), 2 deletions(-)
Reviewed-by: Eric Blake <address@hidden>
> + /* seek to 0 */
> + cmd = g_strdup_printf("{'execute': 'guest-file-seek',"
> + " 'arguments': { 'handle': %" PRId64 ", "
> + " 'offset': %d, 'whence': %d } }",
> + id, 0, SEEK_SET);
We still have a conflict between this series and my proposal to codify 0
rather than relying on platform-specific SEEK_SET; Markus had the
suggestion of using QGA_SET (or QGA_SEEK_SET). Are we trying to get
both your series and my v2 patch into 2.5? Knowing that will help me
decide whether my v2 should be rebased on top of your patches.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v3 0/2] qga: flush explicitly when needed, marcandre . lureau, 2015/11/25
- [Qemu-devel] [PATCH v3 1/2] qga: flush explicitly when needed, marcandre . lureau, 2015/11/25
- [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, marcandre . lureau, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test,
Eric Blake <=
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Michael Roth, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Eric Blake, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Michael Roth, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Eric Blake, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Michael Roth, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Eric Blake, 2015/11/25
- Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test, Michael Roth, 2015/11/25
Re: [Qemu-devel] [PATCH v3 0/2] qga: flush explicitly when needed, Michael Roth, 2015/11/25