[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: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/2] tests: add file-write-read test |
Date: |
Wed, 25 Nov 2015 10:46:08 -0600 |
User-agent: |
alot/0.3.6 |
Quoting Eric Blake (2015-11-25 10:41:35)
> On 11/25/2015 09:21 AM, Michael Roth wrote:
>
> >>> + /* 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.
> >
> > I was planning on pulling in your patch on top of this for the next
> > 2.5 pull, so rebasing on top of this series is probably best.
>
> Okay, will do, and will do quickly so I'm not holding up your pull request.
Thanks! FYI I now have this series applied here if you'd like to base
on that:
https://github.com/mdroth/qemu/commits/qga
>
> --
> Eric Blake eblake redhat com +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>
- [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, 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 <=
- 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